Convert app_bindings.js to the schema_generated_bindings.js infrastructure.

chrome/browser/renderer_host/chrome_render_view_host_observer.cc

Index: chrome/browser/renderer_host/chrome_render_view_host_observer.cc
diff --git a/chrome/browser/renderer_host/chrome_render_view_host_observer.cc b/chrome/browser/renderer_host/chrome_render_view_host_observer.cc
index a094e1abcdfbfb1f95ecd049413da76bdd0b6d2f..7568aefbc7f97561cb90c1c947746811e543871e 100644
--- a/chrome/browser/renderer_host/chrome_render_view_host_observer.cc
+++ b/chrome/browser/renderer_host/chrome_render_view_host_observer.cc
@@ -95,7 +95,6 @@ void ChromeRenderViewHostObserver::InitRenderViewForExtensions() {
   content::RenderProcessHost* process = render_view_host()->process();
 
   if (extension->is_app()) {
-    Send(new ExtensionMsg_ActivateApplication(extension->id()));
     // Though we already record the associated process ID for the renderer in
     // InitRenderViewHostForExtensions, the process might have crashed and been
     // restarted (hence the re-initialization), so we need to update that
@@ -118,14 +117,8 @@ void ChromeRenderViewHostObserver::InitRenderViewForExtensions() {
     }
   }
 
-  if (type == Extension::TYPE_EXTENSION ||
-      type == Extension::TYPE_USER_SCRIPT ||
-      type == Extension::TYPE_PACKAGED_APP ||
-      type == Extension::TYPE_PLATFORM_APP ||
-      (type == Extension::TYPE_HOSTED_APP &&
-       extension->location() == Extension::COMPONENT)) {
+  if (type != Extension::TYPE_THEME)

[Aaron Boodman, 2012/03/06 01:17:21]

I prefer a whitelist here to the blacklist.

I guess it would be even better to do it with a switch and for the default case
CHECK(false) so we don't forget to update this.

[not at google - send to devlin, 2012/03/06 03:53:00]

Done.

(added NOTREACHED() to a version of the patch that i'll upload next round of
comments... though clang will actually check this I believe).

     Send(new ExtensionMsg_ActivateExtension(extension->id()));
-  }
 }
 
 const Extension* ChromeRenderViewHostObserver::GetExtension() {