CastContentBrowserClient should not hold CastBrowserMainParts lifetime.

chromecast/shell/browser/cast_content_browser_client.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROMECAST_SHELL_BROWSER_CAST_CONTENT_BROWSER_CLIENT_H_
 #define CHROMECAST_SHELL_BROWSER_CAST_CONTENT_BROWSER_CLIENT_H_
 
 #include "base/macros.h"
+#include "base/memory/scoped_ptr.h"
 #include "content/public/browser/content_browser_client.h"
 
 namespace chromecast {
 namespace shell {
 
 class CastBrowserMainParts;
 class URLRequestContextFactory;
 
 class CastContentBrowserClient: public content::ContentBrowserClient {
  public:
@@ skipping 43 matching lines @@
       content::ResourceContext* context,
       int render_process_id,
       int opener_id,
       bool* no_javascript_access) OVERRIDE;
   virtual void GetAdditionalMappedFilesForChildProcess(
       const base::CommandLine& command_line,
       int child_process_id,
       std::vector<content::FileDescriptorInfo>* mappings) OVERRIDE;
 
  private:
-  scoped_ptr<CastBrowserMainParts> shell_browser_main_parts_;
-
+  // Note: BrowserMainLoop holds ownership of CastBrowserMainParts after it is
+  // created.
+  CastBrowserMainParts* shell_browser_main_parts_;
   scoped_ptr<URLRequestContextFactory> url_request_context_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(CastContentBrowserClient);
 };
 
 }  // namespace shell
 }  // namespace chromecast
 
 #endif  // CHROMECAST_SHELL_BROWSER_CAST_CONTENT_BROWSER_CLIENT_H_