Added PPB function to return default print settings

ppapi/proxy/ppb_instance_proxy.cc

 // 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 "ppapi/proxy/ppb_instance_proxy.h"
 
 #include "ppapi/c/pp_errors.h"
 #include "ppapi/c/pp_time.h"
 #include "ppapi/c/pp_var.h"
 #include "ppapi/c/ppb_audio_config.h"
@@ skipping 106 matching lines @@
     IPC_MESSAGE_HANDLER(PpapiHostMsg_PPBInstance_RequestInputEvents,
                         OnHostMsgRequestInputEvents)
     IPC_MESSAGE_HANDLER(PpapiHostMsg_PPBInstance_ClearInputEvents,
                         OnHostMsgClearInputEvents)
     IPC_MESSAGE_HANDLER(PpapiMsg_PPPInputEvent_HandleInputEvent_ACK,
                         OnMsgHandleInputEventAck)
     IPC_MESSAGE_HANDLER(PpapiHostMsg_PPBInstance_LockMouse,
                         OnHostMsgLockMouse)
     IPC_MESSAGE_HANDLER(PpapiHostMsg_PPBInstance_UnlockMouse,
                         OnHostMsgUnlockMouse)
+    IPC_MESSAGE_HANDLER(PpapiMsg_PPBPrinting_GetDefaultPrintSettings,
+                        OnHostMsgGetDefaultPrintSettings)
     IPC_MESSAGE_HANDLER(PpapiHostMsg_PPBInstance_SetCursor,
                         OnHostMsgSetCursor)
     IPC_MESSAGE_HANDLER(PpapiHostMsg_PPBInstance_SetTextInputType,
                         OnHostMsgSetTextInputType)
     IPC_MESSAGE_HANDLER(PpapiHostMsg_PPBInstance_UpdateCaretPosition,
                         OnHostMsgUpdateCaretPosition)
     IPC_MESSAGE_HANDLER(PpapiHostMsg_PPBInstance_CancelCompositionText,
                         OnHostMsgCancelCompositionText)
     IPC_MESSAGE_HANDLER(PpapiHostMsg_PPBInstance_UpdateSurroundingText,
                         OnHostMsgUpdateSurroundingText)
@@ skipping 314 matching lines @@
   dispatcher()->Send(new PpapiHostMsg_PPBInstance_LockMouse(
       API_ID_PPB_INSTANCE, instance));
   return PP_OK_COMPLETIONPENDING;
 }
 
 void PPB_Instance_Proxy::UnlockMouse(PP_Instance instance) {
   dispatcher()->Send(new PpapiHostMsg_PPBInstance_UnlockMouse(
       API_ID_PPB_INSTANCE, instance));
 }
 
+PP_Bool PPB_Instance_Proxy::GetDefaultPrintSettings(
+    PP_Instance instance,
+    PP_PrintSettings_Dev* print_settings) {
+  std::string settings_string;
+  bool result;
+  dispatcher()->Send(new PpapiMsg_PPBPrinting_GetDefaultPrintSettings(
+      API_ID_PPB_INSTANCE, instance, &settings_string, &result));
+
+  if (result && print_settings &&

[yzshen1, 2012/06/12 23:14:34]

You could test print_settings at the top of the method and fail early.

[raymes, 2012/06/13 18:23:47]

Done.

+      settings_string.size() == sizeof(*print_settings)) {
+    memcpy(print_settings, &settings_string[0], sizeof(*print_settings));

[yzshen1, 2012/06/12 23:14:34]

I don't think it is a good idea to pass a raw buffer.
Please pass the structure itself. :)

[raymes, 2012/06/13 18:23:47]

This is what was done in the existing code that transferred a
PP_PrintSettings_Dev so I just copied the approach. But I agree that passing the
struct is better.

+    return PP_TRUE;
+  }
+  return PP_FALSE;
+}
+
 void PPB_Instance_Proxy::SetTextInputType(PP_Instance instance,
                                           PP_TextInput_Type type) {
   dispatcher()->Send(new PpapiHostMsg_PPBInstance_SetTextInputType(
       API_ID_PPB_INSTANCE, instance, type));
 }
 
 void PPB_Instance_Proxy::UpdateCaretPosition(PP_Instance instance,
                                              const PP_Rect& caret,
                                              const PP_Rect& bounding_box) {
   dispatcher()->Send(new PpapiHostMsg_PPBInstance_UpdateCaretPosition(
@@ skipping 172 matching lines @@
   if (result != PP_OK_COMPLETIONPENDING)
     cb.Run(result);
 }
 
 void PPB_Instance_Proxy::OnHostMsgUnlockMouse(PP_Instance instance) {
   EnterInstanceNoLock enter(instance);
   if (enter.succeeded())
     enter.functions()->UnlockMouse(instance);
 }
 
+void PPB_Instance_Proxy::OnHostMsgGetDefaultPrintSettings(
+    PP_Instance instance,
+    std::string* settings_string,
+    bool* result) {
+  // TODO(raymes): This just returns some generic settings. Actually hook this
+  // up to the browser to return the real defaults.
+  PP_PrintSettings_Dev generic_print_settings = {
+      // |printable_area|: all of the sheet of paper.
+      { { 0, 0 }, { 612, 792 } },
+      // |content_area|: 0.5" margins all around.
+      { { 36, 36 }, { 540, 720 } },
+      // |paper_size|: 8.5" x 11" (US letter).
+      { 612, 792 },
+      300,  // |dpi|.
+      PP_PRINTORIENTATION_NORMAL,  // |orientation|.
+      PP_PRINTSCALINGOPTION_NONE,  // |print_scaling_option|.
+      PP_FALSE,  // |grayscale|.
+      PP_PRINTOUTPUTFORMAT_PDF  // |format|.
+  };
+  settings_string->resize(sizeof(generic_print_settings));
+  memcpy(&(*settings_string)[0], &generic_print_settings,
+      sizeof(generic_print_settings));
+  *result = true;
+}
+
 #if !defined(OS_NACL)
 void PPB_Instance_Proxy::OnHostMsgResolveRelativeToDocument(
     PP_Instance instance,
     SerializedVarReceiveInput relative,
     SerializedVarReturnValue result) {
   EnterInstanceNoLock enter(instance);
   if (enter.succeeded()) {
     result.Return(dispatcher(),
                   enter.functions()->ResolveRelativeToDocument(
                       instance, relative.Get(dispatcher()), NULL));
@@ skipping 101 matching lines @@
 }
 
 void PPB_Instance_Proxy::MouseLockCompleteInHost(int32_t result,
                                                  PP_Instance instance) {
   dispatcher()->Send(new PpapiMsg_PPBInstance_MouseLockComplete(
       API_ID_PPB_INSTANCE, instance, result));
 }
 
 }  // namespace proxy
 }  // namespace ppapi