Substitute boolean GDI printing for a mode type

printing/pdf_render_settings.h

 // Copyright (c) 2011 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.
 
 #ifndef PRINTING_PDF_RENDER_SETTINGS_H_
 #define PRINTING_PDF_RENDER_SETTINGS_H_
 
 #include "ipc/ipc_message_utils.h"
 #include "ipc/ipc_param_traits.h"
 #include "printing/printing_export.h"
 #include "ui/gfx/geometry/rect.h"
 #include "ui/gfx/ipc/geometry/gfx_param_traits.h"
 #include "ui/gfx/ipc/skia/gfx_skia_param_traits.h"
 
 namespace printing {
 
 // Defining PDF rendering settings.
 struct PdfRenderSettings {
-  PdfRenderSettings() : dpi(0), autorotate(false) {}
-  PdfRenderSettings(gfx::Rect area, int dpi, bool autorotate)
-      : area(area), dpi(dpi), autorotate(autorotate) {}
+  enum class Mode : uint8_t {
+    NORMAL = 0,
+#if defined(OS_WIN)
+    GDI_TEXT,
+#endif
+  };
+
+  PdfRenderSettings() : dpi(0), autorotate(false), mode(Mode::NORMAL) {}
+  PdfRenderSettings(gfx::Rect area, int dpi, bool autorotate, Mode mode)
+      : area(area), dpi(dpi), autorotate(autorotate), mode(mode) {}
   ~PdfRenderSettings() {}
 
   gfx::Rect area;
   int dpi;
   bool autorotate;
+  Mode mode;
 };
 
 }  // namespace printing
 
 namespace IPC {
 template <>
 struct ParamTraits<printing::PdfRenderSettings> {
-  typedef printing::PdfRenderSettings param_type;
+  using param_type = printing::PdfRenderSettings;
   static void Write(base::Pickle* m, const param_type& p) {
     WriteParam(m, p.area);
     WriteParam(m, p.dpi);
     WriteParam(m, p.autorotate);
+    WriteParam(m, static_cast<uint8_t>(p.mode));
   }
 
   static bool Read(const base::Pickle* m,
                    base::PickleIterator* iter,
                    param_type* r) {
-    return ReadParam(m, iter, &r->area) &&
-        ReadParam(m, iter, &r->dpi) &&
-        ReadParam(m, iter, &r->autorotate);
+    uint8_t mode = 0;
+    bool ret = ReadParam(m, iter, &r->area) &&
+               ReadParam(m, iter, &r->dpi) &&
+               ReadParam(m, iter, &r->autorotate) &&
+               ReadParam(m, iter, &mode);
+    if (ret)
+      r->mode = static_cast<printing::PdfRenderSettings::Mode>(mode);

[Tom Sepez, 2017/01/23 21:14:08]

Need to validate that mode is in range here or use the IPC macros instead to
generate its onw traits.

[Vitaly Buka (NO REVIEWS), 2017/01/23 22:03:30]

Better to use IPC macros

[rbpotter, 2017/01/24 02:22:01]

Done.

I switched the entire structure to use the IPC macros, which seems to work.
Let me know if I should do something differently. Defining only the enum
but not the structure with the macros did not compile correctly.

+    return ret;
   }
 
   static void Log(const param_type& p, std::string* l) {
     LogParam(p.area, l);
     l->append(", ");
     LogParam(p.dpi, l);
     l->append(", ");
     LogParam(p.autorotate, l);
+    l->append(", ");
+    LogParam(static_cast<uint8_t>(p.mode), l);
   }
 };
 
 }  // namespace IPC
 
 #endif  // PRINTING_PDF_RENDER_SETTINGS_H_