[WPE] Implement WebsiteDataStore::defaultApplicationCacheDirectory() and friends
authoraperez@igalia.com <aperez@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 8 Aug 2017 16:49:15 +0000 (16:49 +0000)
committeraperez@igalia.com <aperez@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 8 Aug 2017 16:49:15 +0000 (16:49 +0000)
https://bugs.webkit.org/show_bug.cgi?id=175322

Reviewed by Carlos Garcia Campos.

This makes WPE use the same implementation as the GTK+ port, with a preprocessor switch to choose the name of
the base directory inside the user XDG cache directory.

* PlatformGTK.cmake: Add APIWebsiteDataStoreGLib.cpp to the build sources and remove APIWebsiteDataStoreGtk.cpp.
* PlatformWPE.cmake: Add APIWebsiteDataStoreGLib.cpp to the built sources.
* UIProcess/API/APIWebsiteDataStore.cpp: Remove now dead code.
* UIProcess/API/glib/APIWebsiteDataStoreGLib.cpp: Renamed from Source/WebKit/UIProcess/API/gtk/APIWebsiteDataStoreGtk.cpp.
(API::WebsiteDataStore::defaultApplicationCacheDirectory):
(API::WebsiteDataStore::defaultIndexedDBDatabaseDirectory):
(API::WebsiteDataStore::defaultLocalStorageDirectory):
(API::WebsiteDataStore::defaultMediaKeysStorageDirectory):
(API::WebsiteDataStore::defaultWebSQLDatabaseDirectory):
(API::WebsiteDataStore::defaultResourceLoadStatisticsDirectory):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@220408 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit/ChangeLog
Source/WebKit/PlatformGTK.cmake
Source/WebKit/PlatformWPE.cmake
Source/WebKit/UIProcess/API/APIWebsiteDataStore.cpp
Source/WebKit/UIProcess/API/glib/APIWebsiteDataStoreGLib.cpp [moved from Source/WebKit/UIProcess/API/gtk/APIWebsiteDataStoreGtk.cpp with 80% similarity]

index 37ef127..c5e6393 100644 (file)
@@ -1,3 +1,24 @@
+2017-08-08  Adrian Perez de Castro  <aperez@igalia.com>
+
+        [WPE] Implement WebsiteDataStore::defaultApplicationCacheDirectory() and friends
+        https://bugs.webkit.org/show_bug.cgi?id=175322
+
+        Reviewed by Carlos Garcia Campos.
+
+        This makes WPE use the same implementation as the GTK+ port, with a preprocessor switch to choose the name of
+        the base directory inside the user XDG cache directory.
+
+        * PlatformGTK.cmake: Add APIWebsiteDataStoreGLib.cpp to the build sources and remove APIWebsiteDataStoreGtk.cpp.
+        * PlatformWPE.cmake: Add APIWebsiteDataStoreGLib.cpp to the built sources.
+        * UIProcess/API/APIWebsiteDataStore.cpp: Remove now dead code.
+        * UIProcess/API/glib/APIWebsiteDataStoreGLib.cpp: Renamed from Source/WebKit/UIProcess/API/gtk/APIWebsiteDataStoreGtk.cpp.
+        (API::WebsiteDataStore::defaultApplicationCacheDirectory):
+        (API::WebsiteDataStore::defaultIndexedDBDatabaseDirectory):
+        (API::WebsiteDataStore::defaultLocalStorageDirectory):
+        (API::WebsiteDataStore::defaultMediaKeysStorageDirectory):
+        (API::WebsiteDataStore::defaultWebSQLDatabaseDirectory):
+        (API::WebsiteDataStore::defaultResourceLoadStatisticsDirectory):
+
 2017-08-08  Ryan Haddad  <ryanhaddad@apple.com>
 
         Unreviewed, rolling out r220368.
