Title: [117766] trunk/Source
- Revision
- 117766
- Author
- [email protected]
- Date
- 2012-05-21 04:25:19 -0700 (Mon, 21 May 2012)
Log 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:
Modified Paths
Diff
Modified: trunk/Source/Platform/chromium/public/WebFileUtilities.h (117765 => 117766)
--- trunk/Source/Platform/chromium/public/WebFileUtilities.h 2012-05-21 11:25:10 UTC (rev 117765)
+++ trunk/Source/Platform/chromium/public/WebFileUtilities.h 2012-05-21 11:25:19 UTC (rev 117766)
@@ -48,7 +48,6 @@
#else
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; }
virtual bool deleteEmptyDirectory(const WebString& path) { return false; }
Modified: trunk/Source/WebCore/ChangeLog (117765 => 117766)
--- trunk/Source/WebCore/ChangeLog 2012-05-21 11:25:10 UTC (rev 117765)
+++ trunk/Source/WebCore/ChangeLog 2012-05-21 11:25:19 UTC (rev 117766)
@@ -1,3 +1,17 @@
+2012-05-20 Kinuko Yasuda <[email protected]>
+
+ 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 <[email protected]>
Web Inspector: Annotate CSSStyleModel.js for the closure compiler
Modified: trunk/Source/WebCore/platform/FileSystem.h (117765 => 117766)
--- trunk/Source/WebCore/platform/FileSystem.h 2012-05-21 11:25:10 UTC (rev 117765)
+++ trunk/Source/WebCore/platform/FileSystem.h 2012-05-21 11:25:19 UTC (rev 117766)
@@ -151,7 +151,6 @@
static const char PlatformFilePathSeparator = '/';
#endif
-void revealFolderInOS(const String&);
bool fileExists(const String&);
bool deleteFile(const String&);
bool deleteEmptyDirectory(const String&);
Modified: trunk/Source/WebCore/platform/chromium/FileSystemChromium.cpp (117765 => 117766)
--- trunk/Source/WebCore/platform/chromium/FileSystemChromium.cpp 2012-05-21 11:25:10 UTC (rev 117765)
+++ trunk/Source/WebCore/platform/chromium/FileSystemChromium.cpp 2012-05-21 11:25:19 UTC (rev 117766)
@@ -57,11 +57,6 @@
return PlatformSupport::getFileModificationTime(path, result);
}
-void revealFolderInOS(const String& path)
-{
- PlatformSupport::revealFolderInOS(path);
-}
-
String directoryName(const String& path)
{
return PlatformSupport::directoryName(path);
Modified: trunk/Source/WebCore/platform/chromium/PlatformSupport.h (117765 => 117766)
--- trunk/Source/WebCore/platform/chromium/PlatformSupport.h 2012-05-21 11:25:10 UTC (rev 117765)
+++ trunk/Source/WebCore/platform/chromium/PlatformSupport.h 2012-05-21 11:25:19 UTC (rev 117766)
@@ -122,7 +122,6 @@
static bool cookiesEnabled(const Document*);
// File ---------------------------------------------------------------
- static void revealFolderInOS(const String&);
static bool fileExists(const String&);
static bool deleteFile(const String&);
static bool deleteEmptyDirectory(const String&);
Modified: trunk/Source/WebKit/chromium/ChangeLog (117765 => 117766)
--- trunk/Source/WebKit/chromium/ChangeLog 2012-05-21 11:25:10 UTC (rev 117765)
+++ trunk/Source/WebKit/chromium/ChangeLog 2012-05-21 11:25:19 UTC (rev 117766)
@@ -1,3 +1,14 @@
+2012-05-20 Kinuko Yasuda <[email protected]>
+
+ 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 <[email protected]>
[Chromium] Implement WebViewImpl::textInputInfo() for Android
Modified: trunk/Source/WebKit/chromium/src/PlatformSupport.cpp (117765 => 117766)
--- trunk/Source/WebKit/chromium/src/PlatformSupport.cpp 2012-05-21 11:25:10 UTC (rev 117765)
+++ trunk/Source/WebKit/chromium/src/PlatformSupport.cpp 2012-05-21 11:25:19 UTC (rev 117766)
@@ -319,11 +319,6 @@
return WebKit::Platform::current()->fileUtilities()->getFileSize(path, result);
}
-void PlatformSupport::revealFolderInOS(const String& path)
-{
- WebKit::Platform::current()->fileUtilities()->revealFolderInOS(path);
-}
-
bool PlatformSupport::getFileModificationTime(const String& path, time_t& result)
{
double modificationTime;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
