PPAPI: Purposely leak ProxyLock, fix shutdown race

ppapi/shared_impl/test_globals.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 "ppapi/shared_impl/test_globals.h"
 
 namespace ppapi {
 
 TestGlobals::TestGlobals()
     : ppapi::PpapiGlobals(),
       resource_tracker_(ResourceTracker::THREAD_SAFE),
-      callback_tracker_(new CallbackTracker) {
+      callback_tracker_(new CallbackTracker),
+      lock_(NULL) {
 }
 
 TestGlobals::TestGlobals(PpapiGlobals::PerThreadForTest per_thread_for_test)
     : ppapi::PpapiGlobals(per_thread_for_test),
       resource_tracker_(ResourceTracker::THREAD_SAFE),
-      callback_tracker_(new CallbackTracker) {
+      callback_tracker_(new CallbackTracker),
+      lock_(NULL) {
 }
 
 TestGlobals::~TestGlobals() {
 }
 
 ResourceTracker* TestGlobals::GetResourceTracker() {
   return &resource_tracker_;
 }
 
 VarTracker* TestGlobals::GetVarTracker() {
@@ skipping 20 matching lines @@
 }
 
 std::string TestGlobals::GetCmdLine() {
   return std::string();
 }
 
 void TestGlobals::PreCacheFontForFlash(const void* /* logfontw */) {
 }
 
 base::Lock* TestGlobals::GetProxyLock() {
-  return NULL;
+  return lock_;
 }
 
 void TestGlobals::LogWithSource(PP_Instance instance,
                                 PP_LogLevel level,
                                 const std::string& source,
                                 const std::string& value) {
 }
 
 void TestGlobals::BroadcastLogWithSource(PP_Module module,
                                          PP_LogLevel level,
                                          const std::string& source,
                                          const std::string& value) {
 }
 
 MessageLoopShared* TestGlobals::GetCurrentMessageLoop() {
   return NULL;
 }
 
 base::TaskRunner* TestGlobals::GetFileTaskRunner(PP_Instance instance) {
   return NULL;
 }
 
 bool TestGlobals::IsHostGlobals() const {
   // Pretend to be the host-side, for code that expects one or the other.
   // TODO(dmichael): just make it settable which one we're pretending to be?
   return true;
 }
 
 }  // namespace ppapi