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.
Created attachment 146930 [details] patch Wondering if this is the correct way.
Comment on attachment 146930 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=146930&action=review Wondering if the commit queue could correctly handle the change: A renamed to B another A created > Source/WebKit/chromium/WebKit.gyp:-781 > - ['exclude', '/linux/'], The rules have been in Chromium's build/filename_rules.gypi.
Created attachment 146958 [details] patch v2 (try to fix patch issue)
Created attachment 146964 [details] patch v3 (remove changes after move which break patch on bot)
Please wait for approval from abarth@webkit.org, dglazkov@chromium.org, fishd@chromium.org, jamesr@chromium.org or tkent@chromium.org before submitting, as this patch contains changes to the Chromium public API. See also https://trac.webkit.org/wiki/ChromiumWebKitAPI.
Will make a smaller change.
Created attachment 147197 [details] a much smaller patch
Comment on attachment 147197 [details] a much smaller patch Ok. I'm not 100% confident in all of the changes you're making to the GYP file, but the compiler should tell us if we've goofed up there.
Comment on attachment 147197 [details] a much smaller patch Thanks Adam. I've verified the patch on Chromium's try bots.
Comment on attachment 147197 [details] a much smaller patch Clearing flags on attachment: 147197 Committed r120220: <http://trac.webkit.org/changeset/120220>
All reviewed patches have been landed. Closing bug.
Re-opened since this is blocked by 89021
Created attachment 147367 [details] patch for re-applying with some changes in WebKit.gyp reverted
Comment on attachment 147367 [details] patch for re-applying with some changes in WebKit.gyp reverted Clearing flags on attachment: 147367 Committed r120270: <http://trac.webkit.org/changeset/120270>