app-shell: Introduce a ShellRendererMainDelegate.

apps/shell/app/shell_main_delegate.h

Index: apps/shell/app/shell_main_delegate.h
diff --git a/apps/shell/app/shell_main_delegate.h b/apps/shell/app/shell_main_delegate.h
index 45ba122f32378f7acb10febcbc8ed46c5a023c91..051612b40f7b2c845f185e5edd3f504a13754043 100644
--- a/apps/shell/app/shell_main_delegate.h
+++ b/apps/shell/app/shell_main_delegate.h
@@ -18,6 +18,7 @@ class ContentRendererClient;
 
 namespace apps {
 class ShellBrowserMainDelegate;
+class ShellRendererMainDelegate;
 
 class ShellMainDelegate : public content::ContentMainDelegate {
  public:
@@ -35,6 +36,8 @@ class ShellMainDelegate : public content::ContentMainDelegate {
   // The created object is owned by ShellBrowserMainParts.
   virtual ShellBrowserMainDelegate* CreateShellBrowserMainDelegate();
 
+  virtual ShellRendererMainDelegate* CreateShellRendererMainDelegate();

[James Cook, 2014/06/17 17:40:22]

Who owns this object?

[sadrul, 2014/06/17 20:37:38]

Good catch. I have changed this to (1) return a scoped_ptr<>, (2) have the
ShellContentRendererClient own the delegate, and (3) document it here.

+
  private:
   // |process_type| is zygote, renderer, utility, etc. Returns true if the
   // process needs data from resources.pak.