Apply automatic range checks to storage enum types across IPC in android_webview

android_webview/common/print_messages.h

 // Copyright 2013 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.
 
 // IPC messages for printing.
 // Multiply-included message file, hence no include guard.
 
 // TODO(sgurun) copied from chrome/common. Remove after crbug.com/322276
 
 #include <string>
@@ skipping 59 matching lines @@
   bool is_modifiable;
   bool webnode_only;
   bool has_selection;
   bool selection_only;
 };
 
 #endif  // AW_COMMON_PRINT_MESSAGES_H_
 
 #define IPC_MESSAGE_START PrintMsgStart
 
-IPC_ENUM_TRAITS(printing::MarginType)
-IPC_ENUM_TRAITS(blink::WebPrintScalingOption)
+IPC_ENUM_TRAITS_MAX_VALUE(printing::MarginType,
+                          printing::iMarginType::MARGIN_TYPE_LAST)

[nasko, 2014/09/29 15:18:04]

You have a typo in "iMarginType".

[MRV, 2014/09/30 02:52:47]

Done.

+
+IPC_ENUM_TRAITS_MAX_VALUE(
+    blink::WebPrintScalingOption,
+    blink::WebPrintScalingOption::WebPrintScalingOptionLast)
 
 // Parameters for a render request.
 IPC_STRUCT_TRAITS_BEGIN(PrintMsg_Print_Params)
   // Physical size of the page, including non-printable margins,
   // in pixels according to dpi.
   IPC_STRUCT_TRAITS_MEMBER(page_size)
 
   // In pixels according to dpi_x and dpi_y.
   IPC_STRUCT_TRAITS_MEMBER(content_size)
 
@@ skipping 343 matching lines @@
 IPC_SYNC_MESSAGE_ROUTED0_0(PrintHostMsg_SetupScriptedPrintPreview)
 
 // Tell the browser to show the print preview, when the document is sufficiently
 // loaded such that the renderer can determine whether it is modifiable or not.
 IPC_MESSAGE_ROUTED1(PrintHostMsg_ShowScriptedPrintPreview,
                     bool /* is_modifiable */)
 
 // Notify the browser that the PDF in the initiator renderer has disabled print
 // scaling option.
 IPC_MESSAGE_ROUTED0(PrintHostMsg_PrintPreviewScalingDisabled)