Add support for mojo::TypeConverter to control how convenient conversion should be.

mojo/common/common_type_converters.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.
 
 #ifndef MOJO_COMMON_COMMON_TYPE_CONVERTERS_H_
 #define MOJO_COMMON_COMMON_TYPE_CONVERTERS_H_
 
 #include "base/strings/string16.h"
 #include "base/strings/string_piece.h"
 #include "mojo/common/mojo_common_export.h"
 #include "mojo/public/cpp/bindings/array.h"
 #include "mojo/public/cpp/bindings/type_converter.h"
 
 namespace mojo {
 
 template <>
 class MOJO_COMMON_EXPORT TypeConverter<String, base::StringPiece> {
  public:
   static String ConvertFrom(const base::StringPiece& input, Buffer* buf);
   static base::StringPiece ConvertTo(const String& input);
+
+  MOJO_ALLOW_DIRECT_TYPE_CONVERSION();

[darin (slow to review), 2014/04/23 18:01:01]

Why don't you like using the term "implicit" here? Isn't that the opposite of
explicit, and in C++ the explicit keyword is what you use to disallow implicit
constructors. Seems like implicit has the right meaning here, no?

MOJO_ALLOW_IMPLICIT_TYPE_CONVERSION()

[yzshen1, 2014/04/23 18:20:57]

Because it sounds like we allow "explicit conversion" without the macro. And
"explicit conversion" makes me think of things such as "A a =
static_cast<A>(b);", which we don't support without the macro.

(But I don't have a very strong preference. I am fine if you think that is
better.)

 };
 
 template <>
 class MOJO_COMMON_EXPORT TypeConverter<String, base::string16> {
  public:
   static String ConvertFrom(const base::string16& input, Buffer* buf);
   static base::string16 ConvertTo(const String& input);
+
+  MOJO_ALLOW_DIRECT_TYPE_CONVERSION();
 };
 
 }  // namespace mojo
 
 #endif  // MOJO_COMMON_COMMON_TYPE_CONVERTERS_H_