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

Unified Diff: content/renderer/pepper/pepper_flash_renderer_host.cc

Issue 11415140: Refactor 3 PPB_Flash functions to the new resource model. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 8 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
Index: content/renderer/pepper/pepper_flash_renderer_host.cc
diff --git a/content/renderer/pepper/pepper_flash_renderer_host.cc b/content/renderer/pepper/pepper_flash_renderer_host.cc
new file mode 100644
index 0000000000000000000000000000000000000000..e651a61c1e10c44091a2ced9c8b26075251ca589
--- /dev/null
+++ b/content/renderer/pepper/pepper_flash_renderer_host.cc
@@ -0,0 +1,55 @@
+// 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 "content/renderer/pepper/pepper_flash_renderer_host.h"
+
+#include "content/common/view_messages.h"
+#include "content/public/renderer/renderer_ppapi_host.h"
+#include "content/renderer/render_thread_impl.h"
+#include "googleurl/src/gurl.h"
+#include "ipc/ipc_message_macros.h"
+#include "ppapi/c/pp_errors.h"
+#include "ppapi/host/dispatch_host_message.h"
+#include "ppapi/proxy/ppapi_messages.h"
+#include "ppapi/proxy/resource_message_params.h"
+
+namespace content {
+
+PepperFlashRendererHost::PepperFlashRendererHost(
+ RendererPpapiHost* host,
+ PP_Instance instance,
+ PP_Resource resource)
+ : ResourceHost(host->GetPpapiHost(), instance, resource) {
+}
+
+PepperFlashRendererHost::~PepperFlashRendererHost() {
+}
+
+int32_t PepperFlashRendererHost::OnResourceMessageReceived(
+ const IPC::Message& msg,
+ ppapi::host::HostMessageContext* context) {
+ IPC_BEGIN_MESSAGE_MAP(PepperFlashRendererHost, msg)
+ PPAPI_DISPATCH_HOST_RESOURCE_CALL(PpapiHostMsg_Flash_GetProxyForURL,
+ OnMsgGetProxyForURL);
+ IPC_END_MESSAGE_MAP()
+ return PP_ERROR_FAILED;
+}
+
+int32_t PepperFlashRendererHost::OnMsgGetProxyForURL(
+ ppapi::host::HostMessageContext* host_context,
+ const std::string& url) {
+ bool result;
+ std::string proxy;
+ GURL gurl(url);
+ if (!gurl.is_valid())
+ return PP_ERROR_FAILED;
+ RenderThreadImpl::current()->Send(
+ new ViewHostMsg_ResolveProxy(gurl, &result, &proxy));
+ if (!result)
+ return PP_ERROR_FAILED;
+ host_context->reply_msg = PpapiPluginMsg_Flash_GetProxyForURLReply(proxy);
victorhsieh 2012/11/28 07:09:50 you don't need to host()->SendReply?
raymes 2012/11/29 00:08:08 Nope. If you handle the message synchronously then
victorhsieh 2012/11/29 06:17:23 Hmm.. we could have a "WillSyncReply" or something
+ return PP_OK;
+}
+
+} // namespace content

Powered by Google App Engine
This is Rietveld 408576698