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

Unified Diff: chrome/browser/extensions/activity_log/activity_log_browsertest.cc

Issue 2537503002: [Prerender] Get the prerender mode from Finch field trial. (Closed)
Patch Set: Feature API Created 4 years 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
Index: chrome/browser/extensions/activity_log/activity_log_browsertest.cc
diff --git a/chrome/browser/extensions/activity_log/activity_log_browsertest.cc b/chrome/browser/extensions/activity_log/activity_log_browsertest.cc
index e9248b0041ba9cea9bb867bb3c4ee2b11871a215..b290d7a05d0d8633b078ce2560d7da37455f8bd0 100644
--- a/chrome/browser/extensions/activity_log/activity_log_browsertest.cc
+++ b/chrome/browser/extensions/activity_log/activity_log_browsertest.cc
@@ -36,8 +36,6 @@ class ActivityLogPrerenderTest : public ExtensionApiTest {
void SetUpCommandLine(base::CommandLine* command_line) override {
ExtensionBrowserTest::SetUpCommandLine(command_line);
command_line->AppendSwitch(switches::kEnableExtensionActivityLogging);
- command_line->AppendSwitchASCII(switches::kPrerenderMode,
pasko 2016/12/01 16:47:58 if we are removing this, then the tests will need
droger 2016/12/01 17:17:40 I don't have a simple solution unfortunately. We c
droger 2016/12/05 14:48:27 Done.
pasko 2016/12/08 15:23:44 In the last version of the code, the above instruc
droger 2016/12/08 15:47:30 I think they would work, but I did not need it in
- switches::kPrerenderModeSwitchValueEnabled);
}
static void Prerender_Arguments(

Powered by Google App Engine
This is Rietveld 408576698