Improve logic in ensureDefaultStyleSheetsForElement

Improve logic in ensureDefaultStyleSheetsForElement

Improve logic in ensureDefaultStyleSheetsForElement, there
is no need for example to test namespace for Mathml if we
already established the element is an SVG element.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=186262

[rwlbuis, 2014-11-11 19:52:45]

rob.buis@samsung.com changed reviewers:
+ esprehn@chromium.org

[rwlbuis, 2014-11-11 19:52:45]

PTAL. BTW I don't know how those FIXMEs should be interpreted, for instance both
the mathml and svg
sheet could use the "neutral" display property, not
sure what we'd need to check...

[rwlbuis, 2014-12-01 19:15:17]

On 2014/11/11 19:52:45, rwlbuis wrote:
> PTAL. BTW I don't know how those FIXMEs should be interpreted, for instance
both
> the mathml and svg
> sheet could use the "neutral" display property, not
> sure what we'd need to check...

ping

[esprehn, 2014-12-01 19:47:39]

lgtm, the FIXME is hard to fix without some kind of compile assert.

[rwlbuis, 2014-12-01 20:56:16]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2014-12-01 20:57:11]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/708713004/1

[commit-bot: I haz the power, 2014-12-01 22:22:00]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=186262

[Ken Russell (switch to Gerrit), 2014-12-02 01:13:55]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/770103002/ by kbr@chromium.org.

The reason for reverting is: Seems to be the cause of assertion failures like:
crash log for renderer (pid 30072):
STDOUT: #CRASHED - renderer (pid 30072)
STDERR: Xlib:  extension "RANDR" missing on display ":9".
STDERR: [30072:30072:1017/090946:3443219798580:ERROR:renderer_main.cc(227)]
Running without renderer sandbox
STDERR: [30090:30090:1017/090946:3443219850178:ERROR:renderer_main.cc(227)]
Running without renderer sandbox
STDERR: ASSERTION FAILED: !value || (value->isInitialValue())
STDERR: ../../third_party/WebKit/Source/core/css/CSSLineBoxContainValue.h(59) :
const WebCore::CSSLineBoxContainValue* WebCore::toCSSLineBoxContainValue(const
WebCore::CSSValue*)


Example failing build:
http://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac10.6%20%28db...

Please file bugs for changes like this in the future so there is a permanent
record of any reverts and re-lands..