Windows: Make it possible to print text with GDI.

chrome/browser/printing/print_job_worker.h

Index: chrome/browser/printing/print_job_worker.h
diff --git a/chrome/browser/printing/print_job_worker.h b/chrome/browser/printing/print_job_worker.h
index 4bebe01a7cc35ddea91e42e3ca013c16b468d0d6..15bd1156b6332540aed283a39b2b7351c8ad471d 100644
--- a/chrome/browser/printing/print_job_worker.h
+++ b/chrome/browser/printing/print_job_worker.h
@@ -45,12 +45,12 @@ class PrintJobWorker {
   // Initializes the print settings. If |ask_user_for_settings| is true, a
   // Print... dialog box will be shown to ask the user his preference.
   // |is_scripted| should be true for calls coming straight from window.print().
-  void GetSettings(
-      bool ask_user_for_settings,
-      int document_page_count,
-      bool has_selection,
-      MarginType margin_type,
-      bool is_scripted);
+  void GetSettings(bool ask_user_for_settings,
+                   int document_page_count,
+                   bool has_selection,
+                   MarginType margin_type,
+                   bool is_scripted,
+                   bool is_modifiable);

[scottmg, 2016/06/30 17:48:30]

This is probably just my lack of knowledge of this area, but I don't know what
is_modifiable means here. Maybe you could add a comment here, and/or the other
.h, and/or where the ipc struct is filled out.

[Lei Zhang, 2016/06/30 22:29:34]

I added a comment here. There's several "is_modifiable" members in
components/printing/common/print_messages.h, and some of them are documented, so
there's probably enough documentation there. They all mean the same thing.

 
   // Set the new print settings.
   void SetSettings(std::unique_ptr<base::DictionaryValue> new_settings);