Remove unused packaged app window controls code.

chrome/renderer/extensions/chrome_extensions_dispatcher_delegate.cc

Index: chrome/renderer/extensions/chrome_extensions_dispatcher_delegate.cc
diff --git a/chrome/renderer/extensions/chrome_extensions_dispatcher_delegate.cc b/chrome/renderer/extensions/chrome_extensions_dispatcher_delegate.cc
index 745f2b992b4d19168d49907875e098a19ba37f6b..970b24f9ea05d23ba05c942d0b659845c5749e07 100644
--- a/chrome/renderer/extensions/chrome_extensions_dispatcher_delegate.cc
+++ b/chrome/renderer/extensions/chrome_extensions_dispatcher_delegate.cc
@@ -261,33 +261,21 @@ void ChromeExtensionsDispatcherDelegate::PopulateSourceMap(
   // Platform app sources that are not API-specific..
   source_map->RegisterSource("fileEntryBindingUtil",
                              IDR_FILE_ENTRY_BINDING_UTIL_JS);
-  source_map->RegisterSource("tagWatcher", IDR_TAG_WATCHER_JS);
   source_map->RegisterSource("chromeWebViewInternal",
                              IDR_CHROME_WEB_VIEW_INTERNAL_CUSTOM_BINDINGS_JS);
   source_map->RegisterSource("chromeWebView", IDR_CHROME_WEB_VIEW_JS);
 }
 
 void ChromeExtensionsDispatcherDelegate::RequireAdditionalModules(
-    extensions::ScriptContext* context,
-    bool is_within_platform_app) {
-  extensions::ModuleSystem* module_system = context->module_system();
-  extensions::Feature::Context context_type = context->context_type();
-
-  // TODO(kalman, fsamuel): Eagerly calling Require on context startup is
-  // expensive. It would be better if there were a light way of detecting when
-  // a webview or appview is created and only then set up the infrastructure.
-  if (context_type == extensions::Feature::BLESSED_EXTENSION_CONTEXT &&
-      is_within_platform_app &&
-      extensions::GetCurrentChannel() <= version_info::Channel::DEV &&
-      base::CommandLine::ForCurrentProcess()->HasSwitch(
-          extensions::switches::kEnableAppWindowControls)) {
-    module_system->Require("windowControls");
-  }
-
-  // Note: setting up the WebView class here, not the chrome.webview API.
-  // The API will be automatically set up when first used.
+    extensions::ScriptContext* context) {
   if (context->GetAvailability("webViewInternal").is_available()) {
-    module_system->Require("chromeWebView");
+    // TODO(fsamuel): Eagerly calling Require on context startup is expensive.
+    // It would be better if there were a light way of detecting when a webview
+    // or appview is created and only then set up the infrastructure.
+
+    // Note: setting up the WebView class here, not the chrome.webview API.

[tapted, 2016/12/16 03:07:09]

optional nit: this comment could stay above the `if`, and the `Require` comment
be attached to the next line. That might be slightly clearer

[benwells, 2016/12/19 03:54:13]

Done.

+    // The API will be automatically set up when first used.
+    context->module_system()->Require("chromeWebView");
   }
 }