Summary: | RenderWidget doesn't need to cache a FrameView pointer. | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Andreas Kling <kling> | ||||||||
Component: | Layout and Rendering | Assignee: | Andreas Kling <kling> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | buildbot, commit-queue, esprehn+autocc, glenn, kling, kondapallykalyan, rniwa | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Andreas Kling
2013-09-11 06:52:53 PDT
Created attachment 211308 [details]
Patch
Comment on attachment 211308 [details] Patch Rejecting attachment 211308 [details] from commit-queue. New failing tests: editing/inserting/insert-paragraph-after-non-editable-node-before-text.html dom/html/level2/html/HTMLElement02.html http/tests/inspector/network/network-content-replacement-embed.html css2.1/20110323/replaced-intrinsic-002.htm compositing/geometry/empty-embed-rects.html css3/flexbox/flexitem.html compositing/plugins/1x1-composited-plugin.html http/tests/inspector/inspect-element.html accessibility/plugin.html css2.1/20110323/replaced-intrinsic-004.htm editing/execCommand/crash-extend-selection-forward.html dom/xhtml/level2/html/HTMLElement01.xhtml dom/html/level2/html/HTMLElement03.html http/tests/cache/cancel-during-revalidation-succeeded.html http/tests/inspector/network/network-embed.html compositing/plugins/composited-plugin.html editing/execCommand/remove-format-elements.html editing/editability/ignored-content.html editing/pasteboard/paste-noplugin.html compositing/plugins/small-to-large-composited-plugin.html css2.1/20110323/replaced-intrinsic-003.htm compositing/plugins/no-backing-store.html css2.1/20110323/replaced-intrinsic-001.htm compositing/iframes/iframe-position-absolute-with-padding-percentage-crash.html compositing/geometry/object-clip-rects-assertion.html compositing/plugins/large-to-small-composited-plugin.html dom/html/level2/html/AppletsCollection.html editing/selection/legal-positions.html dom/html/level2/html/HTMLElement01.html editing/pasteboard/pasting-tabs.html Full output: http://webkit-queues.appspot.com/results/1737853 Created attachment 211352 [details]
Archive of layout-test-results from webkit-cq-02 for mac-mountainlion
The attached test failures were seen while running run-webkit-tests on the commit-queue.
Bot: webkit-cq-02 Port: mac-mountainlion Platform: Mac OS X 10.8.4
Comment on attachment 211308 [details] Patch Attachment 211308 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/1811028 New failing tests: editing/inserting/insert-paragraph-after-non-editable-node-before-text.html dom/html/level2/html/HTMLElement02.html http/tests/inspector/network/network-content-replacement-embed.html css2.1/20110323/replaced-intrinsic-002.htm compositing/geometry/empty-embed-rects.html css3/flexbox/flexitem.html compositing/plugins/1x1-composited-plugin.html accessibility/plugin.html editing/execCommand/crash-extend-selection-forward.html dom/xhtml/level2/html/HTMLElement01.xhtml editing/inserting/return-with-object-element.html dom/html/level2/html/HTMLElement03.html http/tests/cache/cancel-during-revalidation-succeeded.html http/tests/inspector/network/network-embed.html dom/xhtml/level2/html/HTMLElement03.xhtml compositing/plugins/composited-plugin.html editing/execCommand/remove-format-elements.html editing/editability/ignored-content.html editing/pasteboard/paste-noplugin.html compositing/plugins/small-to-large-composited-plugin.html css2.1/20110323/replaced-intrinsic-003.htm compositing/plugins/no-backing-store.html dom/xhtml/level2/html/HTMLElement02.xhtml css2.1/20110323/replaced-intrinsic-001.htm compositing/iframes/iframe-position-absolute-with-padding-percentage-crash.html compositing/geometry/object-clip-rects-assertion.html compositing/plugins/large-to-small-composited-plugin.html dom/html/level2/html/AppletsCollection.html editing/selection/legal-positions.html dom/html/level2/html/HTMLElement01.html Created attachment 211395 [details]
Archive of layout-test-results from webkit-ews-05 for mac-mountainlion
The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-05 Port: mac-mountainlion Platform: Mac OS X 10.8.4
This won't work until we can stop clearing the RenderObject::node() pointer when destroying widgets. Committed r155817: <http://trac.webkit.org/changeset/155817> |