Diff
Modified: trunk/ChangeLog (183985 => 183986)
--- trunk/ChangeLog 2015-05-08 06:51:55 UTC (rev 183985)
+++ trunk/ChangeLog 2015-05-08 07:56:51 UTC (rev 183986)
@@ -1,3 +1,16 @@
+2015-05-08 Commit Queue <[email protected]>
+
+ Unreviewed, rolling out r183945.
+ https://bugs.webkit.org/show_bug.cgi?id=144789
+
+ "It broke all the GTK+ tests" (Requested by KaL on #webkit).
+
+ Reverted changeset:
+
+ "[GTK] Checks for DEVELOPMENT_BUILD are all wrong"
+ https://bugs.webkit.org/show_bug.cgi?id=144746
+ http://trac.webkit.org/changeset/183945
+
2015-05-07 Myles C. Maxfield <[email protected]>
Cleanup after r183940
Modified: trunk/Source/WebCore/ChangeLog (183985 => 183986)
--- trunk/Source/WebCore/ChangeLog 2015-05-08 06:51:55 UTC (rev 183985)
+++ trunk/Source/WebCore/ChangeLog 2015-05-08 07:56:51 UTC (rev 183986)
@@ -1,3 +1,16 @@
+2015-05-08 Commit Queue <[email protected]>
+
+ Unreviewed, rolling out r183945.
+ https://bugs.webkit.org/show_bug.cgi?id=144789
+
+ "It broke all the GTK+ tests" (Requested by KaL on #webkit).
+
+ Reverted changeset:
+
+ "[GTK] Checks for DEVELOPMENT_BUILD are all wrong"
+ https://bugs.webkit.org/show_bug.cgi?id=144746
+ http://trac.webkit.org/changeset/183945
+
2015-05-07 Chris Dumez <[email protected]>
Throttle RequestAnimationFrame in subframes that are outside the viewport
Modified: trunk/Source/WebCore/platform/gtk/GtkUtilities.cpp (183985 => 183986)
--- trunk/Source/WebCore/platform/gtk/GtkUtilities.cpp 2015-05-08 06:51:55 UTC (rev 183985)
+++ trunk/Source/WebCore/platform/gtk/GtkUtilities.cpp 2015-05-08 07:56:51 UTC (rev 183986)
@@ -54,7 +54,7 @@
return gtk_widget_is_toplevel(widget) && GTK_IS_WINDOW(widget) && !GTK_IS_OFFSCREEN_WINDOW(widget);
}
-#if defined(DEVELOPER_MODE)
+#if defined(DEVELOPMENT_BUILD)
static CString topLevelPath()
{
if (const char* topLevelDirectory = g_getenv("WEBKIT_TOP_LEVEL"))
Modified: trunk/Source/WebCore/platform/gtk/GtkUtilities.h (183985 => 183986)
--- trunk/Source/WebCore/platform/gtk/GtkUtilities.h 2015-05-08 06:51:55 UTC (rev 183985)
+++ trunk/Source/WebCore/platform/gtk/GtkUtilities.h 2015-05-08 07:56:51 UTC (rev 183986)
@@ -28,7 +28,7 @@
IntPoint convertWidgetPointToScreenPoint(GtkWidget*, const IntPoint&);
bool widgetIsOnscreenToplevelWindow(GtkWidget*);
-#if defined(DEVELOPER_MODE)
+#if defined(DEVELOPMENT_BUILD)
CString webkitBuildDirectory();
#endif
Modified: trunk/Source/WebCore/platform/text/gtk/HyphenationLibHyphen.cpp (183985 => 183986)
--- trunk/Source/WebCore/platform/text/gtk/HyphenationLibHyphen.cpp 2015-05-08 06:51:55 UTC (rev 183985)
+++ trunk/Source/WebCore/platform/text/gtk/HyphenationLibHyphen.cpp 2015-05-08 07:56:51 UTC (rev 183986)
@@ -63,7 +63,7 @@
availableLocales.set(AtomicString(extractLocaleFromDictionaryFilePath(filePath)), filePath);
}
-#if defined(DEVELOPER_MODE)
+#if defined(DEVELOPMENT_BUILD)
static void scanTestDictionariesDirectoryIfNecessary(HashMap<AtomicString, String>& availableLocales)
{
// It's unfortunate that we need to look for the dictionaries this way, but
@@ -91,7 +91,7 @@
for (size_t i = 0; i < WTF_ARRAY_LENGTH(gDictionaryDirectories); i++)
scanDirectoryForDicionaries(gDictionaryDirectories[i], availableLocales);
-#if defined(DEVELOPER_MODE)
+#if defined(DEVELOPMENT_BUILD)
scanTestDictionariesDirectoryIfNecessary(availableLocales);
#endif
Modified: trunk/Source/WebKit2/ChangeLog (183985 => 183986)
--- trunk/Source/WebKit2/ChangeLog 2015-05-08 06:51:55 UTC (rev 183985)
+++ trunk/Source/WebKit2/ChangeLog 2015-05-08 07:56:51 UTC (rev 183986)
@@ -1,3 +1,16 @@
+2015-05-08 Commit Queue <[email protected]>
+
+ Unreviewed, rolling out r183945.
+ https://bugs.webkit.org/show_bug.cgi?id=144789
+
+ "It broke all the GTK+ tests" (Requested by KaL on #webkit).
+
+ Reverted changeset:
+
+ "[GTK] Checks for DEVELOPMENT_BUILD are all wrong"
+ https://bugs.webkit.org/show_bug.cgi?id=144746
+ http://trac.webkit.org/changeset/183945
+
2015-05-07 Conrad Shultz <[email protected]>
Don't register for Lookup notifications until needed
Modified: trunk/Source/WebKit2/Shared/gtk/ProcessExecutablePathGtk.cpp (183985 => 183986)
--- trunk/Source/WebKit2/Shared/gtk/ProcessExecutablePathGtk.cpp 2015-05-08 06:51:55 UTC (rev 183985)
+++ trunk/Source/WebKit2/Shared/gtk/ProcessExecutablePathGtk.cpp 2015-05-08 07:56:51 UTC (rev 183986)
@@ -30,7 +30,7 @@
#include <WebCore/FileSystem.h>
#include <glib.h>
-#if defined(DEVELOPER_MODE)
+#if defined(DEVELOPMENT_BUILD)
#include <wtf/gobject/GlibUtilities.h>
#endif
@@ -38,7 +38,7 @@
namespace WebKit {
-#if defined(DEVELOPER_MODE)
+#if defined(DEVELOPMENT_BUILD)
static String getExecutablePath()
{
CString executablePath = getCurrentExecutablePath();
@@ -50,7 +50,7 @@
static String findWebKitProcess(const char* processName)
{
-#if defined(DEVELOPER_MODE)
+#if defined(DEVELOPMENT_BUILD)
static const char* execDirectory = g_getenv("WEBKIT_EXEC_PATH");
if (execDirectory) {
String processPath = pathByAppendingComponent(filenameToString(execDirectory), processName);
Modified: trunk/Source/WebKit2/UIProcess/API/gtk/WebKitWebContext.cpp (183985 => 183986)
--- trunk/Source/WebKit2/UIProcess/API/gtk/WebKitWebContext.cpp 2015-05-08 06:51:55 UTC (rev 183985)
+++ trunk/Source/WebKit2/UIProcess/API/gtk/WebKitWebContext.cpp 2015-05-08 07:56:51 UTC (rev 183986)
@@ -216,7 +216,7 @@
static const char* injectedBundleDirectory()
{
-#if defined(DEVELOPER_MODE)
+#if defined(DEVELOPMENT_BUILD)
const char* bundleDirectory = g_getenv("WEBKIT_INJECTED_BUNDLE_PATH");
if (bundleDirectory && g_file_test(bundleDirectory, G_FILE_TEST_IS_DIR))
return bundleDirectory;
Modified: trunk/Source/WebKit2/UIProcess/gtk/TextCheckerGtk.cpp (183985 => 183986)
--- trunk/Source/WebKit2/UIProcess/gtk/TextCheckerGtk.cpp 2015-05-08 06:51:55 UTC (rev 183985)
+++ trunk/Source/WebKit2/UIProcess/gtk/TextCheckerGtk.cpp 2015-05-08 07:56:51 UTC (rev 183986)
@@ -44,7 +44,7 @@
{
static NeverDestroyed<WebCore::TextCheckerEnchant> checker;
-#if defined(DEVELOPER_MODE)
+#if defined(DEVELOPMENT_BUILD)
// This is a bit of a hack, but ensures that for testing purposes,
// spell checking is properly initialized in WebKitTestRunner while
// running layout tests. We should consider replacing this with some
Modified: trunk/Source/cmake/OptionsGTK.cmake (183985 => 183986)
--- trunk/Source/cmake/OptionsGTK.cmake 2015-05-08 06:51:55 UTC (rev 183985)
+++ trunk/Source/cmake/OptionsGTK.cmake 2015-05-08 07:56:51 UTC (rev 183986)
@@ -76,6 +76,8 @@
WEBKIT_OPTION_DEPEND(USE_GSTREAMER_GL ENABLE_VIDEO)
WEBKIT_OPTION_DEPEND(USE_GSTREAMER_MPEGTS ENABLE_VIDEO)
+# FIXME: There is no reason these should be different.
+SET_AND_EXPOSE_TO_BUILD(DEVELOPMENT_BUILD ${DEVELOPER_MODE})
if (DEVELOPER_MODE)
WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_MINIBROWSER PUBLIC ON)
WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_API_TESTS PRIVATE ON)
Modified: trunk/Tools/ChangeLog (183985 => 183986)
--- trunk/Tools/ChangeLog 2015-05-08 06:51:55 UTC (rev 183985)
+++ trunk/Tools/ChangeLog 2015-05-08 07:56:51 UTC (rev 183986)
@@ -1,3 +1,16 @@
+2015-05-08 Commit Queue <[email protected]>
+
+ Unreviewed, rolling out r183945.
+ https://bugs.webkit.org/show_bug.cgi?id=144789
+
+ "It broke all the GTK+ tests" (Requested by KaL on #webkit).
+
+ Reverted changeset:
+
+ "[GTK] Checks for DEVELOPMENT_BUILD are all wrong"
+ https://bugs.webkit.org/show_bug.cgi?id=144746
+ http://trac.webkit.org/changeset/183945
+
2015-05-07 Anders Carlsson <[email protected]>
Build fixes.
Modified: trunk/Tools/MiniBrowser/gtk/main.c (183985 => 183986)
--- trunk/Tools/MiniBrowser/gtk/main.c 2015-05-08 06:51:55 UTC (rev 183985)
+++ trunk/Tools/MiniBrowser/gtk/main.c 2015-05-08 07:56:51 UTC (rev 183986)
@@ -256,7 +256,7 @@
int main(int argc, char *argv[])
{
gtk_init(&argc, &argv);
-#if defined(DEVELOPER_MODE)
+#if defined(DEVELOPMENT_BUILD)
g_setenv("WEBKIT_INJECTED_BUNDLE_PATH", WEBKIT_INJECTED_BUNDLE_PATH, FALSE);
#endif
