Closed Bug 1374832 Opened 4 years ago Closed 4 years ago
Don't depend on com
.google .android .gms:play-services-ads in Gradle configuration when MMA/Leanplum is enabled
Bug 1374832 - Don't depend on com.google.android.gms:play-services-ads when MMA/Leanplum is enabled.
59 bytes, text/x-review-board-request
Bug 1351585 added a dependency on com.google.android.gms:play-services-ads in the Gradle configuration when MMA/Leanplum is enabled. I remember making a note somewhere about this, but I can't find it in my ~/org or Bugzilla, so I must not have communicated it. AFAIK, we don't actually require play-services-ads for the MMA integration -- it just _looks_ like we do. (The ads/identity looking symbols really come from basement or base, I can't remember which.) I found this while digging into APK differences as part of Bug 1355625. It doesn't block that ticket, but we should unify the included sources as part of the larger "Gradle in automation" effort.
Comment on attachment 8879740 [details] Bug 1374832 - Don't depend on com.google.android.gms:play-services-ads when MMA/Leanplum is enabled. https://reviewboard.mozilla.org/r/151080/#review155978 LGTM
Attachment #8879740 - Flags: review?(max) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/867702253c48 Don't depend on com.google.android.gms:play-services-ads when MMA/Leanplum is enabled. r=maliu
Product: Firefox for Android → Firefox Build System
Target Milestone: Firefox 56 → mozilla56
You need to log in before you can comment on or make changes to this bug.