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

Issue 15937007: Open Source Renderer4.* histograms. (Closed)

Created:
7 years, 7 months ago by hartmanng
Modified:
7 years, 6 months ago
CC:
chromium-reviews, MAD, jar (doing other things)
Visibility:
Public.

Description

Open Source Renderer4.* histograms. BUG=223859 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=202223

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+307 lines, -0 lines) Patch
M tools/metrics/histograms/histograms.xml View 5 chunks +307 lines, -0 lines 0 comments Download

Messages

Total messages: 9 (0 generated)
hartmanng
Please take a look.
7 years, 7 months ago (2013-05-24 18:27:35 UTC) #1
Alexei Svitkine (slow)
lgtm, assuming these are simply copied over from the internal file please set BUG=223859
7 years, 7 months ago (2013-05-24 18:33:15 UTC) #2
hartmanng
On 2013/05/24 18:33:15, Alexei Svitkine wrote: > lgtm, assuming these are simply copied over from ...
7 years, 7 months ago (2013-05-24 18:44:20 UTC) #3
jar (doing other things)
lgtm
7 years, 7 months ago (2013-05-24 19:45:45 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/hartmanng@chromium.org/15937007/1
7 years, 7 months ago (2013-05-24 19:46:07 UTC) #5
commit-bot: I haz the power
Change committed as 202223
7 years, 7 months ago (2013-05-24 23:29:14 UTC) #6
Ilya Sherman
Is there a corresponding CL to remove these from the internal file?
7 years, 6 months ago (2013-05-30 00:01:31 UTC) #7
hartmanng
On 2013/05/30 00:01:31, Ilya Sherman wrote: > Is there a corresponding CL to remove these ...
7 years, 6 months ago (2013-05-30 15:09:24 UTC) #8
Ilya Sherman
7 years, 6 months ago (2013-05-30 22:08:23 UTC) #9
Message was sent while issue was closed.
On 2013/05/30 15:09:24, hartmanng wrote:
> On 2013/05/30 00:01:31, Ilya Sherman wrote:
> > Is there a corresponding CL to remove these from the internal file?
> 
> Yes, and it landed on Monday:
> http://goto.ext.google.com/viewvc/chrome-internal?view=rev&revision=37694

Lovely, thanks :)

Powered by Google App Engine
This is Rietveld 408576698