Oilpan: add transition types to FileSystem APIs.

Source/modules/filesystem/DirectoryEntrySync.h

Index: Source/modules/filesystem/DirectoryEntrySync.h
diff --git a/Source/modules/filesystem/DirectoryEntrySync.h b/Source/modules/filesystem/DirectoryEntrySync.h
index 739581537345fcc33500de08467e7e6bb51f59f7..4b81b5bf9074eb5e21709a3eb09d0fd12d08e2f5 100644
--- a/Source/modules/filesystem/DirectoryEntrySync.h
+++ b/Source/modules/filesystem/DirectoryEntrySync.h
@@ -45,19 +45,19 @@ class FileEntrySync;
 
 class DirectoryEntrySync FINAL : public EntrySync {

[haraken, 2014/03/07 08:24:35]

Shall we add a trace() method to all GarbageCollected sub-classes? The same
comment for other places in this CL.

[sof, 2014/03/07 09:37:36]

I don't think it adds much value, given the class's only function --
FileEntrySync&DirectoryEntrySync are just here to get the kind/type of the
EntrySync right. Ditto for FileEntry and DirectoryEntry. That's why I didn't add
them.

[haraken, 2014/03/10 01:55:26]

I'd prefer to add (and in fact we have added) trace() methods to all
garbage-collected classes even if the classes don't have any members. This is
for programming consistency. Instead of making developers add trace() methods
every time they add new members, it would be simpler and less error-prone to add
trace() methods when we move the class hierarchy to the heap.

[sof, 2014/03/10 10:01:23]

ok, added the methods here & will follow that.

(It doesn't apply here, but if we can be judicious about adding "empty" trace()s
to classes having no other virtuals, I'd prefer that.)

 public:
-    static PassRefPtr<DirectoryEntrySync> create(PassRefPtr<DOMFileSystemBase> fileSystem, const String& fullPath)
+    static PassRefPtrWillBeRawPtr<DirectoryEntrySync> create(PassRefPtrWillBeRawPtr<DOMFileSystemBase> fileSystem, const String& fullPath)
     {
-        return adoptRef(new DirectoryEntrySync(fileSystem, fullPath));
+        return adoptRefWillBeNoop(new DirectoryEntrySync(fileSystem, fullPath));
     }
     virtual bool isDirectory() const OVERRIDE { return true; }
 
-    PassRefPtr<DirectoryReaderSync> createReader();
-    PassRefPtr<FileEntrySync> getFile(const String& path, const Dictionary&, ExceptionState&);
-    PassRefPtr<DirectoryEntrySync> getDirectory(const String& path, const Dictionary&, ExceptionState&);
+    PassRefPtrWillBeRawPtr<DirectoryReaderSync> createReader();
+    PassRefPtrWillBeRawPtr<FileEntrySync> getFile(const String& path, const Dictionary&, ExceptionState&);
+    PassRefPtrWillBeRawPtr<DirectoryEntrySync> getDirectory(const String& path, const Dictionary&, ExceptionState&);
     void removeRecursively(ExceptionState&);
 
 private:
-    DirectoryEntrySync(PassRefPtr<DOMFileSystemBase>, const String& fullPath);
+    DirectoryEntrySync(PassRefPtrWillBeRawPtr<DOMFileSystemBase>, const String& fullPath);
 };
 
 DEFINE_TYPE_CASTS(DirectoryEntrySync, EntrySync, entry, entry->isDirectory(), entry.isDirectory());