Summary: | Move m_type and m_hasNameCache from HTMLCollectionCacheBase to DynamicNodeListCacheBase for better bit packing | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> | ||||||
Component: | DOM | Assignee: | Ryosuke Niwa <rniwa> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | andersca, darin, dglazkov, gustavo, kling, koivisto, philn, sam, webkit.review.bot, xan.lopez | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 89919 | ||||||||
Attachments: |
|
Description
Ryosuke Niwa
2012-07-12 16:39:17 PDT
Created attachment 152095 [details]
Patch
Comment on attachment 152095 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=152095&action=review > Source/WebCore/html/HTMLCollection.cpp:73 > + case InvalidCollectionType: > + break; Oops, this shouldn't be here. I'm getting ahead of myself. Created attachment 152097 [details]
Removed the bits from the future
Comment on attachment 152097 [details] Removed the bits from the future Attachment 152097 [details] did not pass efl-ews (efl): Output: http://queues.webkit.org/results/13230005 Oh oops, I did need InvalidCollectionType in DynamicNodeListCacheBase. Will add back & make sure it builds before landing it. Comment on attachment 152097 [details] Removed the bits from the future Attachment 152097 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/13221423 Comment on attachment 152097 [details] Removed the bits from the future Attachment 152097 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/13221422 Comment on attachment 152097 [details] Removed the bits from the future Attachment 152097 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/13203909 Comment on attachment 152097 [details] Removed the bits from the future Attachment 152097 [details] did not pass qt-wk2-ews (qt): Output: http://queues.webkit.org/results/13209932 Comment on attachment 152097 [details] Removed the bits from the future Attachment 152097 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/13202879 Committed r122531: <http://trac.webkit.org/changeset/122531> Apparently gcc doesn't catch all warnings :( I've landed a build fix in r122533. |