Bug 107138 – Make renderer construction take Element where possible

• Status: RESOLVED
• Resolution: FIXED
• Component: Layout and Rendering

[Antti Koivisto]

This requires changing how anonymous renderers are constructed.

[Antti Koivisto]

Created attachment 183217 [details]
patch

[Dave Hyatt]

Comment on attachment 183217 [details]
patch

r=me

[WebKit Review Bot]

Comment on attachment 183217 [details]
patch

Attachment 183217 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/15925698

[Antti Koivisto]

http://trac.webkit.org/changeset/140023

[Hin-Chung Lam]

This patch seems to failure a number of tests on Lion and Mountain Lion:

http://build.webkit.org/builders/Apple%20Lion%20Release%20WK1%20%28Tests%29/builds/8275
http://build.webkit.org/builders/Apple%20MountainLion%20Release%20WK1%20%28Tests%29/builds/5804

EWS also says these tests are failing on Mac:

Regressions: Unexpected text-only failures (29)
  accessibility/render-counter-text.html [ Failure ]
  editing/selection/5240265.html [ Failure ]
  editing/selection/css-pseudo-element.html [ Failure ]
  editing/selection/focus-crash.html [ Failure ]
  editing/selection/hit-test-anonymous.html [ Failure ]
  editing/selection/undo-crash.html [ Failure ]
  fast/css-generated-content/hover-inline.html [ Failure ]
  fast/dom/Node/mutation-blur.html [ Failure ]
  fast/events/domactivate-sets-underlying-click-event-as-handled.html [ Failure ]
  fast/events/popup-allowed-from-gesture-initiated-form-submit.html [ Failure ]
  fast/events/popup-blocked-from-untrusted-mouse-click.html [ Failure ]
  fast/forms/multiple-form-submission-protection-mouse.html [ Failure ]
  fast/forms/range/slider-delete-while-dragging-thumb.html [ Failure ]
  fast/forms/search-delete-while-cancel-button-clicked.html [ Failure ]
  fast/forms/submit-to-blank-multiple-times.html [ Failure ]
  fast/history/form-submit-in-frame-via-onclick.html [ Failure ]
  fast/history/form-submit-in-frame.html [ Failure ]
  fast/history/gesture-before-onload-form-submit.html [ Failure ]
  fast/history/history_reload.html [ Failure ]
  fast/images/animated-gif-restored-from-bfcache.html [ Failure ]
  fast/loader/javascript-url-in-object.html [ Failure ]
  http/tests/inspector/console-resource-errors.html [ Failure ]
  http/tests/security/mixedContent/redirect-https-to-http-iframe-in-main-frame.html [ Failure ]
  inspector/styles/styles-source-offsets.html [ Failure ]
  media/audio-delete-while-slider-thumb-clicked.html [ Failure ]
  media/media-fullscreen-inline.html [ Failure ]
  platform/mac/accessibility/css-content-attribute.html [ Failure ]
  svg/batik/text/smallFonts.svg [ Failure ]
  svg/dom/SVGScriptElement/script-load-and-error-events.svg [ Failure ]

[WebKit Review Bot]

Re-opened since this is blocked by bug 107176

[Antti Koivisto]

Created attachment 183532 [details]
patch2

For EWS with a RenderText fix

[WebKit Review Bot]

Attachment 183532 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1
Source/WebCore/rendering/RenderText.cpp:139:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Total errors found: 1 in 99 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[WebKit Review Bot]

Comment on attachment 183532 [details]
patch2

Attachment 183532 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/15940845

[Peter Beverloo (cr-android ews)]

Comment on attachment 183532 [details]
patch2

Attachment 183532 [details] did not pass cr-android-ews (chromium-android):
Output: http://queues.webkit.org/results/15939816

[Antti Koivisto]

http://trac.webkit.org/changeset/140244 (with chrome test build fix)