Rewrite YouTube Flash embeds

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 <memory>
 #include <utility>
 
 #include "base/command_line.h"
 #include "base/debug/crash_logging.h"
 #include "base/logging.h"
 #include "base/macros.h"
 #include "base/metrics/field_trial.h"
 #include "base/metrics/histogram.h"
 #include "base/metrics/user_metrics_action.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/time/time.h"
 #include "base/values.h"
 #include "build/build_config.h"
 #include "chrome/common/channel_info.h"
+#include "chrome/common/chrome_features.h"
 #include "chrome/common/chrome_isolated_world_ids.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/crash_keys.h"
 #include "chrome/common/pepper_permission_util.h"
 #include "chrome/common/render_messages.h"
 #include "chrome/common/secure_origin_whitelist.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/grit/generated_resources.h"
 #include "chrome/grit/locale_settings.h"
@@ skipping 1338 matching lines @@
 // chrome.system.network.getNetworkInterfaces provides the same
 // information. Also, the enforcement of sending and binding UDP is already done
 // by chrome extension permission model.
 bool ChromeContentRendererClient::ShouldEnforceWebRTCRoutingPreferences() {
 #if defined(ENABLE_EXTENSIONS)
   return !IsStandaloneExtensionProcess();
 #else
   return true;
 #endif
 }
+
+std::string ChromeContentRendererClient::OverrideFlashEmbedWithHTML(
+    const std::string& str_url) {
+  if (!base::FeatureList::IsEnabled(features::kOverrideYouTubeFlashEmbed))
+    return "";
+
+  GURL url = GURL(str_url);
+  if (!url.is_valid())
+    return "";
+
+  // We'll only modify YouTube Flash embeds. The URLs can be recognized since
+  // they're in the following form: youtube.com/v/VIDEO_ID. So, we check to see
+  // if the given URL does follow that format. We don't modify
+  // any URLs that contain the enablejsapi=1 parameter since the page may be

[mlamouri (slow - plz ping), 2016/08/01 13:43:59]

Can you open a follow-up bug to actually do the rewrite when enablejsapi=1 on
Android?

[kdsilva, 2016/08/02 15:34:46]

Done.

+  // interacting with the YouTube Flash player in Javascript and we don't
+  // want to break working content.
+  if (!url.DomainIs("youtube.com") || url.path().find("/v/") != 0
+      || str_url.find("enablejsapi=1") != std::string::npos)
+    return "";

[dcheng, 2016/08/01 14:50:34]

This should just use GURLs, but note that converting "" to std::string() like
this is surprisingly expensive in general: it's better to just use the
std::string default constructor. A similar comment applies throughout the rest
of this CL for WebString/WebURL

[kdsilva, 2016/08/02 15:34:46]

Done.

+
+  std::string ret_url = str_url;
+  ret_url.replace(ret_url.find("/v/"), 3, "/embed/");

[dcheng, 2016/08/01 14:50:34]

Can we just use GURL and avoid cracking URLs manually? At least we can make it
just match the patch component that way and simplify the matching logic below.

[kdsilva, 2016/08/02 15:34:46]

Done.

+
+  // If the website is using an invalid YouTube URL, we'll try and
+  // fix the URL by ensuring that if there are multiple parameters,
+  // the parameter string begins with a "?" and then follows with a "&"
+  // for each subsequent parameter.

[dcheng, 2016/08/01 14:50:34]

Is it important to fix invalid URLs? Wouldn't they be broken anyway?

[kdsilva, 2016/08/02 15:34:46]

The Flash video player has some URL correction capabilities, so we didn't want
the move to HTML5 to break pages that were already working. I've updated the
comment to mention this, let me know if you have any more questions! 

+  size_t index = ret_url.find_first_of("&?");
+  bool invalid_url = index != std::string::npos && ret_url.at(index) == '&';
+
+  if (!invalid_url)
+    return ret_url;
+
+  // ? should appear first before all parameters
+  ret_url.replace(index, 1, "?");
+
+  // Replace all instances of ? (after the first) with &
+  for (size_t pos = index + 1;
+        (pos= ret_url.find("?",pos )) != std::string::npos; pos += 1)
+      ret_url.replace(pos, 1, "&");
+  return ret_url;
+}