Handle has_more correctly

Source/modules/filesystem/DirectoryReader.h

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 38 matching lines @@
     static PassRefPtr<DirectoryReader> create(PassRefPtr<DOMFileSystemBase> fileSystem, const String& fullPath)
     {
         return adoptRef(new DirectoryReader(fileSystem, fullPath));
     }
 
     void readEntries(PassOwnPtr<EntriesCallback>, PassOwnPtr<ErrorCallback> = nullptr);
 
     DOMFileSystem* filesystem() const { return static_cast<DOMFileSystem*>(m_fileSystem.get()); }
 
 private:
+    class EntriesCallbackHelper;
+    class ErrorCallbackHelper;
+
     DirectoryReader(PassRefPtr<DOMFileSystemBase>, const String& fullPath);
+
+    void addEntries(const Vector<RefPtr<Entry> >& entries);
+
+    void onError(FileError*);
+
+    bool m_isReading;
+    Vector<RefPtr<Entry> > m_entries;
+    RefPtr<FileError> m_error;

[haraken, 2014/03/10 03:54:12]

Since FileError is on the oilpan heap, this should be
RefPtrWillBePersistent<FileError>.

[hashimoto, 2014/03/10 04:46:15]

Thank you for the fix!

It seems that what I should have done is to look at FileError.h and follow the
class's inheritance chain until it reaches 
RefCountedWillBeGarbageCollectedFinalized, then look up the table at
http://www.chromium.org/blink/blink-gc.
Though it'd be even nicer if this could be caught by the compiler with something
like static assertions.

+    OwnPtr<EntriesCallback> m_entriesCallback;
+    OwnPtr<ErrorCallback> m_errorCallback;
 };
 
 }
 
 #endif // DirectoryReader_h