PPAPI: Make TrackedCallback more threadsafe

ppapi/proxy/file_mapping_resource.cc

 // Copyright 2014 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/proxy/file_mapping_resource.h"
 
 #include "base/bind.h"
 #include "base/numerics/safe_conversions.h"
 #include "base/task_runner_util.h"
 #include "ppapi/c/pp_errors.h"
@@ skipping 119 matching lines @@
 
 int64_t FileMappingResource::GetMapPageSize(PP_Instance /* instance */) {
   return DoGetMapPageSize();
 }
 
 void FileMappingResource::OnMapCompleted(
     void** mapped_address_out_param,
     int64_t length,
     scoped_refptr<TrackedCallback> callback,
     const MapResult& map_result) {
-  if (callback->aborted()) {
-    if (map_result.result == PP_OK) {
-      // If the Map operation was successful, we need to Unmap to avoid leaks.
-      // The plugin won't get the address, so doesn't have a chance to do the
-      // Unmap.
-      PpapiGlobals::Get()->GetFileTaskRunner()->PostTask(
-          FROM_HERE,
-          base::Bind(base::IgnoreResult(&FileMappingResource::DoUnmapBlocking),
-                     map_result.address,
-                     length));
-    }
-    return;
-  }
+  // FIXME, just delete this whole file when merging.

[dmichael (off chromium), 2015/02/27 22:04:40]

When I can rebase this on top of removing FileMapping, this change will go
away...  this just makes it compile/pass by ignoring the very special case :)

[bbudge, 2015/03/05 01:29:35]

OK.

   if (map_result.result == PP_OK)
     *mapped_address_out_param = map_result.address;
   if (!callback->is_blocking())
     callback->Run(map_result.result);
 }
 
 }  // namespace proxy
 }  // namespace ppapi