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

Unified Diff: ppapi/proxy/pdf_resource.cc

Issue 554893002: [WORK_IN_PROGRESS] PDF::Save() fix for out-of-process-pdf. Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: better fix for constrained windows 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/pdf_resource.h ('k') | ppapi/proxy/ppapi_messages.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/proxy/pdf_resource.cc
diff --git a/ppapi/proxy/pdf_resource.cc b/ppapi/proxy/pdf_resource.cc
index 1398866e924e419b2a37cfd953024fddd6333d7b..7f085b1105dc40400530d23803e784f7509daf02 100644
--- a/ppapi/proxy/pdf_resource.cc
+++ b/ppapi/proxy/pdf_resource.cc
@@ -141,8 +141,15 @@ void PDFResource::Print() {
Post(RENDERER, PpapiHostMsg_PDF_Print());
}
-void PDFResource::SaveAs() {
- Post(RENDERER, PpapiHostMsg_PDF_SaveAs());
+void PDFResource::SaveAs(const PP_Var& url) {
+ printf("PDFResource::SaveAs() would send PpapiHostMsg_PDF_SaveAs()\n");
+ scoped_refptr<ppapi::StringVar> url_str(
+ ppapi::StringVar::FromPPVar(url));
+ std::string url_to_use;
+ if (url_str.get()) {
+ url_to_use = url_str->value();
+ }
+ Post(RENDERER, PpapiHostMsg_PDF_SaveAs(url_to_use));
}
PP_Bool PDFResource::IsFeatureEnabled(PP_PDFFeature feature) {
« no previous file with comments | « ppapi/proxy/pdf_resource.h ('k') | ppapi/proxy/ppapi_messages.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698