Enable Oilpan by default in modules/filesystem/

Source/modules/filesystem/FileWriter.cpp

Index: Source/modules/filesystem/FileWriter.cpp
diff --git a/Source/modules/filesystem/FileWriter.cpp b/Source/modules/filesystem/FileWriter.cpp
index b352f4c512a96bd7dab48030de4d7517e92a28e0..5f2d070df80c2a083c42dffc121c556f00a5de3a 100644
--- a/Source/modules/filesystem/FileWriter.cpp
+++ b/Source/modules/filesystem/FileWriter.cpp
@@ -46,7 +46,7 @@ static const double progressNotificationIntervalMS = 50;
 
 PassRefPtrWillBeRawPtr<FileWriter> FileWriter::create(ExecutionContext* context)

[haraken, 2014/06/07 08:11:04]

This should be FileWriter*.

Since FileWriter is still RefCountedGarbageCollected, compilers won't complain
about remaining RefPtr<FileWriter> :)

[sof, 2014/06/07 13:18:53]

I'm not yet making EventTarget-derived objects live on the heap by default. Are
we ready for that yet with Oilpan disabled?

[haraken, 2014/06/07 16:45:27]

I think yes. We need to keep FileWriter being RefCountedGarbageCollected, but we
can remove RefPtr<FileWriter> from the code base. RefPtrs to FileWriter objects
should be allowed only when they are treated as EventTargets.

For example, IDBDatabase is an EventTarget but IDBDatabase::create returns a raw
pointer.

[sof, 2014/06/08 16:34:34]

Thanks for the clarification, that's good. Switched to having create() return a
bare pointer.

 {
-    RefPtrWillBeRawPtr<FileWriter> fileWriter(adoptRefWillBeRefCountedGarbageCollected(new FileWriter(context)));
+    RefPtrWillBeRawPtr<FileWriter> fileWriter(adoptRefCountedGarbageCollected(new FileWriter(context)));
     fileWriter->suspendIfNeeded();
     return fileWriter.release();
 }