Quote CppBoundClass/CppVariant in namespace.

webkit/glue/cpp_bound_class.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 definitions for CppBoundClass
 
 // Here's the control flow of a JS method getting forwarded to a class.
 // - Something calls our NPObject with a function like "Invoke".
 // - CppNPObject's static invoke() function forwards it to its attached
 //   CppBoundClass's Invoke() method.
 // - CppBoundClass has then overridden Invoke() to look up the function
 //   name in its internal map of methods, and then calls the appropriate
 //   method.
 
 #include "base/compiler_specific.h"
 #include "base/logging.h"
 #include "base/stl_util.h"
 #include "base/utf_string_conversions.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebBindings.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebFrame.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebString.h"
 #include "webkit/glue/cpp_bound_class.h"
 
 using WebKit::WebBindings;
 using WebKit::WebFrame;
 
+namespace webkit_glue {

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

Nit: Please move this after the anonymous namespace.

+
 namespace {
 
 class CppVariantPropertyCallback : public CppBoundClass::PropertyCallback {
  public:
   CppVariantPropertyCallback(CppVariant* value) : value_(value) { }
 
   virtual bool GetValue(CppVariant* value) {
     value->Set(*value_);
     return true;
   }
@@ skipping 280 matching lines @@
   JSC::JSLock lock(false);
 #endif
 
   // BindToWindowObject will take its own reference to the NPObject, and clean
   // up after itself.  It will also (indirectly) register the object with V8,
   // so we must remember this so we can unregister it when we're destroyed.
   frame->bindToWindowObject(ASCIIToUTF16(classname),
                             NPVARIANT_TO_OBJECT(*GetAsCppVariant()));
   bound_to_frame_ = true;
 }
+
+}  // namespace webkit_glue