Closed Bug 1366662 Opened 8 years ago Closed 7 years ago

(photon) Engineering landing plan and implementation so that Photon UIs can be landed incrementally

Categories

(Firefox for Android Graveyard :: General, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: wesley_huang, Unassigned)

References

Details

Attachments

(1 obsolete file)

No description provided.
Comment on attachment 8876076 [details] Bug 1366662 - Add build config for Photon with default enabled on gradle build, https://reviewboard.mozilla.org/r/147508/#review151942 This adds the flag, but doesn't actually _use_ the `photon` skin flavor dimension in the Gradle build. That is, this is still using the `officialAustralis` Gradle configuration and not the `officialPhoton` Gradle configuration. You'll want to make the following conditional on `MOZ_ANDROID_PHOTON`: http://searchfox.org/mozilla-central/source/mobile/android/base/Makefile.in#237 http://searchfox.org/mozilla-central/source/mobile/android/base/Makefile.in#241 and also http://searchfox.org/mozilla-central/source/mobile/android/base/Makefile.in#483 http://searchfox.org/mozilla-central/source/toolkit/mozapps/installer/upload-files-APK.mk#35 I don't think you need to do this -- skins will be rare -- but we could also make this `MOZ_ANDROID_SKIN_FLAVOR_DIMENSION`, have that be `photon` or `australis`, and substitute it (with the right capitalization) as necessary. No need to actually do this, but be aware that it's a possiblity, if the conditional approach is hard to do. ::: mobile/android/config/mozconfigs/android-api-15-gradle/nightly:5 (Diff revision 1) > . "$topsrcdir/mobile/android/config/mozconfigs/common" > > . "$topsrcdir/mobile/android/config/mozconfigs/android-api-15/nightly" > > +ac_add_options --with-mobile-photon Sebastian: Max wants to get a build with Photon enabled in automation; the Gradle build is there for the taking, so I support doing this. ::: mobile/android/moz.configure:71 (Diff revision 1) > > project_flag('MOZ_ANDROID_DOWNLOAD_CONTENT_SERVICE', > help='Background service for downloading additional content at runtime', > default=True) > > +option('--with-mobile-photon', Let's go with `--enable-photon`, since these options are already `mobile/android` specific. ::: mobile/android/moz.configure:72 (Diff revision 1) > project_flag('MOZ_ANDROID_DOWNLOAD_CONTENT_SERVICE', > help='Background service for downloading additional content at runtime', > default=True) > > +option('--with-mobile-photon', > + help='Build with refreshing Photon UI on mobile', Maybe just 'Enable Photon UI'? The "on mobile" is implied already, and it won't be refreshing (or refreshed) for very long -- it'll be default.
Attachment #8876076 - Flags: review?(nalexander) → review-
Comment on attachment 8876076 [details] Bug 1366662 - Add build config for Photon with default enabled on gradle build, Note I'll be on PTO starting tomorrow until the All-Hands. But go ahead and land as soon as :nalexander gives r+. > Sebastian: Max wants to get a build with Photon enabled in automation; the Gradle build is there for the taking, so I support doing this. Yeah, sounds like a good plan. Let's do this.
Attachment #8876076 - Flags: review?(s.kaspari)
Attachment #8876076 - Attachment is obsolete: true
Blocks: 1372486
No longer blocks: 1372486
Depends on: 1372486
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: