Move *GetWebApplicationInfo IPCs from "view" to "frame".

chrome/renderer/chrome_render_view_observer.cc

Index: chrome/renderer/chrome_render_view_observer.cc
diff --git a/chrome/renderer/chrome_render_view_observer.cc b/chrome/renderer/chrome_render_view_observer.cc
index dd9335a1bc880877a4831721e1ceca956b42b5fd..5445f7fb4ef383ec97c147fbf2de77723eac9f57 100644
--- a/chrome/renderer/chrome_render_view_observer.cc
+++ b/chrome/renderer/chrome_render_view_observer.cc
@@ -16,12 +16,10 @@
 #include "base/strings/utf_string_conversions.h"
 #include "base/trace_event/trace_event.h"
 #include "build/build_config.h"
-#include "chrome/common/chrome_constants.h"
 #include "chrome/common/crash_keys.h"
 #include "chrome/common/render_messages.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/renderer/prerender/prerender_helper.h"
-#include "chrome/renderer/web_apps.h"
 #include "components/web_cache/renderer/web_cache_impl.h"
 #include "content/public/common/bindings_policy.h"
 #include "content/public/renderer/content_renderer_client.h"
@@ -61,8 +59,6 @@ bool ChromeRenderViewObserver::OnMessageReceived(const IPC::Message& message) {
     IPC_MESSAGE_HANDLER(ChromeViewMsg_UpdateBrowserControlsState,
                         OnUpdateBrowserControlsState)
 #endif
-    IPC_MESSAGE_HANDLER(ChromeViewMsg_GetWebApplicationInfo,
-                        OnGetWebApplicationInfo)
     IPC_MESSAGE_HANDLER(ChromeViewMsg_SetWindowFeatures, OnSetWindowFeatures)
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()
@@ -86,50 +82,6 @@ void ChromeRenderViewObserver::OnUpdateBrowserControlsState(
 }
 #endif
 
-void ChromeRenderViewObserver::OnGetWebApplicationInfo() {
-  WebFrame* main_frame = render_view()->GetWebView()->MainFrame();
-  DCHECK(main_frame);
-
-  WebApplicationInfo web_app_info;
-  web_apps::ParseWebAppFromWebDocument(main_frame, &web_app_info);
-
-  // The warning below is specific to mobile but it doesn't hurt to show it even
-  // if the Chromium build is running on a desktop. It will get more exposition.
-  // TODO(mlamouri): Associate this message with an actual frame, to avoid the
-  // need to check whether or not the main frame is local.

[Łukasz Anforowicz, 2017/06/15 04:05:39]

This is the TODO being addressed by this CL.

-  if (web_app_info.mobile_capable == WebApplicationInfo::MOBILE_CAPABLE_APPLE &&
-      main_frame->IsWebLocalFrame()) {
-    blink::WebConsoleMessage message(
-        blink::WebConsoleMessage::kLevelWarning,
-        "<meta name=\"apple-mobile-web-app-capable\" content=\"yes\"> is "
-        "deprecated. Please include <meta name=\"mobile-web-app-capable\" "
-        "content=\"yes\"> - "
-        "http://developers.google.com/chrome/mobile/docs/installtohomescreen");
-    main_frame->ToWebLocalFrame()->AddMessageToConsole(message);

[Łukasz Anforowicz, 2017/06/15 04:05:39]

After this CL, we can avoid ToWebLocalFrame "cast".  This helps with a follow-up
attempt (https://codereview.chromium.org/2928033002) to move GetDocument
accessor from WebFrame to WebLocalFrame (since in case of WebRemoteFrameImpl
GetDocument returns a likely-to-crash null document).

-  }
-
-  // Prune out any data URLs in the set of icons.  The browser process expects
-  // any icon with a data URL to have originated from a favicon.  We don't want
-  // to decode arbitrary data URLs in the browser process.  See
-  // http://b/issue?id=1162972
-  for (std::vector<WebApplicationInfo::IconInfo>::iterator it =
-          web_app_info.icons.begin(); it != web_app_info.icons.end();) {
-    if (it->url.SchemeIs(url::kDataScheme))
-      it = web_app_info.icons.erase(it);
-    else
-      ++it;
-  }
-
-  // Truncate the strings we send to the browser process.
-  web_app_info.title =
-      web_app_info.title.substr(0, chrome::kMaxMetaTagAttributeLength);
-  web_app_info.description =
-      web_app_info.description.substr(0, chrome::kMaxMetaTagAttributeLength);
-
-  Send(new ChromeViewHostMsg_DidGetWebApplicationInfo(
-      routing_id(), web_app_info));
-}
-
 void ChromeRenderViewObserver::OnSetWindowFeatures(
     const blink::mojom::WindowFeatures& window_features) {
   render_view()->GetWebView()->SetWindowFeatures(