Reflected attributes should be removed when 'null', 'undefined' or 'false' is specified

This is a follow-up CL to address arv's comment in https://codereview.chromium.org/386353002/#msg9.

Note that this CL will change existing behavior for reflected DOM attributes of HTMLMarqueeElement. Previously those reflected DOM attributes were not removed when 'null' or 'undefined' is specified. This was a bug. The new behavior implemented in this CL aligns with the spec and Firefox.

I don't have a concern about compatibility since this just affects reflected attributes of HTMLMarqueeElement.

BUG=341031
TEST=fast/html/marquee-element.html

Review URL: https://codereview.chromium.org/393543002

git-svn-id: svn://svn.chromium.org/blink/trunk@178076 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove updateFirstLetter and firstLineBlock from RenderTable.

These methods were added when first letter support was initially added in
https://src.chromium.org/viewvc/blink?revision=5249&view=revision. Removing
them causes no test failures, but there does not appear to be any tests that
where added initially.

BUG=391288

Review URL: https://codereview.chromium.org/370593002

git-svn-id: svn://svn.chromium.org/blink/trunk@178077 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Oilpan: have HTMLImport::recalcTreeState() use on-heap local variables.

R=wibling@chromium.org,haraken@chromium.org
BUG=357163

Review URL: https://codereview.chromium.org/394463003

git-svn-id: svn://svn.chromium.org/blink/trunk@178078 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Revert of DevTools: added injectedScript.evaluateWithDetails, that return exception details if it occured (https://codereview.chromium.org/289423002/)

Reason for revert:
Speculative revert for failing telemetry unit tests. Tests that evaluate scripts that are purposely designed to crash the tab are no longer passing. By intersecting the Mac and windows builder regression ranges I obtained the following range: r178051-r178054.  This is the only change in that range that seems related to script execution.

Original issue's description:
> Added V8InjectedScriptHost::evaluateWithDetails. This method returns {result: evaluation result, exceptionDetails: information about the exception, if it occurred}.
> InjectedScriptSource.evaluate and InjectedScriptSource.callFunctionOn uses it. There functions returns additional field with exceptionDetails, if exception occurred.
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=178051

TBR=falken@chromium.org
NOTRY=true

Review URL: https://codereview.chromium.org/389363002

git-svn-id: svn://svn.chromium.org/blink/trunk@178079 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove NeedsRebaseline of a virtual test

Our rebaseline-o-matic script failed to handle it.

BUG=380472
TBR=wangxianzhu@chromium.org

Review URL: https://codereview.chromium.org/389393004

git-svn-id: svn://svn.chromium.org/blink/trunk@178081 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Constify metrics list references in SVGTextLayoutEngine

The text layout engine should not mutate the metrics list, so make the
various references to it const to clarify this a bit.

Review URL: https://codereview.chromium.org/390873002

git-svn-id: svn://svn.chromium.org/blink/trunk@178082 bbb929c8-8fbe-4397-9dbb-9b2b20218538