Changeset 117766 in webkit

Timestamp:

May 21, 2012, 4:25:19 AM (14 years ago)

Author:

kinuko@chromium.org

Message:

Cleanup: Remove WebCore::revealFolderInOS() which is no longer used anywhere
​https://bugs.webkit.org/show_bug.cgi?id=86993

Reviewed by Adam Barth.

Source/WebCore:

No new tests as this must be just removing dead code.

• platform/FileSystem.h:
• platform/chromium/FileSystemChromium.cpp:
• platform/chromium/PlatformSupport.h:

(PlatformSupport):

Source/WebKit/chromium:

• public/WebFileUtilities.h:

(WebFileUtilities):

• src/PlatformSupport.cpp:

Location:

trunk/Source

Files:

• Platform/chromium/public/WebFileUtilities.h (modified) (1 diff)
• WebCore/ChangeLog (modified) (1 diff)
• WebCore/platform/FileSystem.h (modified) (1 diff)
• WebCore/platform/chromium/FileSystemChromium.cpp (modified) (1 diff)
• WebCore/platform/chromium/PlatformSupport.h (modified) (1 diff)
• WebKit/chromium/ChangeLog (modified) (1 diff)
• WebKit/chromium/src/PlatformSupport.cpp (modified) (1 diff)

trunk/Source/Platform/chromium/public/WebFileUtilities.h

@@ -49 +49 @@
     typedef int FileHandle;
 #endif
-    virtual void revealFolderInOS(const WebString& path) { }
     virtual bool fileExists(const WebString& path) { return false; }
     virtual bool deleteFile(const WebString& path) { return false; }

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-05-20  Kinuko Yasuda  <kinuko@chromium.org>
+
+        Cleanup: Remove WebCore::revealFolderInOS() which is no longer used anywhere
+        https://bugs.webkit.org/show_bug.cgi?id=86993
+
+        Reviewed by Adam Barth.
+
+        No new tests as this must be just removing dead code.
+
+        * platform/FileSystem.h:
+        * platform/chromium/FileSystemChromium.cpp:
+        * platform/chromium/PlatformSupport.h:
+        (PlatformSupport):
+
 2012-05-21  Alexander Pavlov  <apavlov@chromium.org>
 

trunk/Source/WebCore/platform/FileSystem.h

@@ -152 +152 @@
 #endif
 
-void revealFolderInOS(const String&);
 bool fileExists(const String&);
 bool deleteFile(const String&);

trunk/Source/WebCore/platform/chromium/FileSystemChromium.cpp

@@ -56 +56 @@
 {
     return PlatformSupport::getFileModificationTime(path, result);
-}
-
-void revealFolderInOS(const String& path)
-{
-    PlatformSupport::revealFolderInOS(path);
 }
 

trunk/Source/WebCore/platform/chromium/PlatformSupport.h

@@ -123 +123 @@
 
     // File ---------------------------------------------------------------
-    static void revealFolderInOS(const String&);
     static bool fileExists(const String&);
     static bool deleteFile(const String&);

trunk/Source/WebKit/chromium/ChangeLog

@@ +1 @@
+2012-05-20  Kinuko Yasuda  <kinuko@chromium.org>
+
+        Cleanup: Remove WebCore::revealFolderInOS() which is no longer used anywhere
+        https://bugs.webkit.org/show_bug.cgi?id=86993
+
+        Reviewed by Adam Barth.
+
+        * public/WebFileUtilities.h:
+        (WebFileUtilities):
+        * src/PlatformSupport.cpp:
+
 2012-05-21  Adam Barth  <abarth@webkit.org>
 

trunk/Source/WebKit/chromium/src/PlatformSupport.cpp

@@ -318 +318 @@
 {
     return WebKit::Platform::current()->fileUtilities()->getFileSize(path, result);
-}
-
-void PlatformSupport::revealFolderInOS(const String& path)
-{
-    WebKit::Platform::current()->fileUtilities()->revealFolderInOS(path);
 }