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)
+IPC_ENUM_TRAITS_MAX_VALUE(blink::WebPrintScalingOption,
+                      blink::WebPrintScalingOption::WebPrintScalingOptionLast)

[nasko, 2014/09/26 05:19:27]

Please format per the style guide or use clang-format.

[MRV, 2014/09/26 05:22:50]

Dear Mr.nasko, thanks for your feedback.
I have made changed as per style guide, but it is crossing 80 characters, so I
prioritized in this way.
Please let me know, how to handle this issue.

 
 // 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)