Title: [120603] trunk
Revision
120603
Author
wangxian...@chromium.org
Date
2012-06-18 09:55:20 -0700 (Mon, 18 Jun 2012)

Log Message

[Chromium] Change back "linuxish" to "linux" and include WebFontRendering.cpp on Android
https://bugs.webkit.org/show_bug.cgi?id=89228

Reviewed by Tony Chang.

Source/WebKit/chromium:

* WebKit.gyp:
* public/linux/WebFontRendering.h:
(WebKit):
(WebFontRendering):
* public/linuxish/WebFontRendering.h: Removed.
* src/linux/WebFontRendering.cpp: Renamed from Source/WebKit/chromium/src/linuxish/WebFontRendering.cpp.
(WebKit):
(WebKit::WebFontRendering::setHinting):
(WebKit::WebFontRendering::setAutoHint):
(WebKit::WebFontRendering::setUseBitmaps):
(WebKit::WebFontRendering::setAntiAlias):
(WebKit::WebFontRendering::setSubpixelRendering):
(WebKit::WebFontRendering::setSubpixelPositioning):
(WebKit::WebFontRendering::setLCDOrder):
(WebKit::WebFontRendering::setLCDOrientation):

Tools:

* DumpRenderTree/chromium/LayoutTestController.cpp:
* DumpRenderTree/chromium/TestShellAndroid.cpp:

Modified Paths

Added Paths

Removed Paths

Diff

Modified: trunk/Source/WebKit/chromium/ChangeLog (120602 => 120603)


--- trunk/Source/WebKit/chromium/ChangeLog	2012-06-18 16:40:07 UTC (rev 120602)
+++ trunk/Source/WebKit/chromium/ChangeLog	2012-06-18 16:55:20 UTC (rev 120603)
@@ -1,3 +1,26 @@
+2012-06-18  Xianzhu Wang  <wangxian...@chromium.org>
+
+        [Chromium] Change back "linuxish" to "linux" and include WebFontRendering.cpp on Android
+        https://bugs.webkit.org/show_bug.cgi?id=89228
+
+        Reviewed by Tony Chang.
+
+        * WebKit.gyp:
+        * public/linux/WebFontRendering.h:
+        (WebKit):
+        (WebFontRendering):
+        * public/linuxish/WebFontRendering.h: Removed.
+        * src/linux/WebFontRendering.cpp: Renamed from Source/WebKit/chromium/src/linuxish/WebFontRendering.cpp.
+        (WebKit):
+        (WebKit::WebFontRendering::setHinting):
+        (WebKit::WebFontRendering::setAutoHint):
+        (WebKit::WebFontRendering::setUseBitmaps):
+        (WebKit::WebFontRendering::setAntiAlias):
+        (WebKit::WebFontRendering::setSubpixelRendering):
+        (WebKit::WebFontRendering::setSubpixelPositioning):
+        (WebKit::WebFontRendering::setLCDOrder):
+        (WebKit::WebFontRendering::setLCDOrientation):
+
 2012-06-18  Hironori Bono  <hb...@chromium.org>
 
         [chromium] Spellchecker should show suggestions only when right-clicking a misspelled word.

Modified: trunk/Source/WebKit/chromium/WebKit.gyp (120602 => 120603)


--- trunk/Source/WebKit/chromium/WebKit.gyp	2012-06-18 16:40:07 UTC (rev 120602)
+++ trunk/Source/WebKit/chromium/WebKit.gyp	2012-06-18 16:55:20 UTC (rev 120603)
@@ -284,8 +284,8 @@
                 'public/android/WebSandboxSupport.h',
                 'public/gtk/WebInputEventFactory.h',
                 'public/linux/WebFontRenderStyle.h',
+                'public/linux/WebFontRendering.h',
                 'public/linux/WebRenderTheme.h',
-                'public/linuxish/WebFontRendering.h',
                 'public/mac/WebInputEventFactory.h',
                 'public/mac/WebSandboxSupport.h',
                 'public/mac/WebScreenInfoFactory.h',
@@ -428,9 +428,9 @@
                 'src/PrerendererClientImpl.cpp',
                 'src/android/WebInputEventFactory.cpp',
                 'src/linux/WebFontInfo.cpp',
+                'src/linux/WebFontRendering.cpp',
                 'src/linux/WebFontRenderStyle.cpp',
                 'src/linux/WebRenderTheme.cpp',
