Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(216)

Unified Diff: chrome/renderer/chrome_content_renderer_client.cc

Issue 2468423009: Remove plugins::MobileYouTubePlugin, deprecated by Flash embed override. (Closed)
Patch Set: Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/renderer/resources/plugins/mobile_youtube_plugin.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/renderer/chrome_content_renderer_client.cc
diff --git a/chrome/renderer/chrome_content_renderer_client.cc b/chrome/renderer/chrome_content_renderer_client.cc
index df645908b681a27c8d4342d601a2d1d137ff419c..36dd62a209f3baf3f6c45f8ac17eb2010a9adba4 100644
--- a/chrome/renderer/chrome_content_renderer_client.cc
+++ b/chrome/renderer/chrome_content_renderer_client.cc
@@ -75,7 +75,6 @@
#include "components/network_hints/renderer/prescient_networking_dispatcher.h"
#include "components/password_manager/content/renderer/credential_manager_client.h"
#include "components/pdf/renderer/pepper_pdf_host.h"
-#include "components/plugins/renderer/mobile_youtube_plugin.h"
#include "components/signin/core/common/profile_management_switches.h"
#include "components/startup_metric_utils/common/startup_metric.mojom.h"
#include "components/subresource_filter/content/renderer/ruleset_dealer.h"
@@ -137,6 +136,8 @@
#include "chrome/common/plugin_utils.h"
#include "chrome/renderer/plugins/chrome_plugin_placeholder.h"
#include "chrome/renderer/plugins/power_saver_info.h"
+#else
+#include "components/plugins/renderer/plugin_placeholder.h"
#endif
#if BUILDFLAG(ENABLE_PRINTING)
@@ -571,18 +572,6 @@ bool ChromeContentRendererClient::OverrideCreatePlugin(
orig_mime_type, &output));
*plugin = CreatePlugin(render_frame, frame, params, output);
#else // !defined(ENABLE_PLUGINS)
-
-#if defined(OS_ANDROID)
- if (plugins::MobileYouTubePlugin::IsYouTubeURL(url, orig_mime_type)) {
- base::StringPiece template_html(
- ResourceBundle::GetSharedInstance().GetRawDataResource(
- IDR_MOBILE_YOUTUBE_PLUGIN_HTML));
- *plugin = (new plugins::MobileYouTubePlugin(render_frame, frame, params,
- template_html))->plugin();
- return true;
- }
-#endif // defined(OS_ANDROID)
-
PluginUMAReporter::GetInstance()->ReportPluginMissing(orig_mime_type, url);
*plugin = NonLoadablePluginPlaceholder::CreateNotSupportedPlugin(
render_frame, frame, params)->plugin();
« no previous file with comments | « no previous file | chrome/renderer/resources/plugins/mobile_youtube_plugin.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698