Block plugins for platform apps

chrome/browser/ui/extensions/shell_window.cc

 // Copyright (c) 2012 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.
 
 #include "chrome/browser/ui/extensions/shell_window.h"
 
 #include "chrome/browser/chrome_plugin_service_filter.h"
 #include "chrome/browser/extensions/extension_process_manager.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/chrome_notification_types.h"
@@ skipping 48 matching lines @@
     }
     case content::NOTIFICATION_APP_TERMINATING:
       Close();
       break;
     default:
       NOTREACHED() << "Received unexpected notification";
   }
 }
 
 void ShellWindow::RenderViewCreated(RenderViewHost* render_view_host) {
+  // Notify anyone interested that the app has started.

[Mihai Parparita -not on Chrome, 2012/01/25 19:52:49]

We might have non-ShellWindow UIs for platform apps (e.g. for panels). It seems
like this should be moved to ExtensionProcessManager::CreateShellHost.

Also, you might be able to use the existing NOTIFICATION_EXTENSION_HOST_CREATED
notification instead, and check if the extension is a platform app.

[benwells, 2012/01/27 04:03:52]

Done: just used NOTIFICATION_EXTENSION_HOST_CREATED.

+  content::NotificationService::current()->Notify(
+      chrome::NOTIFICATION_PLATFORM_APP_STARTED,
+      content::Source<Profile>(host_->profile()),
+      content::Details<const Extension>(host_->extension()));
   DisableNPAPIPlugins();
 }
 
 ShellWindow::ShellWindow(ExtensionHost* host)
     : host_(host) {
   // Close the window in response to window.close() and the like.
   registrar_.Add(this, chrome::NOTIFICATION_EXTENSION_HOST_VIEW_SHOULD_CLOSE,
                  content::Source<Profile>(host->profile()));
   // Also close if the window if the extension has been unloaded (parallels
   // NOTIFICATION_EXTENSION_UNLOADED closing the app's tabs in TabStripModel).
@@ skipping 21 matching lines @@
 }
 
 void ShellWindow::ClearDisabledNPAPIPlugins() {
   int render_process_id = host_->render_process_host()->GetID();
   int render_view_id = host_->render_view_host()->routing_id();
   ChromePluginServiceFilter* filter =
       ChromePluginServiceFilter::GetInstance();
   filter->ClearDisabledNPAPIForRenderView(render_process_id,
                                           render_view_id);
 }