[Webkit-unassigned] [Bug 88802] New: [Chromium] Let Android and Linux share some platform-specific APIs

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=88802

           Summary: [Chromium] Let Android and Linux share some
                    platform-specific APIs
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebKit API
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: wangxianzhu at chromium.org
                CC: fishd at chromium.org, abarth at webkit.org,
                    klobag at chromium.org
            Blocks: 66687


Some Linux-specific APIs under the following directories:

Source/WebKit/chromium/public/linux
Source/WebKit/chromium/public/platform/linux
Source/Platform/chromium/public/linux

can be shared with Android.

At least we need WebFontRendering to set some font rendering configurations from chromium on Android.
Other interfaces like WebFontRenderStyle, WebSandboxSupport, WebThemeEngine, WebFontInfo, etc. can also be shared.

We have some Android-specific changes in WebKit/WebCore for font configurations, but with the above APIs, we can move those changes into Chromium.

For example, in FontPlatformDataHarfBuzz.cpp:

#if OS(ANDROID)
// Slight hinting renders much better than normal hinting on Android.
static SkPaint::Hinting skiaHinting = SkPaint::kSlight_Hinting;
#else
static SkPaint::Hinting skiaHinting = SkPaint::kNormal_Hinting;
#endif

the change can be removed if we can use Source/WebKit/chromium/public/linux/WebFontRendering.h and Source/WebKit/chromium/src/linux/WebFontRendering.cpp on Android.

*/linux/* files are excluded on Android. Adding them back in gyp needs a tricky 'target_conditions' section.

According to the rules defined in Chromium's build/filename_rules.gypi, we should name the directories 'linuxish' if shared between Android and Linux.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.