Enable features API for content shell and webview

content/browser/browser_main_loop.cc

Index: content/browser/browser_main_loop.cc
diff --git a/content/browser/browser_main_loop.cc b/content/browser/browser_main_loop.cc
index 0959081b941f108b6bc3b82313420424e7e808e5..8df360a3d155a28b4e6ef80581d8285319b97bdd 100644
--- a/content/browser/browser_main_loop.cc
+++ b/content/browser/browser_main_loop.cc
@@ -695,9 +695,12 @@ int BrowserMainLoop::PreCreateThreads() {
     result_code_ = parts_->PreCreateThreads();
   }
 
-  // Initialize an instance of FeatureList. This will be a no-op if an instance
-  // was already set up by the embedder.
-  base::FeatureList::InitializeInstance();
+  // Initialize from command line flags only if we haven't yet.

[no sievers, 2016/03/30 18:36:03]

nit: the 'only if we haven't yet' comment does not match the code. Did you mean
it's safe to call this repeatedly and it does nothing the second time?

[Changwan Ryu, 2016/03/31 00:31:44]

Updated the comment.

+  const base::CommandLine* command_line =
+      base::CommandLine::ForCurrentProcess();
+  base::FeatureList::InitializeInstance(
+      command_line->GetSwitchValueASCII(switches::kEnableFeatures),
+      command_line->GetSwitchValueASCII(switches::kDisableFeatures));
 
   // TODO(chrisha): Abstract away this construction mess to a helper function,
   // once MemoryPressureMonitor is made a concrete class.