Implement asynchronous interface/plumbing for GetDefaultPrintSettings.

ppapi/cpp/dev/printing_dev.h

Index: ppapi/cpp/dev/printing_dev.h
diff --git a/ppapi/cpp/dev/printing_dev.h b/ppapi/cpp/dev/printing_dev.h
index eb20e563641f039bebad9f100661b71afd9d29fa..4d2278ed5e6accb4782660c18d6daf60f79661cb 100644
--- a/ppapi/cpp/dev/printing_dev.h
+++ b/ppapi/cpp/dev/printing_dev.h
@@ -11,6 +11,7 @@
 
 namespace pp {
 
+class CompletionCallback;
 class Instance;
 
 // You would typically use this either via inheritance on your instance or
@@ -35,10 +36,15 @@ class Printing_Dev {
   // interface.
   static bool IsAvailable();
 
-  // Outputs the default print settings for the default printer into
-  // |print_settings|. Returns false on error.
+  // Deprecated in PPB_Printing_Dev 0.7.
   bool GetDefaultPrintSettings(PP_PrintSettings_Dev* print_settings) const;

[dmichael (off chromium), 2012/07/23 18:20:09]

Can you just get rid of this? Since this is an internal-only interface, I'd
rather not have to worry about source compatibility.

[raymes, 2012/07/24 00:37:08]

If I remove this, flash wont compile I have to do this in 2 commits. I added a
TODO though.

On 2012/07/23 18:20:09, dmichael wrote:

 
+  // Outputs the default print settings for the default printer into
+  // |print_settings|. The completion callback is called when the settings are
+  // ready to be used.
+  int32_t GetDefaultPrintSettings(PP_PrintSettings_Dev* print_settings,
+                                  const CompletionCallback& callback) const;
+
  private:
   InstanceHandle associated_instance_;
 };