Adding Android specific behavior when overriding 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"
@@ skipping 1403 matching lines @@
 
   // 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.
   if (!url.DomainIs("youtube.com") && !url.DomainIs("youtube-nocookie.com"))
     return GURL();
   if (url.path().find("/v/") != 0)
     return GURL();
 
   std::string url_str = url.spec();
+  internal::YouTubeRewriteStatus result = internal::NUM_PLUGIN_ERROR;
 
   // 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. We do this because the Flash video player
   // has some URL correction capabilities so we don't want this move to HTML5
   // to break webpages that used to work.
   size_t index = url_str.find_first_of("&?");
   bool invalid_url = index != std::string::npos && url_str.at(index) == '&';
 
   if (invalid_url) {
     // ? should appear first before all parameters
     url_str.replace(index, 1, "?");
 
     // Replace all instances of ? (after the first) with &
     for (size_t pos = index + 1;
          (pos = url_str.find("?", pos)) != std::string::npos; pos += 1) {
       url_str.replace(pos, 1, "&");
     }
   }
 
   GURL corrected_url = GURL(url_str);
-
-  // We don't modify any URLs that contain the enablejsapi=1 parameter
-  // since the page may be interacting with the YouTube Flash player in
-  // Javascript and we don't want to break working content.
+  // Unless we're on an Android device, we don't modify any URLs that contain
+  // the enablejsapi=1 parameter since the page may be interacting with the
+  // YouTube Flash player in Javascript and we don't want to break working
+  // content. If we're on an Android device and the URL contains the
+  // enablejsapi=1 parameter, we do override the URL.
   if (corrected_url.query().find("enablejsapi=1") != std::string::npos) {
+#if defined(OS_ANDROID)
+    result = internal::SUCCESS_ENABLEJSAPI;
+#else
     RecordYouTubeRewriteUMA(internal::FAILURE_ENABLEJSAPI);
     return GURL();
+#endif
   }
 
   // Change the path to use the YouTube HTML5 API
   std::string path = corrected_url.path();
   path.replace(path.find("/v/"), 3, "/embed/");
 
   url::Replacements<char> r;
   r.SetPath(path.c_str(), url::Component(0, path.length()));
 
-  RecordYouTubeRewriteUMA(invalid_url ? internal::SUCCESS_PARAMS_REWRITE
-                                      : internal::SUCCESS);
+  if (result == internal::NUM_PLUGIN_ERROR)
+    result = invalid_url ? internal::SUCCESS_PARAMS_REWRITE : internal::SUCCESS;
+
+  RecordYouTubeRewriteUMA(result);
   return corrected_url.ReplaceComponents(r);
 }