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

Unified Diff: chrome/renderer/plugins/chrome_plugin_placeholder.cc

Issue 2154773002: Implement Just-In-Time Flash updates. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: git pull && gclient sync Created 4 years, 5 months 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
Index: chrome/renderer/plugins/chrome_plugin_placeholder.cc
diff --git a/chrome/renderer/plugins/chrome_plugin_placeholder.cc b/chrome/renderer/plugins/chrome_plugin_placeholder.cc
index 0f35197e27cbaacbde7c35fa4d943306f4e8f3fe..03fc57abe7e0fc2cfd153b7fe8f2f44e4a7961b5 100644
--- a/chrome/renderer/plugins/chrome_plugin_placeholder.cc
+++ b/chrome/renderer/plugins/chrome_plugin_placeholder.cc
@@ -74,10 +74,10 @@ ChromePluginPlaceholder::~ChromePluginPlaceholder() {
if (context_menu_request_id_ && render_frame())
render_frame()->CancelContextMenu(context_menu_request_id_);
-#if defined(ENABLE_PLUGIN_INSTALLATION)
if (placeholder_routing_id_ == MSG_ROUTING_NONE)
return;
RenderThread::Get()->RemoveRoute(placeholder_routing_id_);
+#if defined(ENABLE_PLUGIN_INSTALLATION)
if (has_host_) {
RenderThread::Get()->Send(new ChromeViewHostMsg_RemovePluginPlaceholderHost(
routing_id(), placeholder_routing_id_));
@@ -175,18 +175,16 @@ void ChromePluginPlaceholder::SetStatus(
status_ = status;
}
-#if defined(ENABLE_PLUGIN_INSTALLATION)
int32_t ChromePluginPlaceholder::CreateRoutingId() {
placeholder_routing_id_ = RenderThread::Get()->GenerateRoutingID();
RenderThread::Get()->AddRoute(placeholder_routing_id_, this);
return placeholder_routing_id_;
}
-#endif
bool ChromePluginPlaceholder::OnMessageReceived(const IPC::Message& message) {
-#if defined(ENABLE_PLUGIN_INSTALLATION)
bool handled = true;
IPC_BEGIN_MESSAGE_MAP(ChromePluginPlaceholder, message)
+#if defined(ENABLE_PLUGIN_INSTALLATION)
IPC_MESSAGE_HANDLER(ChromeViewMsg_FoundMissingPlugin, OnFoundMissingPlugin)
IPC_MESSAGE_HANDLER(ChromeViewMsg_DidNotFindMissingPlugin,
OnDidNotFindMissingPlugin)
@@ -198,12 +196,18 @@ bool ChromePluginPlaceholder::OnMessageReceived(const IPC::Message& message) {
OnErrorDownloadingPlugin)
IPC_MESSAGE_HANDLER(ChromeViewMsg_CancelledDownloadingPlugin,
OnCancelledDownloadingPlugin)
+#endif
+ IPC_MESSAGE_HANDLER(ChromeViewMsg_PluginComponentUpdateDownloading,
+ OnPluginComponentUpdateDownloading)
+ IPC_MESSAGE_HANDLER(ChromeViewMsg_PluginComponentUpdateSuccess,
+ OnPluginComponentUpdateSuccess)
+ IPC_MESSAGE_HANDLER(ChromeViewMsg_PluginComponentUpdateFailure,
+ OnPluginComponentUpdateFailure)
IPC_MESSAGE_UNHANDLED(handled = false)
IPC_END_MESSAGE_MAP()
if (handled)
return true;
-#endif
// We don't swallow these messages because multiple blocked plugins and other
// objects have an interest in them.
@@ -257,6 +261,19 @@ void ChromePluginPlaceholder::OnCancelledDownloadingPlugin() {
}
#endif // defined(ENABLE_PLUGIN_INSTALLATION)
+void ChromePluginPlaceholder::OnPluginComponentUpdateDownloading() {
+ SetMessage(l10n_util::GetStringFUTF16(IDS_PLUGIN_DOWNLOADING, plugin_name_));
+}
+
+void ChromePluginPlaceholder::OnPluginComponentUpdateSuccess() {
+ PluginListChanged();
+}
+
+void ChromePluginPlaceholder::OnPluginComponentUpdateFailure() {
+ SetMessage(l10n_util::GetStringFUTF16(IDS_PLUGIN_DOWNLOAD_ERROR_SHORT,
+ plugin_name_));
+}
+
void ChromePluginPlaceholder::PluginListChanged() {
if (!GetFrame() || !plugin())
return;
« no previous file with comments | « chrome/renderer/plugins/chrome_plugin_placeholder.h ('k') | components/component_updater/component_updater_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698