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)
Firefox for Android Graveyard
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: wesley_huang, Unassigned)
References
Details
Attachments
(1 obsolete file)
No description provided.
Comment hidden (mozreview-request) |
Comment 2•7 years ago
|
||
mozreview-review |
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 3•7 years ago
|
||
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)
Updated•7 years ago
|
Attachment #8876076 -
Attachment is obsolete: true
Updated•7 years ago
|
Updated•7 years ago
|
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•