Title: [118873] trunk/Source
- Revision
- 118873
- Author
- pilg...@chromium.org
- Date
- 2012-05-29 18:33:50 -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.
Part of a refactoring series. See tracking bug 82948.
Source/Platform:
* chromium/public/Platform.h:
(WebKit::Platform::fileExists):
(Platform):
Source/WebCore:
* platform/chromium/FileSystemChromium.cpp:
(WebCore::fileExists):
* platform/chromium/PlatformSupport.h:
(PlatformSupport):
Source/WebKit/chromium:
* src/PlatformSupport.cpp:
(WebCore):
Modified Paths
Diff
Modified: trunk/Source/Platform/ChangeLog (118872 => 118873)
--- trunk/Source/Platform/ChangeLog 2012-05-30 01:24:51 UTC (rev 118872)
+++ trunk/Source/Platform/ChangeLog 2012-05-30 01:33:50 UTC (rev 118873)
@@ -1,3 +1,16 @@
+2012-05-29 Mark Pilgrim <pilg...@chromium.org>
+
+ [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:
+ (WebKit::Platform::fileExists):
+ (Platform):
+
2012-05-29 Shawn Singh <shawnsi...@chromium.org>
[chromium] make WebTransformationMatrix object usable by non-webkit code
Modified: trunk/Source/Platform/chromium/public/Platform.h (118872 => 118873)
--- trunk/Source/Platform/chromium/public/Platform.h 2012-05-30 01:24:51 UTC (rev 118872)
+++ trunk/Source/Platform/chromium/public/Platform.h 2012-05-30 01:33:50 UTC (rev 118873)
@@ -106,6 +106,9 @@
// 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 (118872 => 118873)
--- trunk/Source/WebCore/ChangeLog 2012-05-30 01:24:51 UTC (rev 118872)
+++ trunk/Source/WebCore/ChangeLog 2012-05-30 01:33:50 UTC (rev 118873)
@@ -1,3 +1,17 @@
+2012-05-29 Mark Pilgrim <pilg...@chromium.org>
+
+ [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.
+
+ * platform/chromium/FileSystemChromium.cpp:
+ (WebCore::fileExists):
+ * platform/chromium/PlatformSupport.h:
+ (PlatformSupport):
+
2012-05-29 Kentaro Hara <hara...@chromium.org>
[V8] Rename v8Null() to v8NullWithCheck()
Modified: trunk/Source/WebCore/platform/chromium/FileSystemChromium.cpp (118872 => 118873)
--- trunk/Source/WebCore/platform/chromium/FileSystemChromium.cpp 2012-05-30 01:24:51 UTC (rev 118872)
+++ trunk/Source/WebCore/platform/chromium/FileSystemChromium.cpp 2012-05-30 01:33:50 UTC (rev 118873)
@@ -36,6 +36,8 @@
#include "PlatformString.h"
#include "PlatformSupport.h"
+#include <public/Platform.h>
+
namespace WebCore {
bool deleteFile(const String& path)
@@ -88,7 +90,7 @@
bool fileExists(const String& path)
{
- return PlatformSupport::fileExists(path);
+ return WebKit::Platform::current()->fileExists(path);
}
PlatformFileHandle openFile(const String& path, FileOpenMode mode)
Modified: trunk/Source/WebCore/platform/chromium/PlatformSupport.h (118872 => 118873)
--- trunk/Source/WebCore/platform/chromium/PlatformSupport.h 2012-05-30 01:24:51 UTC (rev 118872)
+++ trunk/Source/WebCore/platform/chromium/PlatformSupport.h 2012-05-30 01:33:50 UTC (rev 118873)
@@ -123,7 +123,6 @@
static bool cookiesEnabled(const Document*);
// File ---------------------------------------------------------------
- static bool fileExists(const String&);
static bool deleteFile(const String&);
static bool deleteEmptyDirectory(const String&);
static bool getFileMetadata(const String&, FileMetadata& result);
Modified: trunk/Source/WebKit/chromium/ChangeLog (118872 => 118873)
--- trunk/Source/WebKit/chromium/ChangeLog 2012-05-30 01:24:51 UTC (rev 118872)
+++ trunk/Source/WebKit/chromium/ChangeLog 2012-05-30 01:33:50 UTC (rev 118873)
@@ -1,3 +1,15 @@
+2012-05-29 Mark Pilgrim <pilg...@chromium.org>
+
+ [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.
+
+ * src/PlatformSupport.cpp:
+ (WebCore):
+
2012-05-29 Alec Flett <alecfl...@chromium.org>
IndexedDB: Align codes and names for IDB-specific and DOM-specific errors/exceptions
Modified: trunk/Source/WebKit/chromium/src/PlatformSupport.cpp (118872 => 118873)
--- trunk/Source/WebKit/chromium/src/PlatformSupport.cpp 2012-05-30 01:24:51 UTC (rev 118872)
+++ trunk/Source/WebKit/chromium/src/PlatformSupport.cpp 2012-05-30 01:33:50 UTC (rev 118873)
@@ -302,11 +302,6 @@
// File ------------------------------------------------------------------------
-bool PlatformSupport::fileExists(const String& path)
-{
- return WebKit::Platform::current()->fileUtilities()->fileExists(path);
-}
-
bool PlatformSupport::deleteFile(const String& path)
{
return WebKit::Platform::current()->fileUtilities()->deleteFile(path);
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
