servo / skia

Skia
BSD 3-Clause "New" or "Revised" License
109 stars 61 forks source link

Android 7 support #155

Open Dushistov opened 6 years ago

Dushistov commented 6 years ago

On Android 7 I can not see any text drawn by skia.

I suppose this is because of path mentioned in src/ports/SkFontConfigParser_android.cpp - /system/etc/system_fonts.xml not exists on Android 7, instead there is /system/etc/fonts.xml, any plans to merge at least https://github.com/google/skia/commit/94fa4b99e2a53e997a90c7808cc5263f1bf40c9f ?

jdm commented 6 years ago

The work to update to a newer skia revision tends to be done by whoever needs the new revision. We would accept a pull request to perform an update.

Dushistov commented 6 years ago

@jdm

The work to update to a newer skia revision tends to be done by whoever needs the new revision

What variant do you accept? Cherry-pick or merge also unrelated changes?

jdm commented 6 years ago

We would accept either, but performing a full update to a new skia would probably be preferred.