Scope and clean up uses of AccessibilityMode.

content/common/content_param_traits.cc

 // Copyright (c) 2012 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.
 
 #include "content/common/content_param_traits.h"
 
 #include <stddef.h>
 
 #include "base/strings/string_number_conversions.h"
+#include "content/common/accessibility_mode.h"
 #include "content/common/message_port.h"
 #include "ipc/ipc_mojo_param_traits.h"
 #include "net/base/ip_endpoint.h"
 #include "ui/events/blink/web_input_event_traits.h"
 
 namespace IPC {
 
 void ParamTraits<WebInputEventPointer>::GetSize(base::PickleSizer* s,
                                                 const param_type& p) {
   s->AddData(p->size());
@@ skipping 62 matching lines @@
   if (!ParamTraits<mojo::MessagePipeHandle>::Read(m, iter, &handle))
     return false;
   *r = content::MessagePort(mojo::ScopedMessagePipeHandle(handle));
   return true;
 }
 
 void ParamTraits<content::MessagePort>::Log(const param_type& p,
                                             std::string* l) {
 }
 
+void ParamTraits<content::AccessibilityMode>::GetSize(base::PickleSizer* s,
+                                                      const param_type& p) {
+  IPC::GetParamSize(s, p.mode());
+}
+
+void ParamTraits<content::AccessibilityMode>::Write(base::Pickle* m,
+                                                    const param_type& p) {
+  IPC::WriteParam(m, p.mode());
+}
+
+bool ParamTraits<content::AccessibilityMode>::Read(const base::Pickle* m,
+                                                   base::PickleIterator* iter,
+                                                   param_type* r) {
+  uint32_t value;
+  if (!IPC::ReadParam(m, iter, &value))
+    return false;
+  *r = content::AccessibilityMode(value);
+  return true;
+}
+
+void ParamTraits<content::AccessibilityMode>::Log(const param_type& p,
+                                                  std::string* l) {}

[dmazzoni, 2017/03/13 22:18:15]

nit: any reason not to log this? kind of obscure but i don't see any reason not
to

[dougt, 2017/03/14 03:18:23]

Acknowledged. I didn't really see a need yet. Fwiw, other protocols don't log
(for example MessagePort). Happy to do in a follow up.

 }  // namespace IPC
 
 // Generate param traits size methods.
 #include "ipc/param_traits_size_macros.h"
 namespace IPC {
 #undef CONTENT_COMMON_CONTENT_PARAM_TRAITS_MACROS_H_
 #include "content/common/content_param_traits_macros.h"
 }
 
 // Generate param traits write methods.
 #include "ipc/param_traits_write_macros.h"
 namespace IPC {
 #undef CONTENT_COMMON_CONTENT_PARAM_TRAITS_MACROS_H_
 #include "content/common/content_param_traits_macros.h"
 }  // namespace IPC
 
 // Generate param traits read methods.
 #include "ipc/param_traits_read_macros.h"
 namespace IPC {
 #undef CONTENT_COMMON_CONTENT_PARAM_TRAITS_MACROS_H_
 #include "content/common/content_param_traits_macros.h"
 }  // namespace IPC
 
 // Generate param traits log methods.
 #include "ipc/param_traits_log_macros.h"
 namespace IPC {
 #undef CONTENT_COMMON_CONTENT_PARAM_TRAITS_MACROS_H_
 #include "content/common/content_param_traits_macros.h"
 }  // namespace IPC