Title: [118894] trunk/Source
Revision
118894
Author
[email protected]
Date
2012-05-29 22:08:36 -0700 (Tue, 29 May 2012)

Log Message

[Chromium] Move fileExists to Platform.h
https://bugs.webkit.org/show_bug.cgi?id=87531

Reviewed by Adam Barth.

Call fileExists through fileUtilities(), not directly.

Source/Platform:

* chromium/public/Platform.h:
(Platform):

Source/WebCore:

* platform/chromium/FileSystemChromium.cpp:
(WebCore::fileExists):

Modified Paths

Diff

Modified: trunk/Source/Platform/ChangeLog (118893 => 118894)


--- trunk/Source/Platform/ChangeLog	2012-05-30 04:37:43 UTC (rev 118893)
+++ trunk/Source/Platform/ChangeLog	2012-05-30 05:08:36 UTC (rev 118894)
@@ -5,6 +5,18 @@
 
         Reviewed by Adam Barth.
 
+        Call fileExists through fileUtilities(), not directly.
+
+        * chromium/public/Platform.h:
+        (Platform):
+
+2012-05-29  Mark Pilgrim  <[email protected]>
+
+        [Chromium] Move fileExists to Platform.h
+        https://bugs.webkit.org/show_bug.cgi?id=87531
+
+        Reviewed by Adam Barth.
+
         Part of a refactoring series. See tracking bug 82948.
 
         * chromium/public/Platform.h:

Modified: trunk/Source/Platform/chromium/public/Platform.h (118893 => 118894)


--- trunk/Source/Platform/chromium/public/Platform.h	2012-05-30 04:37:43 UTC (rev 118893)
+++ trunk/Source/Platform/chromium/public/Platform.h	2012-05-30 05:08:36 UTC (rev 118894)
@@ -106,9 +106,7 @@
     // Must return non-null.
     virtual WebFileSystem* fileSystem() { return 0; }
 
-    virtual bool fileExists(const WebString&) { return false; }
 
-
     // Gamepad -------------------------------------------------------------
 
     virtual void sampleGamepads(WebGamepads& into) { into.length = 0; }

Modified: trunk/Source/WebCore/ChangeLog (118893 => 118894)


--- trunk/Source/WebCore/ChangeLog	2012-05-30 04:37:43 UTC (rev 118893)
+++ trunk/Source/WebCore/ChangeLog	2012-05-30 05:08:36 UTC (rev 118894)
@@ -1,3 +1,15 @@
+2012-05-29  Mark Pilgrim  <[email protected]>
+
+        [Chromium] Move fileExists to Platform.h
+        https://bugs.webkit.org/show_bug.cgi?id=87531
+
+        Reviewed by Adam Barth.
+
+        Call fileExists through fileUtilities(), not directly.
+
+        * platform/chromium/FileSystemChromium.cpp:
+        (WebCore::fileExists):
+
 2012-05-29  Kentaro Hara  <[email protected]>
 
         Remove a dummy style sheet from CSSParser::parseFlowThread()

Modified: trunk/Source/WebCore/platform/chromium/FileSystemChromium.cpp (118893 => 118894)


--- trunk/Source/WebCore/platform/chromium/FileSystemChromium.cpp	2012-05-30 04:37:43 UTC (rev 118893)
+++ trunk/Source/WebCore/platform/chromium/FileSystemChromium.cpp	2012-05-30 05:08:36 UTC (rev 118894)
@@ -37,6 +37,7 @@
 #include "PlatformSupport.h"
 
 #include <public/Platform.h>
+#include <public/WebFileUtilities.h>
 
 namespace WebCore {
 
@@ -90,7 +91,7 @@
 
 bool fileExists(const String& path)
 {
-    return WebKit::Platform::current()->fileExists(path);
+    return WebKit::Platform::current()->fileUtilities()->fileExists(path);
 }
 
 PlatformFileHandle openFile(const String& path, FileOpenMode mode)
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to