[Extensions] Make chrome://extensions use developerPrivate for inspect

chrome/browser/ui/webui/extensions/extension_settings_handler.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/webui/extensions/extension_settings_handler.h"
 
 #include "apps/app_load_service.h"
 #include "apps/saved_files_service.h"
 #include "base/auto_reset.h"
 #include "base/base64.h"
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/location.h"
 #include "base/message_loop/message_loop.h"
 #include "base/metrics/histogram.h"
 #include "base/prefs/pref_service.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/values.h"
 #include "base/version.h"
 #include "chrome/browser/background/background_contents.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chrome_notification_types.h"
-#include "chrome/browser/devtools/devtools_window.h"
 #include "chrome/browser/extensions/api/extension_action/extension_action_api.h"
 #include "chrome/browser/extensions/component_loader.h"
 #include "chrome/browser/extensions/crx_installer.h"
-#include "chrome/browser/extensions/devtools_util.h"
 #include "chrome/browser/extensions/error_console/error_console.h"
 #include "chrome/browser/extensions/extension_action_manager.h"
 #include "chrome/browser/extensions/extension_error_reporter.h"
 #include "chrome/browser/extensions/extension_management.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/extension_tab_util.h"
 #include "chrome/browser/extensions/extension_ui_util.h"
 #include "chrome/browser/extensions/extension_util.h"
 #include "chrome/browser/extensions/install_verifier.h"
 #include "chrome/browser/extensions/path_util.h"
@@ skipping 684 matching lines @@
     management_policy_ = ExtensionSystem::Get(
         extension_service_->profile())->management_policy();
   }
 
   web_ui()->RegisterMessageCallback("extensionSettingsRequestExtensionsData",
       base::Bind(&ExtensionSettingsHandler::HandleRequestExtensionsData,
                  AsWeakPtr()));
   web_ui()->RegisterMessageCallback("extensionSettingsToggleDeveloperMode",
       base::Bind(&ExtensionSettingsHandler::HandleToggleDeveloperMode,
                  AsWeakPtr()));
-  web_ui()->RegisterMessageCallback("extensionSettingsInspect",
-      base::Bind(&ExtensionSettingsHandler::HandleInspectMessage,
-                 AsWeakPtr()));
   web_ui()->RegisterMessageCallback("extensionSettingsLaunch",
       base::Bind(&ExtensionSettingsHandler::HandleLaunchMessage,
                  AsWeakPtr()));
   web_ui()->RegisterMessageCallback("extensionSettingsRepair",
       base::Bind(&ExtensionSettingsHandler::HandleRepairMessage,
                  AsWeakPtr()));
   web_ui()->RegisterMessageCallback("extensionSettingsEnableErrorCollection",
       base::Bind(&ExtensionSettingsHandler::HandleEnableErrorCollectionMessage,
                  AsWeakPtr()));
   web_ui()->RegisterMessageCallback("extensionSettingsAllowOnAllUrls",
@@ skipping 240 matching lines @@
   Profile* profile = Profile::FromWebUI(web_ui());
   if (profile->IsSupervised())
     return;
 
   bool developer_mode_on;
   CHECK(args->GetBoolean(0, &developer_mode_on));
   profile->GetPrefs()->SetBoolean(prefs::kExtensionsUIDeveloperMode,
                                   developer_mode_on);
 }
 
-void ExtensionSettingsHandler::HandleInspectMessage(
-    const base::ListValue* args) {
-  std::string extension_id;
-  std::string render_process_id_str;
-  std::string render_view_id_str;
-  int render_process_id;
-  int render_view_id;
-  bool incognito;
-  CHECK_EQ(4U, args->GetSize());
-  CHECK(args->GetString(0, &extension_id));
-  CHECK(args->GetString(1, &render_process_id_str));
-  CHECK(args->GetString(2, &render_view_id_str));
-  CHECK(args->GetBoolean(3, &incognito));
-  CHECK(base::StringToInt(render_process_id_str, &render_process_id));
-  CHECK(base::StringToInt(render_view_id_str, &render_view_id));
-
-  if (render_process_id == -1) {
-    // This message is for a lazy background page. Start the page if necessary.
-    Profile* profile = Profile::FromWebUI(web_ui());
-    const Extension* extension =
-        ExtensionRegistry::Get(profile)->enabled_extensions().GetByID(
-            extension_id);
-    DCHECK(extension);
-    if (incognito)
-      profile = profile->GetOffTheRecordProfile();
-    devtools_util::InspectBackgroundPage(extension, profile);
-    return;
-  }
-
-  RenderViewHost* host = RenderViewHost::FromID(render_process_id,
-                                                render_view_id);
-  if (!host || !WebContents::FromRenderViewHost(host)) {
-    // This can happen if the host has gone away since the page was displayed.
-    return;
-  }
-
-  DevToolsWindow::OpenDevToolsWindow(WebContents::FromRenderViewHost(host));
-}
-
 void ExtensionSettingsHandler::HandleLaunchMessage(
     const base::ListValue* args) {
   CHECK_EQ(1U, args->GetSize());
   std::string extension_id;
   CHECK(args->GetString(0, &extension_id));
   const Extension* extension =
       extension_service_->GetExtensionById(extension_id, false);
   OpenApplication(AppLaunchParams(extension_service_->profile(), extension,
                                   extensions::LAUNCH_CONTAINER_WINDOW,
                                   NEW_WINDOW,
@@ skipping 326 matching lines @@
 }
 
 void ExtensionSettingsHandler::OnReinstallComplete(
     bool success,
     const std::string& error,
     webstore_install::Result result) {
   MaybeUpdateAfterNotification();
 }
 
 }  // namespace extensions