Bug 137221 - Use is<>() / downcast<>() for Document
Summary: Use is<>() / downcast<>() for Document
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL:
Keywords:
Depends on: 137056
Blocks:
  Show dependency treegraph
 
Reported: 2014-09-29 11:37 PDT by Chris Dumez
Modified: 2014-09-29 15:23 PDT (History)
5 users (show)

See Also:


Attachments
Patch (82.98 KB, patch)
2014-09-29 11:54 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from webkit-ews-15 for mac-mountainlion-wk2 (166.42 KB, application/zip)
2014-09-29 13:02 PDT, Build Bot
no flags Details
Patch (84.07 KB, patch)
2014-09-29 13:18 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (83.41 KB, patch)
2014-09-29 13:25 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (83.27 KB, patch)
2014-09-29 13:30 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Dumez 2014-09-29 11:37:03 PDT
Use is<>() / downcast<>() for Document instead of isDocumentNode() / toDocument(). This is a step towards getting rid of the NODE_TYPE_CASTS() macro.
Comment 1 Chris Dumez 2014-09-29 11:54:06 PDT
Created attachment 238864 [details]
Patch
Comment 2 WebKit Commit Bot 2014-09-29 11:55:40 PDT
Attachment 238864 [details] did not pass style-queue:


ERROR: Source/WebCore/dom/Document.h:1745:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
ERROR: Source/WebCore/svg/SVGDocument.h:57:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
ERROR: Source/WebCore/html/ImageDocument.h:87:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
ERROR: Source/WebCore/html/MediaDocument.h:61:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
ERROR: Source/WebCore/html/PluginDocument.h:66:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
ERROR: Source/WebCore/html/HTMLDocument.h:102:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
Total errors found: 6 in 69 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Build Bot 2014-09-29 13:02:50 PDT
Comment on attachment 238864 [details]
Patch

Attachment 238864 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/6601009887444992

New failing tests:
accessibility/container-node-delete-causes-crash.html
accessibility/canvas-description-and-role.html
accessibility/canvas-fallback-content.html
accessibility/first-letter-text-transform-causes-crash.html
accessibility/content-editable-as-textarea.html
accessibility/contenteditable-table-check-causes-crash.html
accessibility/aria-namefrom-author.html
accessibility/accessibility-object-detached.html
compositing/generated-content.html
canvas/philip/tests/2d.composite.globalComposite.html
accessibility/aria-required.html
compositing/absolute-inside-out-of-view-fixed.html
accessibility/aria-inherits-presentational.html
accessibility/help-text.html
accessibility/aria-labelledby-overrides-aria-labeledby.html
accessibility/aria-mappings.html
accessibility/aria-hidden-hides-all-elements.html
accessibility/content-editable-set-inner-text-generates-axvalue-notification.html
accessibility/heading-crash-after-hidden.html
accessibility/aria-setsize-posinset.html
accessibility/heading-level.html
accessibility/duplicate-child-nodes.html
accessibility/aria-hidden-false-works-in-subtrees.html
accessibility/accessibility-node-memory-management.html
compositing/direct-image-compositing.html
accessibility/aria-invalid.html
compositing/clip-child-by-non-stacking-ancestor.html
compositing/compositing-visible-descendant.html
compositing/document-background-color.html
Comment 4 Build Bot 2014-09-29 13:02:56 PDT
Created attachment 238870 [details]
Archive of layout-test-results from webkit-ews-15 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-15  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5
Comment 5 Chris Dumez 2014-09-29 13:18:21 PDT
Created attachment 238872 [details]
Patch
Comment 6 Chris Dumez 2014-09-29 13:25:24 PDT
Created attachment 238873 [details]
Patch
Comment 7 WebKit Commit Bot 2014-09-29 13:27:59 PDT
Attachment 238873 [details] did not pass style-queue:


ERROR: Source/WebCore/dom/Document.h:1745:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
ERROR: Source/WebCore/svg/SVGDocument.h:57:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
ERROR: Source/WebCore/html/ImageDocument.h:87:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
ERROR: Source/WebCore/html/MediaDocument.h:61:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
ERROR: Source/WebCore/html/PluginDocument.h:66:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
ERROR: Source/WebCore/html/HTMLDocument.h:102:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
Total errors found: 6 in 68 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 8 Chris Dumez 2014-09-29 13:30:08 PDT
Created attachment 238874 [details]
Patch
Comment 9 WebKit Commit Bot 2014-09-29 13:33:04 PDT
Attachment 238874 [details] did not pass style-queue:


ERROR: Source/WebCore/dom/Document.h:1745:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
ERROR: Source/WebCore/svg/SVGDocument.h:57:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
ERROR: Source/WebCore/html/ImageDocument.h:87:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
ERROR: Source/WebCore/html/MediaDocument.h:61:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
ERROR: Source/WebCore/html/PluginDocument.h:66:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
ERROR: Source/WebCore/html/HTMLDocument.h:102:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
Total errors found: 6 in 68 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 10 WebKit Commit Bot 2014-09-29 15:23:25 PDT
Comment on attachment 238874 [details]
Patch

Clearing flags on attachment: 238874

Committed r174089: <http://trac.webkit.org/changeset/174089>
Comment 11 WebKit Commit Bot 2014-09-29 15:23:29 PDT
All reviewed patches have been landed.  Closing bug.