Bug 99955 - [Chromium] Needs to track ShadowRoot usage
Summary: [Chromium] Needs to track ShadowRoot usage
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Hajime Morrita
URL:
Keywords:
Depends on:
Blocks: 94082
  Show dependency treegraph
 
Reported: 2012-10-21 22:56 PDT by Hajime Morrita
Modified: 2012-10-22 19:05 PDT (History)
3 users (show)

See Also:


Attachments
Patch (2.19 KB, patch)
2012-10-22 01:13 PDT, Hajime Morrita
no flags Details | Formatted Diff | Diff
Patch for landing (2.24 KB, patch)
2012-10-22 18:31 PDT, Hajime Morrita
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Hajime Morrita 2012-10-21 22:56:56 PDT
This upstreams crbug.com/157020 that aims to add an UMA instrumentation for tracking ShadowRoot usage.
Comment 1 Hajime Morrita 2012-10-22 01:13:23 PDT
Created attachment 169845 [details]
Patch
Comment 2 WebKit Review Bot 2012-10-22 04:13:46 PDT
Comment on attachment 169845 [details]
Patch

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

New failing tests:
fast/dom/shadow/shadow-root-js-api.html
Comment 3 Dimitri Glazkov (Google) 2012-10-22 15:07:38 PDT
Comment on attachment 169845 [details]
Patch

Can you fix the layout test before landing?
Comment 4 Hajime Morrita 2012-10-22 18:31:23 PDT
Created attachment 170041 [details]
Patch for landing
Comment 5 WebKit Review Bot 2012-10-22 19:05:17 PDT
Comment on attachment 170041 [details]
Patch for landing

Clearing flags on attachment: 170041

Committed r132171: <http://trac.webkit.org/changeset/132171>
Comment 6 WebKit Review Bot 2012-10-22 19:05:22 PDT
All reviewed patches have been landed.  Closing bug.