index 3dcbaa1..f68b823 100644 (file)
@@ -116,6 +116,7 @@ list(APPEND WebKit2_SOURCES
     UIProcess/API/C/gtk/WKTextCheckerGtk.cpp
     UIProcess/API/C/gtk/WKView.cpp
 
+    UIProcess/API/glib/APIWebsiteDataStoreGLib.cpp
     UIProcess/API/glib/IconDatabase.cpp
     UIProcess/API/glib/WebKitApplicationInfo.cpp
     UIProcess/API/glib/WebKitAuthenticationRequest.cpp
@@ -172,7 +173,6 @@ list(APPEND WebKit2_SOURCES
     UIProcess/API/glib/WebKitWebsiteDataManager.cpp
     UIProcess/API/glib/WebKitWindowProperties.cpp
 
-    UIProcess/API/gtk/APIWebsiteDataStoreGtk.cpp
     UIProcess/API/gtk/PageClientImpl.cpp
     UIProcess/API/gtk/WebKitAuthenticationDialog.cpp
     UIProcess/API/gtk/WebKitColorChooser.cpp
index 419dceb..199f0c5 100644 (file)
@@ -131,6 +131,7 @@ list(APPEND WebKit2_SOURCES
 
     UIProcess/API/C/wpe/WKView.cpp
 
+    UIProcess/API/glib/APIWebsiteDataStoreGLib.cpp
     UIProcess/API/glib/IconDatabase.cpp
     UIProcess/API/glib/WebKitApplicationInfo.cpp
     UIProcess/API/glib/WebKitAuthenticationRequest.cpp
index 4e8ca1b..0db5dd9 100644 (file)
@@ -87,65 +87,6 @@ void WebsiteDataStore::setResourceLoadStatisticsEnabled(bool enabled)
     m_websiteDataStore->setResourceLoadStatisticsEnabled(enabled);
 }
 
-#if !PLATFORM(COCOA) && !PLATFORM(GTK)
-WebKit::WebsiteDataStore::Configuration WebsiteDataStore::defaultDataStoreConfiguration()
-{
-    // FIXME: Fill everything in.
-    WebKit::WebsiteDataStore::Configuration configuration;
-
-    return configuration;
-}
-
-String WebsiteDataStore::websiteDataDirectoryFileSystemRepresentation(const String&)
-{
-    // FIXME: Implement.
-    return String();
-}
-
-String WebsiteDataStore::defaultLocalStorageDirectory()
-{
-    // FIXME: Implement.
-    return String();
-}
-
-String WebsiteDataStore::defaultWebSQLDatabaseDirectory()
-{
-    // FIXME: Implement.
-    return String();
-}
-
-String WebsiteDataStore::defaultNetworkCacheDirectory()
-{
-    // FIXME: Implement.
-    return String();
-}
-
-String WebsiteDataStore::defaultApplicationCacheDirectory()
-{
-    // FIXME: Implement.
-    return String();
-}
-
-String WebsiteDataStore::defaultMediaKeysStorageDirectory()
-{
-    // FIXME: Implement.
-    return String();
-}
-
-String WebsiteDataStore::defaultIndexedDBDatabaseDirectory()
-{
-    // FIXME: Implement.
-    return String();
-}
-
-String WebsiteDataStore::defaultResourceLoadStatisticsDirectory()
-{
-    // FIXME: Implement.
-    return String();
-}
-
-#endif
-    
 #if !PLATFORM(COCOA)
 String WebsiteDataStore::defaultMediaCacheDirectory()
 {
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2015 Igalia S.L.
+ * Copyright (C) 2015-2017 Igalia S.L.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
 
 #include <WebCore/FileSystem.h>
 
+#if PLATFORM(GTK)
+#define BASE_DIRECTORY "webkitgtk"
+#elif PLATFORM(WPE)
+#define BASE_DIRECTORY "wpe"
+#endif
+
 namespace API {
 
 String WebsiteDataStore::defaultApplicationCacheDirectory()
 {
-    return cacheDirectoryFileSystemRepresentation("webkitgtk" G_DIR_SEPARATOR_S "applications");
+    return cacheDirectoryFileSystemRepresentation(BASE_DIRECTORY G_DIR_SEPARATOR_S "applications");
 }
 
 // FIXME: The other directories in this file are shared between all applications using WebKitGTK+.
@@ -44,27 +50,27 @@ String WebsiteDataStore::defaultNetworkCacheDirectory()
 
 String WebsiteDataStore::defaultIndexedDBDatabaseDirectory()
 {
-    return websiteDataDirectoryFileSystemRepresentation("webkitgtk" G_DIR_SEPARATOR_S "databases" G_DIR_SEPARATOR_S "indexeddb");
+    return websiteDataDirectoryFileSystemRepresentation(BASE_DIRECTORY G_DIR_SEPARATOR_S "databases" G_DIR_SEPARATOR_S "indexeddb");
 }
 
 String WebsiteDataStore::defaultLocalStorageDirectory()
 {
-    return websiteDataDirectoryFileSystemRepresentation("webkitgtk" G_DIR_SEPARATOR_S "localstorage");
+    return websiteDataDirectoryFileSystemRepresentation(BASE_DIRECTORY G_DIR_SEPARATOR_S "localstorage");
 }
 
 String WebsiteDataStore::defaultMediaKeysStorageDirectory()
 {
-    return websiteDataDirectoryFileSystemRepresentation("webkitgtk" G_DIR_SEPARATOR_S "mediakeys");
+    return websiteDataDirectoryFileSystemRepresentation(BASE_DIRECTORY G_DIR_SEPARATOR_S "mediakeys");
 }
 
 String WebsiteDataStore::defaultWebSQLDatabaseDirectory()
 {
-    return websiteDataDirectoryFileSystemRepresentation("webkitgtk" G_DIR_SEPARATOR_S "databases");
+    return websiteDataDirectoryFileSystemRepresentation(BASE_DIRECTORY G_DIR_SEPARATOR_S "databases");
 }
 
 String WebsiteDataStore::defaultResourceLoadStatisticsDirectory()
 {
-    return websiteDataDirectoryFileSystemRepresentation("webkitgtk" G_DIR_SEPARATOR_S "ResourceLoadStatistics");
+    return websiteDataDirectoryFileSystemRepresentation(BASE_DIRECTORY G_DIR_SEPARATOR_S "ResourceLoadStatistics");
 }
 
 String WebsiteDataStore::cacheDirectoryFileSystemRepresentation(const String& directoryName)