Plugin Power Saver: Implement srcset syntax for posters.

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/debug/crash_logging.h"
 #include "base/logging.h"
 #include "base/metrics/field_trial.h"
@@ skipping 296 matching lines @@
       return;
     }
   }
 
   if (power_saver_enabled)
     RecordPosterParamPresence(POSTER_PRESENCE_NO_PARAM_PPS_ENABLED);
   else
     RecordPosterParamPresence(POSTER_PRESENCE_NO_PARAM_PPS_DISABLED);
 }
 
-GURL GetPluginInstancePosterImage(const blink::WebPluginParams& params,
-                                  const GURL& page_base_url) {
+std::string GetPluginInstancePoster(const blink::WebPluginParams& params) {

[Tom Sepez, 2015/03/05 20:28:42]

nit: maybe GetPluginInstancePosterAttribute(), since this gets an attribute
string, rather that the "Poster" itself, which implies in my tiny mind, some
sort of object representing a poster.

[tommycli, 2015/03/06 00:17:31]

Done.

   DCHECK_EQ(params.attributeNames.size(), params.attributeValues.size());
 
   for (size_t i = 0; i < params.attributeNames.size(); ++i) {
     if (params.attributeNames[i].utf8() == "poster" &&
         !params.attributeValues[i].isEmpty()) {
-      return page_base_url.Resolve(params.attributeValues[i].utf8());
+      return params.attributeValues[i].utf8();
     }
   }
-  return GURL();
+  return std::string();
 }
 #endif
 
 #if defined(ENABLE_EXTENSIONS)
 bool IsStandaloneExtensionProcess() {
   return base::CommandLine::ForCurrentProcess()->HasSwitch(
       extensions::switches::kExtensionProcess);
 }
 #endif
 }  // namespace
@@ skipping 437 matching lines @@
         info.type == content::WebPluginInfo::PLUGIN_TYPE_NPAPI) {
         if (observer->AreNPAPIPluginsBlocked())
           status_value =
               ChromeViewHostMsg_GetPluginInfo_Status::kNPAPINotSupported;
     }
 #endif
 
     auto create_blocked_plugin =
         [&render_frame, &frame, &params, &info, &identifier, &group_name](
             int template_id, const base::string16& message) {
-      return ChromePluginPlaceholder::CreateBlockedPlugin(
-          render_frame, frame, params, info, identifier, group_name,
-          template_id, message, GURL());
-    };
+          return ChromePluginPlaceholder::CreateBlockedPlugin(
+              render_frame, frame, params, info, identifier, group_name,
+              template_id, message, std::string(), GURL());
+        };
     switch (status_value) {
       case ChromeViewHostMsg_GetPluginInfo_Status::kNotFound: {
         NOTREACHED();
         break;
       }
       case ChromeViewHostMsg_GetPluginInfo_Status::kAllowed:
       case ChromeViewHostMsg_GetPluginInfo_Status::kPlayImportantContent: {
 #if !defined(DISABLE_NACL) && defined(ENABLE_EXTENSIONS)
         const bool is_nacl_plugin =
             info.name == ASCIIToUTF16(nacl::kNaClPluginName);
@@ skipping 58 matching lines @@
 #if defined(ENABLE_PLUGINS)
         bool power_saver_enabled =
             status_value ==
             ChromeViewHostMsg_GetPluginInfo_Status::kPlayImportantContent;
         bool blocked_for_background_tab =
             render_frame->IsHidden() && power_saver_enabled;
 
         if (info.name == ASCIIToUTF16(content::kFlashPluginName))
           TrackPosterParamPresence(params, power_saver_enabled);
 
-        GURL poster_url;
-        if (power_saver_enabled) {
-          poster_url =
-              GetPluginInstancePosterImage(params, frame->document().url());
-        }
+        std::string poster_attribute;
+        if (power_saver_enabled)
+          poster_attribute = GetPluginInstancePoster(params);
 
         // Delay loading plugins if prerendering.
         // TODO(mmenke):  In the case of prerendering, feed into
         //                ChromeContentRendererClient::CreatePlugin instead, to
         //                reduce the chance of future regressions.
         bool is_prerendering =
             prerender::PrerenderHelper::IsPrerendering(render_frame);
         if (blocked_for_background_tab || is_prerendering ||
-            poster_url.is_valid()) {
+            !poster_attribute.empty()) {
           placeholder = ChromePluginPlaceholder::CreateBlockedPlugin(
               render_frame, frame, params, info, identifier, group_name,
-              poster_url.is_valid() ? IDR_PLUGIN_POSTER_HTML
-                                    : IDR_BLOCKED_PLUGIN_HTML,
+              poster_attribute.empty() ? IDR_BLOCKED_PLUGIN_HTML
+                                       : IDR_PLUGIN_POSTER_HTML,
               l10n_util::GetStringFUTF16(IDS_PLUGIN_BLOCKED, group_name),
-              poster_url);
+              poster_attribute, frame->document().url());
           placeholder->set_blocked_for_background_tab(
               blocked_for_background_tab);
           placeholder->set_blocked_for_prerendering(is_prerendering);
           placeholder->set_power_saver_enabled(power_saver_enabled);
           placeholder->set_allow_loading(true);
           break;
         }
 
         scoped_ptr<content::PluginInstanceThrottler> throttler =
             PluginInstanceThrottler::Create(power_saver_enabled);
@@ skipping 749 matching lines @@
   if (mime_type == content::kBrowserPluginMimeType) {
     return new extensions::ExtensionsGuestViewContainer(render_frame);
   } else {
     return new extensions::MimeHandlerViewContainer(
         render_frame, mime_type, original_url);
   }
 #else
   return NULL;
 #endif
 }