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

Unified Diff: ppapi/proxy/ppp_printing_proxy.cc

Issue 552423003: PPAPI: Make CallWhileUnlocked more permissive. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 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
« no previous file with comments | « ppapi/proxy/ppp_content_decryptor_private_proxy.cc ('k') | ppapi/proxy/video_capture_resource.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/proxy/ppp_printing_proxy.cc
diff --git a/ppapi/proxy/ppp_printing_proxy.cc b/ppapi/proxy/ppp_printing_proxy.cc
index 9d58c7801110100fc7c6a627bdcf764c903adaba..7b2ae55d9277d9ae7de95cb0d9bcc96559a36c54 100644
--- a/ppapi/proxy/ppp_printing_proxy.cc
+++ b/ppapi/proxy/ppp_printing_proxy.cc
@@ -167,11 +167,8 @@ void PPP_Printing_Proxy::OnPluginMsgBegin(PP_Instance instance,
return;
memcpy(&settings, &settings_string[0], sizeof(settings));
- if (ppp_printing_impl_) {
- *result = CallWhileUnlocked(ppp_printing_impl_->Begin,
- instance,
- const_cast<const PP_PrintSettings_Dev*>(&settings));
- }
+ if (ppp_printing_impl_)
+ *result = CallWhileUnlocked(ppp_printing_impl_->Begin, instance, &settings);
}
void PPP_Printing_Proxy::OnPluginMsgPrintPages(
@@ -183,7 +180,7 @@ void PPP_Printing_Proxy::OnPluginMsgPrintPages(
PP_Resource plugin_resource = CallWhileUnlocked(
ppp_printing_impl_->PrintPages,
- instance, &pages[0], static_cast<uint32_t>(pages.size()));
+ instance, &pages[0], pages.size());
ResourceTracker* resource_tracker = PpapiGlobals::Get()->GetResourceTracker();
Resource* resource_object = resource_tracker->GetResource(plugin_resource);
if (!resource_object)
« no previous file with comments | « ppapi/proxy/ppp_content_decryptor_private_proxy.cc ('k') | ppapi/proxy/video_capture_resource.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698