1252694 - Set Android GCM sender ID in branding

Status: RESOLVED FIXED

(Firefox Build System :: Android Studio and Gradle Integration, defect)

Description

[Nick Alexander :nalexander [he/him]]

Bug 1207708 added support for GCM sender IDs to the Fennec build system.  In the fullness of time, I now realize that there's no need for the secrecy around sender IDs.  In addition, we had plans to allow changing the sender ID dynamically, but newer GPS libraries suggest that's not the best approach.  (Those libraries bake the sender ID into the Android resources directly.)  Let's simplify this.

Comment 1

[Nick Alexander :nalexander [he/him]]

Attachment: MozReview Request: Bug 1252694 - Set MOZ_ANDROID_GCM_SENDERID directly in branding. r?glandium

Non-Mozilla distributors may wish to run their own autopush endpoint,
using a sender ID and corresponding Google API key that they control.
This simplifies that just a little bit, and gets Mozilla's release
engineering out of the business of managing non-sensitive secrets.

In the future, this sender ID will be baked into the Android APK's
string resources, in accordance with newer Google Play Services
library requirements.

Review commit: https://reviewboard.mozilla.org/r/37499/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/37499/

Comment 2

[Mike Hommey [:glandium]]

Comment on attachment 8725474 [details]
MozReview Request: Bug 1252694 - Set MOZ_ANDROID_GCM_SENDERID directly in branding. r?glandium

https://reviewboard.mozilla.org/r/37499/#review34035

Comment 3

[Mike Hommey [:glandium]]

Comment on attachment 8725474 [details]
MozReview Request: Bug 1252694 - Set MOZ_ANDROID_GCM_SENDERID directly in branding. r?glandium

https://reviewboard.mozilla.org/r/37499/#review34037

Sorry for the confusion, I had forgotten to check the ship-it box.

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/2c58d017fcf9

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/2c58d017fcf9