Implement Pepper proxy for PPB_DirectoryReader

ppapi/thunk/ppb_directory_reader_thunk.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/c/dev/ppb_directory_reader_dev.h"
 #include "ppapi/c/pp_completion_callback.h"
 #include "ppapi/c/pp_errors.h"
+#include "ppapi/shared_impl/proxy_lock.h"
 #include "ppapi/shared_impl/tracked_callback.h"
 #include "ppapi/thunk/enter.h"
 #include "ppapi/thunk/thunk.h"
 #include "ppapi/thunk/ppb_directory_reader_api.h"
 #include "ppapi/thunk/resource_creation_api.h"
 
 namespace ppapi {
 namespace thunk {
 
 namespace {
 
 PP_Resource Create(PP_Resource directory_ref) {
+  ppapi::ProxyAutoLock lock;
   Resource* object =
       PpapiGlobals::Get()->GetResourceTracker()->GetResource(directory_ref);
   if (!object)

[yzshen1, 2013/01/24 21:55:43]

You could move this check into CreateDirectoryReader(), right?
Then you don't have to GetResource() twice.

[nhiroki, 2013/01/25 12:27:17]

I see. I removed GetResource() from CreateDirectoryReader() and changed
CreateDirectoryReader() to take PP_Instance object as an argument.

     return 0;
-  EnterResourceCreation enter(object->pp_instance());
+  EnterResourceCreationNoLock enter(object->pp_instance());
   if (enter.failed())
     return 0;
   return enter.functions()->CreateDirectoryReader(directory_ref);
 }
 
 PP_Bool IsDirectoryReader(PP_Resource resource) {
   EnterResource<PPB_DirectoryReader_API> enter(resource, false);
   return PP_FromBool(enter.succeeded());
 }
 
@@ skipping 14 matching lines @@
 };
 
 }  // namespace
 
 const PPB_DirectoryReader_Dev_0_5* GetPPB_DirectoryReader_Dev_0_5_Thunk() {
   return &g_ppb_directory_reader_thunk;
 }
 
 }  // namespace thunk
 }  // namespace ppapi