Nathan Froyd [:froydnj] Reporter
	Description • 7 years ago

We're planning on not enabling Stylo on Android for the initial release, but it'd be good to do it sooner rather than later.

	Makoto Kato [:m_kato] Assignee
	Comment 1 • 7 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=b83d8a6a1323daf7d42c75148246b9f4b4bbd476 for android-api-16.

When using gcc 4.9, APK is 1.6MB+
When using clang 5.0 (the latest NDK), APK is 1.4MB+

	Makoto Kato [:m_kato] Assignee
	Comment 2 • 7 years ago

(clang's data is old changeset, not same changeset)

	Chris Peterson [:cpeterson]
	Comment 3 • 7 years ago

status-firefox57=wontfix unless someone thinks this bug should block 57

	Makoto Kato [:m_kato] Assignee
	Comment 4 • 7 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=9008dc8fe81f62ea1f23529c876ddd4be97c04cf

	Jan Beich
	Comment 6 • 7 years ago

(In reply to Makoto Kato [:m_kato] from comment #5)
> All tests are passed with stylo, So let's turn on stylo even if Android.

Despite Try build from comment 4 using "enable_stylo = None" ?

	Makoto Kato [:m_kato] Assignee
	Comment 7 • 7 years ago

(In reply to Jan Beich from comment #6)
> (In reply to Makoto Kato [:m_kato] from comment #5)
> > All tests are passed with stylo, So let's turn on stylo even if Android.
> 
> Despite Try build from comment 4 using "enable_stylo = None" ?

Ah, I send it again. https://treeherder.mozilla.org/#/jobs?repo=try&revision=691b5f43a2beb2db880cb0a64aabb3255f9b7304

	Makoto Kato [:m_kato] Assignee
	Comment 8 • 7 years ago

(Last week, xpcshell was passed again, also, reftests was already passed )

	James Willcox (:snorp) (jwillcox@mozilla.com) (he/him)
	Comment 9 • 7 years ago
mozreview-review

Comment on attachment 8930416 [details]
Bug 1366049 - enable Stylo for Android.

https://reviewboard.mozilla.org/r/201586/#review206892

	Bob Clary [:bc] (inactive)
	Comment 10 • 7 years ago

I don't see that Autophone has been tested yet. I have submitted tests using your try build at:
<https://treeherder.mozilla.org/#/jobs?repo=try&revision=691b5f43a2beb2db880cb0a64aabb3255f9b7304&group_state=expanded&filter-searchStr=autophone>

Let's wait for that to complete before landing.

	Pulsebot
	Comment 11 • 7 years ago

Pushed by m_kato@ga2.so-net.ne.jp:
https://hg.mozilla.org/integration/autoland/rev/9353a526c194
enable Stylo for Android. r=snorp

	Noemi Erli[:noemi_erli]
	Comment 12 • 7 years ago

Backed out changeset 9353a526c194 (bug 1366049) for bustage in /python/mozbuild/mozbuild/test/configure/lint.py r=backout on a CLOSED TREE

https://hg.mozilla.org/integration/autoland/rev/565c65d130adbeb633655fcddc8af6b2aaf818fb

https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=9353a526c1940a9ce4e0c07b79ba08727b8fa176&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception&filter-resultStatus=retry&filter-resultStatus=usercancel&filter-resultStatus=runnable

	Pulsebot
	Comment 14 • 7 years ago

Pushed by m_kato@ga2.so-net.ne.jp:
https://hg.mozilla.org/integration/autoland/rev/db56323cd08f
enable Stylo for Android. r=snorp

	Cosmin Sabou [:CosminS]
	Comment 15 • 7 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/db56323cd08f

	Liz Henry (:lizzard Please n-i to RyanVM, jcristau, or pascal)
	Comment 16 • 7 years ago

Adding a release note for 59 dev edition/beta; New, Added support for Stylo for Android

	Chris Mills [:cmills]
	Comment 17 • 7 years ago

I have added a note to the Fx59 rel notes to cover this; let me know if you think this reads OK, and if there's anything more you'd like me to say:

https://developer.mozilla.org/en-US/Firefox/Releases/59#Stylo_comes_to_Firefox_for_Android_in_59

	Chris Peterson [:cpeterson]
	Comment 18 • 7 years ago

(In reply to Chris Mills (Mozilla, MDN editor) [:cmills] from comment #17)
> I have added a note to the Fx59 rel notes to cover this; let me know if you
> think this reads OK, and if there's anything more you'd like me to say:
> 
> https://developer.mozilla.org/en-US/Firefox/Releases/
> 59#Stylo_comes_to_Firefox_for_Android_in_59

cmills, we are slipping Stylo for Android from Firefox 59 to 60, so we should remove Stylo from the Firefox 59's Android release notes.

	Chris Mills [:cmills]
	Comment 19 • 7 years ago

(In reply to Chris Peterson [:cpeterson] from comment #18)
> (In reply to Chris Mills (Mozilla, MDN editor) [:cmills] from comment #17)
> > I have added a note to the Fx59 rel notes to cover this; let me know if you
> > think this reads OK, and if there's anything more you'd like me to say:
> > 
> > https://developer.mozilla.org/en-US/Firefox/Releases/
> > 59#Stylo_comes_to_Firefox_for_Android_in_59
> 
> cmills, we are slipping Stylo for Android from Firefox 59 to 60, so we
> should remove Stylo from the Firefox 59's Android release notes.

Cool, thanks for letting me know - removed.

	Julien Cristau [:jcristau]
	Comment 20 • 6 years ago

Added in the draft 60.0beta release notes: