Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(172)

Issue 193683002: Remove histogram in ShadowRoot constructor (Closed)

Created:
6 years, 9 months ago by abarth-chromium
Modified:
6 years, 9 months ago
Reviewers:
esprehn
CC:
blink-reviews, webcomponents-bugzilla_chromium.org, sof, eae+blinkwatch, dglazkov+blink, adamk+blink_chromium.org, rwlbuis, Inactive, Hajime Morrita
Visibility:
Public.

Description

Remove histogram in ShadowRoot constructor This histogram takes too much time. If we want to track these stats, we can use a UseCounter instead. R=esprehn@chromium.org BUG=351135 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=168882

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -5 lines) Patch
M Source/core/dom/shadow/ShadowRoot.cpp View 1 chunk +0 lines, -5 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
abarth-chromium
6 years, 9 months ago (2014-03-11 00:22:49 UTC) #1
esprehn
lgtm
6 years, 9 months ago (2014-03-11 00:30:49 UTC) #2
esprehn
The CQ bit was checked by esprehn@chromium.org
6 years, 9 months ago (2014-03-11 00:36:59 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/abarth@chromium.org/193683002/1
6 years, 9 months ago (2014-03-11 00:37:00 UTC) #4
commit-bot: I haz the power
6 years, 9 months ago (2014-03-11 04:31:13 UTC) #5
Message was sent while issue was closed.
Change committed as 168882

Powered by Google App Engine
This is Rietveld 408576698