Chromium Code Reviews

Issue 1663273002: [Merge M48] "[Merge M49] Add Startup.SameVersionStartupCount histogram" (Closed)

Created:
4 years, 10 months ago by gab
Modified:
4 years, 10 months ago
Reviewers:
fdoray
CC:
chromium-reviews, blundell+watchlist_chromium.org, sdefresne+watchlist_chromium.org, asvitkine+watch_chromium.org, droger+watchlist_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@merge_m48_b1_time_since_last_startup
Target Ref:
refs/pending/branch-heads/2564
Project:
chromium
Visibility:
Public.

Description

[Merge M48] "[Merge M49] Add Startup.SameVersionStartupCount histogram" BUG=580211 Review URL: https://codereview.chromium.org/1619273003 Cr-Commit-Position: refs/heads/master@{#371073} (cherry picked from commit 38c76899548104cb4b1639df94ffdc005e2cba8e) Conflicts: components/startup_metric_utils.gypi components/startup_metric_utils/browser/BUILD.gn components/startup_metric_utils/browser/DEPS R=fdoray@chromium.org TBR=fdoray@chromium.org, fdoray@chromium.org Review URL: https://codereview.chromium.org/1652083003 . Cr-Commit-Position: refs/branch-heads/2623@{#223} Cr-Branched-From: 92d77538a86529ca35f9220bd3cd512cbea1f086-refs/heads/master@{#369907} (cherry picked from commit 1ab633f031e2b659857004cdcf82a1dc5fe38019) Conflicts: components/startup_metric_utils.gypi components/startup_metric_utils/browser/BUILD.gn components/startup_metric_utils/browser/DEPS components/startup_metric_utils/browser/startup_metric_utils.cc tools/metrics/histograms/histograms.xml Committed: https://chromium.googlesource.com/chromium/src/+/ace10ad69de6fbd7aaec25762dba2852a58600e6

Patch Set 1 #

Unified diffs Side-by-side diffs Stats (+106 lines, -14 lines)
M chrome/browser/chrome_browser_main.cc View 1 chunk +1 line, -1 line 0 comments
M components/OWNERS View 1 chunk +3 lines, -0 lines 0 comments
M components/startup_metric_utils.gypi View 1 chunk +3 lines, -0 lines 0 comments
M components/startup_metric_utils/browser/BUILD.gn View 2 chunks +3 lines, -0 lines 0 comments
A components/startup_metric_utils/browser/DEPS View 1 chunk +3 lines, -0 lines 0 comments
A components/startup_metric_utils/browser/pref_names.h View 1 chunk +21 lines, -0 lines 0 comments
A components/startup_metric_utils/browser/pref_names.cc View 1 chunk +22 lines, -0 lines 0 comments
M components/startup_metric_utils/browser/startup_metric_utils.h View 3 chunks +10 lines, -7 lines 0 comments
M components/startup_metric_utils/browser/startup_metric_utils.cc View 5 chunks +30 lines, -6 lines 0 comments
M tools/metrics/histograms/histograms.xml View 1 chunk +10 lines, -0 lines 0 comments

Depends on Patchset:

Issue 1667903003 Patch 20001

Dependent Patchsets:

Issue 1663853003 Patch 20001

Messages

Total messages: 7 (4 generated)
gab
Planned merges to M48 -- @fdoray PTAL.
4 years, 10 months ago (2016-02-03 20:34:09 UTC) #3
fdoray
lgtm
4 years, 10 months ago (2016-02-03 20:45:54 UTC) #4
gab
4 years, 10 months ago (2016-02-04 01:48:00 UTC) #7
Message was sent while issue was closed. 
Committed patchset #1 (id:40001) manually as
ace10ad69de6fbd7aaec25762dba2852a58600e6 (presubmit successful).

Powered by Google App Engine