Rename sync and async file system access methods in blink. [blink] (1/4)

Source/web/WorkerPermissionClient.h

Index: Source/web/WorkerPermissionClient.h
diff --git a/Source/web/WorkerPermissionClient.h b/Source/web/WorkerPermissionClient.h
index d9478a26480b9fe587c265551777fda78d69ddd5..afbbec9242810699af88365590852e43782de987 100644
--- a/Source/web/WorkerPermissionClient.h
+++ b/Source/web/WorkerPermissionClient.h
@@ -54,7 +54,8 @@ public:
 
     bool allowDatabase(const WebString& name, const WebString& displayName, unsigned long estimatedSize);
     bool allowFileSystem();
-    void requestFileSystemAccess(const WebPermissionCallbacks&);
+    bool requestFileSystemAccessSync();
+    void requestFileSystemAccessAsync(const WebPermissionCallbacks&);

[kinuko, 2014/05/16 10:18:26]

Hmm, do we need this?

[Xi Han, 2014/05/16 13:05:51]

In this CL, I only do renaming, since this function is still used in
LocalFileSystemClient.cpp now. It will be removed in the following CL (issue
277353002).

On 2014/05/16 10:18:26, kinuko wrote:

[kinuko, 2014/05/19 08:38:04]

It's fine since now it's landed, but I don't think we call this method in
LocalFileSystemClient.cpp any more, even without waiting for issue 277353002.
(Line 78 in the file, where we call
WorkerPermissionClient::from(*toWorkerGlobalScope(context))->requestFileSystemAccessSync()
seems to be the only code LocalFileSystemClient calls into this)

     bool allowIndexedDB(const WebString& name);
 
     static const char* supplementName();