Supporting high dpi favicons in Instant Extended.

chrome/renderer/chrome_content_renderer_client.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/renderer/chrome_content_renderer_client.h"
 
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/metrics/histogram.h"
 #include "base/path_service.h"
 #include "base/string_util.h"
+#include "base/strings/string_number_conversions.h"
 #include "base/utf_string_conversions.h"
 #include "base/values.h"
 #include "chrome/common/child_process_logging.h"
 #include "chrome/common/chrome_content_client.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/content_settings_pattern.h"
 #include "chrome/common/extensions/chrome_manifest_handlers.h"
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/extensions/extension_constants.h"
 #include "chrome/common/extensions/extension_manifest_constants.h"
 #include "chrome/common/extensions/extension_process_policy.h"
 #include "chrome/common/extensions/extension_set.h"
 #include "chrome/common/extensions/permissions/chrome_api_permissions.h"
 #include "chrome/common/external_ipc_fuzzer.h"
+#include "chrome/common/favicon_types.h"
+#include "chrome/common/favicon_url_parser.h"
 #include "chrome/common/localized_error.h"
 #include "chrome/common/render_messages.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/renderer/benchmarking_extension.h"
 #include "chrome/renderer/chrome_render_process_observer.h"
 #include "chrome/renderer/chrome_render_view_observer.h"
 #include "chrome/renderer/content_settings_observer.h"
 #include "chrome/renderer/extensions/chrome_v8_context.h"
 #include "chrome/renderer/extensions/chrome_v8_extension.h"
 #include "chrome/renderer/extensions/dispatcher.h"
@@ skipping 156 matching lines @@
   // Treat CDM invocations like JavaScript.
   if (plugin.name == ASCIIToUTF16(kWidevineCdmDisplayName)) {
     DCHECK(plugin.type == WebPluginInfo::PLUGIN_TYPE_PEPPER_OUT_OF_PROCESS);
     return true;
   }
 #endif  // WIDEVINE_CDM_AVAILABLE
 
   return false;
 }
 
+content::RenderView* GetRenderViewFromWebFrame(WebKit::WebFrame* webframe) {
+  if (!webframe)
+    return NULL;
+  WebKit::WebView* webview = webframe->view();
+  if (!webview)
+    return NULL;
+  return content::RenderView::FromWebView(webview);
+}
+
 }  // namespace
 
 namespace chrome {
 
 ChromeContentRendererClient::ChromeContentRendererClient() {
   g_current_client = this;
 }
 
 ChromeContentRendererClient::~ChromeContentRendererClient() {
   g_current_client = NULL;
@@ skipping 765 matching lines @@
   }
 
   if (url.SchemeIs(chrome::kExtensionResourceScheme) &&
       !extensions::ResourceRequestPolicy::CanRequestExtensionResourceScheme(
           url,
           frame)) {
     *new_url = GURL(chrome::kExtensionResourceInvalidRequestURL);
     return true;
   }
 
+  const content::RenderView* render_view = GetRenderViewFromWebFrame(frame);
+  if (SearchBox* search_box = SearchBox::Get(render_view)) {
+
+    if (url.SchemeIs(chrome::kChromeSearchScheme) &&
+        (url.host() == "thumb" || url.host() == "favicon")) {

[sreeram, 2013/06/04 22:01:56]

Use kChromeUIThumbnailHost, kChromeUIFaviconHost and kChromeUITouchIconHost
instead.

Ideally, we should use Favicon::GetSource() and ThumbnailSource::GetSource(),
but we can't get at those methods from here in the renderer.

[pedro (no code reviews), 2013/06/07 23:34:21]

Done.

+
+      // TODO(pedrosimonetti): DO NOT SUBMIT
+      LOG(ERROR) << "~~~~~~~~~~~~~ ::WillSendRequest " << url.spec();
+
+      // Strip leading slash.
+      std::string raw_path = url.path().substr(1);
+      std::string params = "";
+      bool success = true;
+
+      if (url.host() == "favicon") {
+        std::string favicon_url;
+        bool is_icon_url = false;
+        int size_in_dip = 16;
+        ui::ScaleFactor scale_factor = ui::SCALE_FACTOR_100P;
+
+        success = chrome::ParseFaviconPath(raw_path, false, chrome::FAVICON,
+            &is_icon_url, &favicon_url, &size_in_dip, &scale_factor, &params);
+        // Now that the favicon URL has been parsed, update |raw_path| with
+        // |favicon_url| which it is expected to be the id of a Most Visited

[sreeram, 2013/06/04 22:01:56]

which it is -> which is

[pedro (no code reviews), 2013/06/07 23:34:21]

Done.

+        // item in Instant Extended (which would normally be the URL whose
+        // favicon is being requested).

[sreeram, 2013/06/04 22:01:56]

This comment is kinda superfluous. Just delete it.

[pedro (no code reviews), 2013/06/07 23:34:21]

Done.

+        raw_path = favicon_url;
+      }
+
+      // Try to convert the |raw_path| to int, which is expected to be the id
+      // of a Most Visited item, and then check if there is a Most Visited item
+      // with that id.
+      InstantRestrictedID id;
+      InstantMostVisitedItem item;
+      if (success &&
+          base::StringToInt(raw_path, &id) &&
+          search_box->GetMostVisitedItemWithID(id, &item)) {
+        // Translate the resource URL replacing the id with the URL of the Most
+        // Visited item associated with it. The |params| variable will ensure
+        // that advanced favicon parameters are preserved, allowing high dpi
+        // favicons to be served from chrome-search schema.
+        GURL item_url = item.url;
+        std::string translated_url = url.GetOrigin().spec() + params +
+            item_url.spec();

[sreeram, 2013/06/04 22:01:56]

Do you need to URL-escape the item_url?

[pedro (no code reviews), 2013/06/07 23:34:21]

Good question. I'm not sure if we need to do that.

+        *new_url = GURL(translated_url);
+        // TODO(pedrosimonetti): DO NOT SUBMIT
+        LOG(ERROR) << "OK: " << translated_url;
+        return true;
+      } else {
+        *new_url = GURL(chrome::kChromeSearchInvalidRequestUrl);
+        // TODO(pedrosimonetti): DO NOT SUBMIT
+        LOG(ERROR) << "FAIL: " << url.spec();
+        return true;
+      }
+    }
+  }
+
   return false;
 }
 
 bool ChromeContentRendererClient::ShouldPumpEventsDuringCookieMessage() {
   // We no longer pump messages, even under Chrome Frame. We rely on cookie
   // read requests handled by CF not putting up UI or causing other actions
   // that would require us to pump messages. This fixes http://crbug.com/110090.
   return false;
 }
 
@@ skipping 181 matching lines @@
 
   if (container->element().shadowHost().isNull())
     return false;
 
   WebString tag_name = container->element().shadowHost().tagName();
   return tag_name.equals(WebString::fromUTF8(kWebViewTagName)) ||
     tag_name.equals(WebString::fromUTF8(kAdViewTagName));
 }
 
 }  // namespace chrome