Unreviewed, rolling out r108860.

http://trac.webkit.org/changeset/108860
https://bugs.webkit.org/show_bug.cgi?id=79544

"Breaks Linux compile" (Requested by enne on #webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-02-24

Source/Platform:

* Platform.gypi:
* chromium/public/WebColor.h:
(WebKit):
* chromium/public/WebColorName.h: Renamed from Source/WebKit/chromium/public/WebColorName.h.
(WebKit):

Source/WebKit/chromium:

* WebKit.gyp:
* public/platform/WebColorName.h:
* src/WebColor.cpp: Renamed from Source/WebKit/chromium/src/WebColorName.cpp.
(WebKit):
(WebKit::toCSSValueKeyword):
(WebKit::setNamedColors):

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

diff --git a/Source/Platform/ChangeLog b/Source/Platform/ChangeLog
index 55284819cf15eecfeec483c619a3301cd9387c57..ec652d8c35a9c7768dba0e9fded6f940df635eb4 100644 (file)
--- a/Source/Platform/ChangeLog
+++ b/Source/Platform/ChangeLog
@@ -1,3 +1,17 @@
+2012-02-24  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r108860.
+        http://trac.webkit.org/changeset/108860
+        https://bugs.webkit.org/show_bug.cgi?id=79544
+
+        "Breaks Linux compile" (Requested by enne on #webkit).
+
+        * Platform.gypi:
+        * chromium/public/WebColor.h:
+        (WebKit):
+        * chromium/public/WebColorName.h: Renamed from Source/WebKit/chromium/public/WebColorName.h.
+        (WebKit):
+
 2012-02-24  James Robinson  <jamesr@chromium.org>
 
         [chromium] WebKit::setColorNames is a client API

diff --git a/Source/Platform/Platform.gypi b/Source/Platform/Platform.gypi
index afa14713f5d6aacb8319d75a67736a6f23b1349a..a8385305b04dab283f0ab419ce434f9b9bbf36fa 100644 (file)
--- a/Source/Platform/Platform.gypi
+++ b/Source/Platform/Platform.gypi
@@ -34,6 +34,7 @@
             'chromium/public/WebCString.h',
             'chromium/public/WebCanvas.h',
             'chromium/public/WebColor.h',
+            'chromium/public/WebColorName.h',
             'chromium/public/WebCommon.h',
             'chromium/public/WebContentLayer.h',
             'chromium/public/WebContentLayerClient.h',

diff --git a/Source/Platform/chromium/public/WebColor.h b/Source/Platform/chromium/public/WebColor.h
index d449e36c12d94de0ede751893034882e5173d873..da45eb1c6e495864ec4ab2a97c9aa84419ca96c7 100644 (file)
--- a/Source/Platform/chromium/public/WebColor.h
+++ b/Source/Platform/chromium/public/WebColor.h
@@ -31,10 +31,16 @@
 #ifndef WebColor_h
 #define WebColor_h
 
+#include "WebColorName.h"
+#include "WebCommon.h"
+
 namespace WebKit {
 
 typedef unsigned WebColor;  // RGBA quad. Equivalent to SkColor.
 
+// Sets the values of a set of named colors.
+WEBKIT_EXPORT void setNamedColors(const WebColorName*, const WebColor*, size_t length);
+
 } // namespace WebKit
 
 #endif

diff --git a/Source/WebKit/chromium/public/WebColorName.h b/Source/Platform/chromium/public/WebColorName.h
similarity index 91%
rename from Source/WebKit/chromium/public/WebColorName.h
rename to Source/Platform/chromium/public/WebColorName.h
index 625eb5f607a92cf95e41086554dfd0f1f38e0a73..f97ed265f3be60e38f4634b988805bc7fe3a0375 100644 (file)
--- a/Source/WebKit/chromium/public/WebColorName.h
+++ b/Source/Platform/chromium/public/WebColorName.h
@@ -31,9 +31,6 @@
 #ifndef WebColorName_h
 #define WebColorName_h
 
-#include <public/WebColor.h>
-#include <public/WebCommon.h>
-
 namespace WebKit {
 
 enum WebColorName {
@@ -69,9 +66,6 @@ enum WebColorName {
     WebColorWindowText
 };
 
-// Sets the values of a set of named colors.
-WEBKIT_EXPORT void setNamedColors(const WebColorName*, const WebColor*, size_t length);
-
-} // namespace WebKit
+}  // namespace WebKit
 
 #endif

diff --git a/Source/WebKit/chromium/ChangeLog b/Source/WebKit/chromium/ChangeLog
index 836fdda1a61a30e2785ce174949324ef2cdc2929..6d94e0f5d1e01e975c851b847092e17cf1fcf776 100644 (file)
--- a/Source/WebKit/chromium/ChangeLog
+++ b/Source/WebKit/chromium/ChangeLog
@@ -1,3 +1,18 @@
+2012-02-24  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r108860.
+        http://trac.webkit.org/changeset/108860
+        https://bugs.webkit.org/show_bug.cgi?id=79544
+
+        "Breaks Linux compile" (Requested by enne on #webkit).
+
+        * WebKit.gyp:
+        * public/platform/WebColorName.h:
+        * src/WebColor.cpp: Renamed from Source/WebKit/chromium/src/WebColorName.cpp.
+        (WebKit):
+        (WebKit::toCSSValueKeyword):
+        (WebKit::setNamedColors):
+
 2012-02-24  James Robinson  <jamesr@chromium.org>
 
         [chromium] WebKit::setColorNames is a client API

diff --git a/Source/WebKit/chromium/WebKit.gyp b/Source/WebKit/chromium/WebKit.gyp
index 3da0477a0114de9f8b7604dfd1bb50c8c7c64fa0..f8e58b383b1b3c51475338c1f5f6399903f43c7e 100644 (file)
--- a/Source/WebKit/chromium/WebKit.gyp
+++ b/Source/WebKit/chromium/WebKit.gyp
@@ -112,7 +112,6 @@
                 'public/WebCache.h',
                 'public/WebColorChooser.h',
                 'public/WebColorChooserClient.h',
-                'public/WebColorName.h',
                 'public/WebCommonWorkerClient.h',
                 'public/WebCompositionUnderline.h',
                 'public/WebCompositor.h',
@@ -293,6 +292,7 @@
                 'public/platform/WebCanvas.h',
                 'public/platform/WebClipboard.h',
                 'public/platform/WebColor.h',
+                'public/platform/WebColorName.h',
                 'public/platform/WebCommon.h',
                 'public/platform/WebContentLayer.h',
                 'public/platform/WebContentLayerClient.h',
@@ -494,9 +494,9 @@
                 'src/WebBlob.cpp',
                 'src/WebBlobData.cpp',
                 'src/WebCache.cpp',
+                'src/WebColor.cpp',
                 'src/WebColorChooserClientImpl.cpp',
                 'src/WebColorChooserClientImpl.h',
-                'src/WebColorName.cpp',
                 'src/WebCommon.cpp',
                 'src/WebCompositorImpl.cpp',
                 'src/WebCompositorImpl.h',

diff --git a/Source/WebKit/chromium/public/platform/WebColorName.h b/Source/WebKit/chromium/public/platform/WebColorName.h
index c3a8afeddd96e37a3f1de74e7870b4fb85baba35..17df3f2886c40fa61d5ef0bcf818af74396e7e65 100644 (file)
--- a/Source/WebKit/chromium/public/platform/WebColorName.h
+++ b/Source/WebKit/chromium/public/platform/WebColorName.h
@@ -23,7 +23,4 @@
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-// FIXME: This is logically part of the WebKit client API and doesn't belong in platform. This is a forwarding header to avoid requiring a two-sided commit.
-// As soon as all chromium code is updated for the new location of this header, remove this.
-
-#include "../WebColorName.h"
+#include "../../../../Platform/chromium/public/WebColorName.h"

diff --git a/Source/WebKit/chromium/src/WebColorName.cpp b/Source/WebKit/chromium/src/WebColor.cpp
similarity index 96%
rename from Source/WebKit/chromium/src/WebColorName.cpp
rename to Source/WebKit/chromium/src/WebColor.cpp
index 04d8ed8a83dcaf840dd91e21b5e77f1ee149bed3..737338a8b2e585f7c01bffa59c0e3bd6f61c4ad5 100644 (file)
--- a/Source/WebKit/chromium/src/WebColorName.cpp
+++ b/Source/WebKit/chromium/src/WebColor.cpp
@@ -29,21 +29,21 @@
  */
 
 #include "config.h"
-#include "WebColorName.h"
+#include "platform/WebColor.h"
 
-#include "CSSValueKeywords.h"
 #include "Color.h"
+#include "CSSValueKeywords.h"
 #include "RenderTheme.h"
 #include "UnusedParam.h"
-#include <public/WebColor.h>
+#include "platform/WebColorName.h"
 
 using namespace::WebCore;
 
 namespace WebKit {
 
-static int toCSSValueKeyword(WebColorName name)
+static int toCSSValueKeyword(WebColorName in_value)
 {
-    switch (name) {
+    switch (in_value) {
     case WebColorActiveBorder:
         return CSSValueActiveborder;
     case WebColorActiveCaption: