Quote CppBoundClass/CppVariant in namespace.

webkit/glue/cpp_binding_example.cc

-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// 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.
 
 // This file contains the definition for CppBindingExample, which is used in
 // cpp_bound_class_unittest.
 
 #include "cpp_binding_example.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 
+namespace webkit_glue {

[tony, 2012/05/08 20:52:26]

Nit: Please move this after the anonymous namespace.

[Xianzhu, 2012/05/09 16:38:24]

I put it here to avoid webkit_glue:: prefixes in the anonymous namespace block.
Is there any drawback to place anonymous namespace inside of a named namespace?

+
 namespace {
 
 class PropertyCallbackExample : public CppBoundClass::PropertyCallback {
  public:
   virtual bool GetValue(CppVariant* value) {
     value->Set(value_);
     return true;
   }
 
   virtual bool SetValue(const CppVariant& value) {
@@ skipping 99 matching lines @@
 }
 
 void CppBindingExample::same(CppVariant* result) {
   result->Set(42);
 }
 
 void CppBindingExample::fallbackMethod(const CppArgumentList& args,
                                        CppVariant* result) {
   printf("Error: unknown JavaScript method invoked.\n");
 }
+
+}  // namespace webkit_glue