Add "Request app banner" context menu in DevTools

chrome/browser/devtools/chrome_devtools_manager_delegate.cc

 // Copyright 2014 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/devtools/chrome_devtools_manager_delegate.h"
 
 #include "build/build_config.h"
 #include "chrome/browser/devtools/devtools_network_protocol_handler.h"
+#include "chrome/browser/devtools/devtools_protocol_constants.h"
+#include "chrome/browser/ui/tab_contents/core_tab_helper.h"
+#include "chrome/browser/ui/tab_contents/core_tab_helper_delegate.h"
+#include "chrome/common/chrome_switches.h"
+#include "content/public/browser/devtools_agent_host.h"
+#include "content/public/browser/render_frame_host.h"
 
 #if !defined(OS_ANDROID)
 #include "chrome/browser/devtools/devtools_window.h"
 #include "chrome/browser/profiles/profile.h"
-#include "content/public/browser/devtools_agent_host.h"
 #endif  // !defined(OS_ANDROID)
 
+namespace {
+
+scoped_ptr<base::DictionaryValue> RequestAppBanner(
+    content::DevToolsAgentHost* agent_host,
+    int command_id,
+    base::DictionaryValue* params) {
+  content::WebContents* web_contents = agent_host->GetWebContents();
+  const bool ret = CoreTabHelper::FromWebContents(web_contents)
+                       ->delegate()
+                       ->RequestAppBanner(web_contents);
+  if (!ret) {
+    web_contents->GetMainFrame()->AddMessageToConsole(
+        content::CONSOLE_MESSAGE_LEVEL_DEBUG,
+        "App banners are currently disabled. Please check chrome://flags/#" +
+            std::string(switches::kEnableAddToShelf));
+  }
+  scoped_ptr<base::DictionaryValue> result(new base::DictionaryValue());
+  result->SetBoolean(chrome::devtools::kResult, ret);
+  return DevToolsProtocol::CreateSuccessResponse(command_id, std::move(result));
+}
+
+}  // namespace
+
 ChromeDevToolsManagerDelegate::ChromeDevToolsManagerDelegate()
     : network_protocol_handler_(new DevToolsNetworkProtocolHandler()) {
 }
 
 ChromeDevToolsManagerDelegate::~ChromeDevToolsManagerDelegate() {
 }
 
 void ChromeDevToolsManagerDelegate::Inspect(
     content::BrowserContext* browser_context,
     content::DevToolsAgentHost* agent_host) {
 #if !defined(OS_ANDROID)
   content::DevToolsAgentHost::Type type = agent_host->GetType();
   if (type != content::DevToolsAgentHost::TYPE_SHARED_WORKER &&
       type != content::DevToolsAgentHost::TYPE_SERVICE_WORKER) {
     // TODO(horo): Support other types of DevToolsAgentHost when necessary.
     NOTREACHED() << "Inspect() only supports workers.";
   }
   if (Profile* profile = Profile::FromBrowserContext(browser_context))
     DevToolsWindow::OpenDevToolsWindowForWorker(profile, agent_host);
 #endif  // !defined(OS_ANDROID)
 }
 
 base::DictionaryValue* ChromeDevToolsManagerDelegate::HandleCommand(
     content::DevToolsAgentHost* agent_host,
     base::DictionaryValue* command_dict) {
-  return network_protocol_handler_->HandleCommand(agent_host, command_dict);
+  int id = 0;

[gone, 2016/01/19 18:35:00]

nit: Can you break this up with newlines?  It's hard to parse.

[horo, 2016/01/20 02:26:34]

Done.

+  std::string method;
+  base::DictionaryValue* params = nullptr;
+  if (!DevToolsProtocol::ParseCommand(command_dict, &id, &method, &params))
+    return nullptr;
+  base::DictionaryValue* ret =
+      network_protocol_handler_->HandleCommand(agent_host, id, method, params);
+  if (ret)
+    return ret;
+  if (method == ::chrome::devtools::Page::requestAppBanner::kName)
+    return RequestAppBanner(agent_host, id, params).release();
+  return nullptr;
 }
 
 void ChromeDevToolsManagerDelegate::DevToolsAgentStateChanged(
     content::DevToolsAgentHost* agent_host,
     bool attached) {
   network_protocol_handler_->DevToolsAgentStateChanged(agent_host, attached);
 }