Chromecast: end-to-end browser test based on content::BrowserTest.

chromecast/shell/browser/cast_browser_main_parts.h

Index: chromecast/shell/browser/cast_browser_main_parts.h
diff --git a/chromecast/shell/browser/cast_browser_main_parts.h b/chromecast/shell/browser/cast_browser_main_parts.h
index e620eb24ac930ceb44fda5865a3db3e8fc8b137e..4710cac9c5cbb34cf38527ff8d5a9c29c3f23bd4 100644
--- a/chromecast/shell/browser/cast_browser_main_parts.h
+++ b/chromecast/shell/browser/cast_browser_main_parts.h
@@ -9,10 +9,7 @@
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "content/public/browser/browser_main_parts.h"
-
-namespace content {
-struct MainFunctionParams;
-}
+#include "content/public/common/main_function_params.h"
 
 namespace chromecast {
 namespace shell {
@@ -37,6 +34,8 @@ class CastBrowserMainParts : public content::BrowserMainParts {
  private:
   scoped_ptr<CastBrowserProcess> cast_browser_process_;
 
+  // For running browser tests.
+  const content::MainFunctionParams parameters_;

[lcwu1, 2014/09/08 19:49:15]

The way the data members are broken up and where you place the comment make it
look like both parameters_ and url_request_context_factory_ are both used for
running browser tests. Maybe we should remove the empty line on line 36, or
append the comment on line 37 to line 38.

[gunsch, 2014/09/08 21:34:45]

Done.

   URLRequestContextFactory* const url_request_context_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(CastBrowserMainParts);