Assert sequence validity on non-thread-safe RefCount manipulations (2)

ppapi/shared_impl/tracked_callback.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/tracked_callback.h"
 
 #include "base/bind.h"
 #include "base/compiler_specific.h"
 #include "base/location.h"
 #include "base/logging.h"
@@ skipping 210 matching lines @@
   DCHECK(!completed_);
 
   // We will be removed; maintain a reference to ensure we won't be deleted
   // until we're done.
   scoped_refptr<TrackedCallback> thiz = this;
   completed_ = true;
   // We may not have a valid resource, in which case we're not in the tracker.
   if (resource_id_)
     tracker_->Remove(thiz);
   tracker_ = NULL;
+
+  // Relax the cross-thread access restriction to non-thread-safe RefCount.
+  // |lock_| protects the access to Resource instances.
+  base::ScopedAllowCrossThreadRefCountAccess
+      allow_cross_thread_ref_count_access;
   target_loop_ = NULL;
 }
 
 void TrackedCallback::PostRunWithLock(int32_t result) {
   lock_.AssertAcquired();
   if (completed_) {
     NOTREACHED();
     return;
   }
   if (result == PP_ERROR_ABORTED)
@@ skipping 40 matching lines @@
   // tracker. Then MarkAsCompleted before waking up the blocked thread,
   // which could potentially re-enter.
   scoped_refptr<TrackedCallback> thiz(this);
   MarkAsCompletedWithLock();
   // Wake up the blocked thread. See BlockUntilComplete for where the thread
   // Wait()s.
   operation_completed_condvar_->Signal();
 }
 
 }  // namespace ppapi