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

Unified Diff: chrome_frame/test/reliability/page_load_test.cc

Issue 28283002: Track execution phase of browser process when we log if the broswer has (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fixed jar's comments Created 7 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/common/pref_names.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome_frame/test/reliability/page_load_test.cc
diff --git a/chrome_frame/test/reliability/page_load_test.cc b/chrome_frame/test/reliability/page_load_test.cc
index 389a3ab529d5ff414c3587cbaaf5ce2a70a7c10b..666b967f1db06b8b86c1a0159ca92bbb0128e60a 100644
--- a/chrome_frame/test/reliability/page_load_test.cc
+++ b/chrome_frame/test/reliability/page_load_test.cc
@@ -136,6 +136,7 @@ class PageLoadTest : public testing::Test {
int crash_dump_count;
// These are stability metrics recorded by Chrome itself
bool browser_clean_exit;
+ int browser_execution_phase;
int browser_launch_count;
int page_load_count;
int browser_crash_count;
@@ -446,6 +447,7 @@ class PageLoadTest : public testing::Test {
return;
scoped_refptr<PrefRegistrySimple> registry = new PrefRegistrySimple();
registry->RegisterBooleanPref(prefs::kStabilityExitedCleanly, false);
+ registry->RegisterIntegerPref(prefs::kStabilityExecutionPhase, 0);
registry->RegisterIntegerPref(prefs::kStabilityLaunchCount, -1);
registry->RegisterIntegerPref(prefs::kStabilityPageLoadCount, -1);
registry->RegisterIntegerPref(prefs::kStabilityCrashCount, 0);
@@ -457,6 +459,8 @@ class PageLoadTest : public testing::Test {
metrics->browser_clean_exit =
local_state->GetBoolean(prefs::kStabilityExitedCleanly);
+ metrics->browser_execution_phase =
+ local_state->GetInteger(prefs::kStabilityExecutionPhase);
metrics->browser_launch_count =
local_state->GetInteger(prefs::kStabilityLaunchCount);
metrics->page_load_count =
« no previous file with comments | « chrome/common/pref_names.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698