-                'src/linuxish/WebFontRendering.cpp',
                 'src/x11/WebScreenInfoFactory.cpp',
                 'src/mac/WebInputEventFactory.mm',
                 'src/mac/WebScreenInfoFactory.mm',
@@ -776,7 +776,6 @@
                     'include_dirs': [
                         'public/x11',
                         'public/linux',
-                        'public/linuxish',
                     ],
                 }, { # else: use_x11 != 1
                     'sources/': [
@@ -799,20 +798,13 @@
                 ['OS=="android"', {
                     'include_dirs': [
                         'public/android',
-                        'public/linuxish',
+                        'public/linux', # We need linux/WebFontRendering.h on Android.
                     ],
                 }, { # else: OS!="android"
                     'sources/': [
                         ['exclude', '/android/'],
                     ],
                 }],
-                # FIXME: Here and other places duplicate rules in Chromium's build/filename_rules.gypi.
-                # However without the duplication, gyp_webkit fails at least on chromium-win-release.
-                ['OS!="android" and OS!="linux" and OS!="openbsd" and OS!="freebsd"', {
-                    'sources/': [
-                        ['exclude', '/linuxish/'],
-                    ],
-                }],
                 # TODO: we exclude CG.cpp on both sides of the below conditional. Move elsewhere?
                 ['OS=="mac"', {
                     'include_dirs': [
@@ -857,6 +849,13 @@
                     },
                 }],
             ],
+            'target_conditions': [
+                ['OS=="android"', {
+                    'sources/': [
+                        ['include', '^src/linux/WebFontRendering\\.cpp$'],
+                    ],
+                }],
+            ],
         },
         {
             'target_name': 'inspector_resources',

Modified: trunk/Source/WebKit/chromium/public/linux/WebFontRendering.h (120602 => 120603)


--- trunk/Source/WebKit/chromium/public/linux/WebFontRendering.h	2012-06-18 16:40:07 UTC (rev 120602)
+++ trunk/Source/WebKit/chromium/public/linux/WebFontRendering.h	2012-06-18 16:55:20 UTC (rev 120603)
@@ -27,5 +27,30 @@
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
-// FIXME: Remove once http://codereview.chromium.org/10544103/ is landed.
-#include "../linuxish/WebFontRendering.h"
+
+#ifndef WebFontRendering_h
+#define WebFontRendering_h
+
+#include "../platform/WebCommon.h"
+#include <SkFontHost.h>
+#include <SkPaint.h>
+
+namespace WebKit {
+
+class WebFontRendering {
+public:
+    // Set global font renderering preferences.
+
+    WEBKIT_EXPORT static void setHinting(SkPaint::Hinting);
+    WEBKIT_EXPORT static void setAutoHint(bool);
+    WEBKIT_EXPORT static void setUseBitmaps(bool);
+    WEBKIT_EXPORT static void setAntiAlias(bool);
+    WEBKIT_EXPORT static void setSubpixelRendering(bool);
+    WEBKIT_EXPORT static void setSubpixelPositioning(bool);
+    WEBKIT_EXPORT static void setLCDOrder(SkFontHost::LCDOrder);
+    WEBKIT_EXPORT static void setLCDOrientation(SkFontHost::LCDOrientation);
+};
+
+} // namespace WebKit
+
+#endif

Deleted: trunk/Source/WebKit/chromium/public/linuxish/WebFontRendering.h (120602 => 120603)


--- trunk/Source/WebKit/chromium/public/linuxish/WebFontRendering.h	2012-06-18 16:40:07 UTC (rev 120602)
+++ trunk/Source/WebKit/chromium/public/linuxish/WebFontRendering.h	2012-06-18 16:55:20 UTC (rev 120603)
@@ -1,56 +0,0 @@
-/*
- * Copyright (C) 2009 Google Inc. All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions are
- * met:
- *
- *     * Redistributions of source code must retain the above copyright
- * notice, this list of conditions and the following disclaimer.
- *     * Redistributions in binary form must reproduce the above
- * copyright notice, this list of conditions and the following disclaimer
- * in the documentation and/or other materials provided with the
- * distribution.
- *     * Neither the name of Google Inc. nor the names of its
- * contributors may be used to endorse or promote products derived from
- * this software without specific prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
- * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
- * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
- * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
- * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
- * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
- * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
- * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#ifndef WebFontRendering_h
-#define WebFontRendering_h
-
-#include "../platform/WebCommon.h"
-#include <SkFontHost.h>
-#include <SkPaint.h>
-
-namespace WebKit {
-
-class WebFontRendering {
-public:
-    // Set global font renderering preferences.
-
-    WEBKIT_EXPORT static void setHinting(SkPaint::Hinting);
-    WEBKIT_EXPORT static void setAutoHint(bool);
-    WEBKIT_EXPORT static void setUseBitmaps(bool);
-    WEBKIT_EXPORT static void setAntiAlias(bool);
-    WEBKIT_EXPORT static void setSubpixelRendering(bool);
-    WEBKIT_EXPORT static void setSubpixelPositioning(bool);
-    WEBKIT_EXPORT static void setLCDOrder(SkFontHost::LCDOrder);
-    WEBKIT_EXPORT static void setLCDOrientation(SkFontHost::LCDOrientation);
-};
-
-} // namespace WebKit
-
-#endif

Copied: trunk/Source/WebKit/chromium/src/linux/WebFontRendering.cpp (from rev 120602, trunk/Source/WebKit/chromium/src/linuxish/WebFontRendering.cpp) (0 => 120603)


--- trunk/Source/WebKit/chromium/src/linux/WebFontRendering.cpp	                        (rev 0)
+++ trunk/Source/WebKit/chromium/src/linux/WebFontRendering.cpp	2012-06-18 16:55:20 UTC (rev 120603)
@@ -0,0 +1,95 @@
+/*
+ * Copyright (C) 2009 Google Inc. All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are
+ * met:
+ *
+ *     * Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ *     * Redistributions in binary form must reproduce the above
+ * copyright notice, this list of conditions and the following disclaimer
+ * in the documentation and/or other materials provided with the
+ * distribution.
+ *     * Neither the name of Google Inc. nor the names of its
+ * contributors may be used to endorse or promote products derived from
+ * this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#include "config.h"
+#include "WebFontRendering.h"
+
+#include "FontPlatformData.h"
+
+#if OS(LINUX)
+#include "WebFontInfo.h"
+#endif
+
+using WebCore::FontPlatformData;
+
+namespace WebKit {
+
+// static
+void WebFontRendering::setHinting(SkPaint::Hinting hinting)
+{
+    FontPlatformData::setHinting(hinting);
+}
+
+// static
+void WebFontRendering::setAutoHint(bool useAutoHint)
+{
+    FontPlatformData::setAutoHint(useAutoHint);
+}
+
+// static
+void WebFontRendering::setUseBitmaps(bool useBitmaps)
+{
+    FontPlatformData::setUseBitmaps(useBitmaps);
+}
+
+// static
+void WebFontRendering::setAntiAlias(bool useAntiAlias)
+{
+    FontPlatformData::setAntiAlias(useAntiAlias);
+}
+
+// static
+void WebFontRendering::setSubpixelRendering(bool useSubpixelRendering)
+{
+    FontPlatformData::setSubpixelRendering(useSubpixelRendering);
+}
+
+// static
+void WebFontRendering::setSubpixelPositioning(bool useSubpixelPositioning)
+{
+    FontPlatformData::setSubpixelPositioning(useSubpixelPositioning);
+#if OS(LINUX)
+    WebFontInfo::setSubpixelPositioning(useSubpixelPositioning);
+#endif
+}
+
+// static
+void WebFontRendering::setLCDOrder(SkFontHost::LCDOrder order)
+{
+    SkFontHost::SetSubpixelOrder(order);
+}
+
+// static
+void WebFontRendering::setLCDOrientation(SkFontHost::LCDOrientation orientation)
+{
+    SkFontHost::SetSubpixelOrientation(orientation);
+}
+
+} // namespace WebKit

Deleted: trunk/Source/WebKit/chromium/src/linuxish/WebFontRendering.cpp (120602 => 120603)


--- trunk/Source/WebKit/chromium/src/linuxish/WebFontRendering.cpp	2012-06-18 16:40:07 UTC (rev 120602)
+++ trunk/Source/WebKit/chromium/src/linuxish/WebFontRendering.cpp	2012-06-18 16:55:20 UTC (rev 120603)
@@ -1,95 +0,0 @@
-/*
- * Copyright (C) 2009 Google Inc. All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions are
- * met:
- *
- *     * Redistributions of source code must retain the above copyright
- * notice, this list of conditions and the following disclaimer.
- *     * Redistributions in binary form must reproduce the above
- * copyright notice, this list of conditions and the following disclaimer
- * in the documentation and/or other materials provided with the
- * distribution.
- *     * Neither the name of Google Inc. nor the names of its
- * contributors may be used to endorse or promote products derived from
- * this software without specific prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
- * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
- * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
- * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
- * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
- * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
- * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
- * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#include "config.h"
-#include "WebFontRendering.h"
-
-#include "FontPlatformData.h"
-
-#if OS(LINUX)
-#include "WebFontInfo.h"
-#endif
-
-using WebCore::FontPlatformData;
-
-namespace WebKit {
-
-// static
-void WebFontRendering::setHinting(SkPaint::Hinting hinting)
-{
-    FontPlatformData::setHinting(hinting);
-}
-
-// static
-void WebFontRendering::setAutoHint(bool useAutoHint)
-{
-    FontPlatformData::setAutoHint(useAutoHint);
-}
-
-// static
-void WebFontRendering::setUseBitmaps(bool useBitmaps)
-{
-    FontPlatformData::setUseBitmaps(useBitmaps);
-}
-
-// static
-void WebFontRendering::setAntiAlias(bool useAntiAlias)
-{
-    FontPlatformData::setAntiAlias(useAntiAlias);
-}
-
-// static
-void WebFontRendering::setSubpixelRendering(bool useSubpixelRendering)
-{
-    FontPlatformData::setSubpixelRendering(useSubpixelRendering);
-}
-
-// static
-void WebFontRendering::setSubpixelPositioning(bool useSubpixelPositioning)
-{
-    FontPlatformData::setSubpixelPositioning(useSubpixelPositioning);
-#if OS(LINUX)
-    WebFontInfo::setSubpixelPositioning(useSubpixelPositioning);
-#endif
-}
-
-// static
-void WebFontRendering::setLCDOrder(SkFontHost::LCDOrder order)
-{
-    SkFontHost::SetSubpixelOrder(order);
-}
-
-// static
-void WebFontRendering::setLCDOrientation(SkFontHost::LCDOrientation orientation)
-{
-    SkFontHost::SetSubpixelOrientation(orientation);
-}
-
-} // namespace WebKit

Modified: trunk/Tools/ChangeLog (120602 => 120603)


--- trunk/Tools/ChangeLog	2012-06-18 16:40:07 UTC (rev 120602)
+++ trunk/Tools/ChangeLog	2012-06-18 16:55:20 UTC (rev 120603)
@@ -1,3 +1,13 @@
+2012-06-18  Xianzhu Wang  <wangxian...@chromium.org>
+
+        [Chromium] Change back "linuxish" to "linux" and include WebFontRendering.cpp on Android
+        https://bugs.webkit.org/show_bug.cgi?id=89228
+
+        Reviewed by Tony Chang.
+
+        * DumpRenderTree/chromium/LayoutTestController.cpp:
+        * DumpRenderTree/chromium/TestShellAndroid.cpp:
+
 2012-06-18  Mario Sanchez Prada  <msanc...@igalia.com>
 
         [GTK] Get rid of DumpRenderTreeSupportGtk::{in|de}crementAccessibilityValue

Modified: trunk/Tools/DumpRenderTree/chromium/LayoutTestController.cpp (120602 => 120603)


--- trunk/Tools/DumpRenderTree/chromium/LayoutTestController.cpp	2012-06-18 16:40:07 UTC (rev 120602)
+++ trunk/Tools/DumpRenderTree/chromium/LayoutTestController.cpp	2012-06-18 16:55:20 UTC (rev 120603)
@@ -79,7 +79,7 @@
 #endif
 
 #if OS(LINUX) || OS(ANDROID)
-#include "linuxish/WebFontRendering.h"
+#include "linux/WebFontRendering.h"
 #endif
 
 using namespace WebCore;

Modified: trunk/Tools/DumpRenderTree/chromium/TestShellAndroid.cpp (120602 => 120603)


--- trunk/Tools/DumpRenderTree/chromium/TestShellAndroid.cpp	2012-06-18 16:40:07 UTC (rev 120602)
+++ trunk/Tools/DumpRenderTree/chromium/TestShellAndroid.cpp	2012-06-18 16:55:20 UTC (rev 120603)
@@ -31,7 +31,7 @@
 #include "config.h"
 #include "TestShell.h"
 
-#include "linuxish/WebFontRendering.h"
+#include "linux/WebFontRendering.h"
 #include <android/log.h>
 #include <errno.h>
 #include <fcntl.h>
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to