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

Issue 10545139: Performance Monitor Database (Closed)

Created:
8 years, 6 months ago by eaugusti
Modified:
8 years, 6 months ago
Reviewers:
Aaron Boodman
CC:
chromium-reviews
Base URL:
http://git.chromium.org/chromium/src.git@master
Visibility:
Public.

Description

Performance Monitor Database The database is the core data store for the Performance Monitor. This CL encapsulates an initial release of the databse without support for metrics or events. BUG=130212 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=141896

Patch Set 1 #

Patch Set 2 : Windows and Mac specific code #

Patch Set 3 : Fixed windows fail #

Unified diffs Side-by-side diffs Delta from patch set Stats (+442 lines, -0 lines) Patch
A chrome/browser/performance_monitor/database.h View 1 2 1 chunk +162 lines, -0 lines 0 comments Download
A chrome/browser/performance_monitor/database.cc View 1 2 1 chunk +198 lines, -0 lines 0 comments Download
A chrome/browser/performance_monitor/database_unittest.cc View 1 2 1 chunk +79 lines, -0 lines 0 comments Download
M chrome/chrome_browser.gypi View 1 chunk +2 lines, -0 lines 0 comments Download
M chrome/chrome_tests.gypi View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Aaron Boodman
LGTM
8 years, 6 months ago (2012-06-12 18:35:34 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/eaugusti@chromium.org/10545139/1
8 years, 6 months ago (2012-06-12 18:35:59 UTC) #2
commit-bot: I haz the power
Try job failure for 10545139-1 (retry) on mac_rel for step "compile" (clobber build). It's a ...
8 years, 6 months ago (2012-06-12 19:01:08 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/eaugusti@chromium.org/10545139/2002
8 years, 6 months ago (2012-06-13 15:27:07 UTC) #4
commit-bot: I haz the power
8 years, 6 months ago (2012-06-13 16:41:05 UTC) #5
Change committed as 141896

Powered by Google App Engine
This is Rietveld 408576698