Closed Bug 1272732 Opened 4 years ago Closed 4 years ago

Specifying --disable-android-apz breaks Fennec build.

Categories

(Firefox for Android :: Toolbar, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 49
Tracking Status
firefox48 --- fixed
firefox49 --- fixed

People

(Reporter: rbarker, Assigned: rbarker)

References

Details

Attachments

(1 file, 1 obsolete file)

When specifying --disable-android-apz, the build will fail trying to build file FlingOverScrollerAnimation.cpp which should only be include in APZ builds.
Summary: Fennec JPZ build broken in 49 nightly → Specifying --disable-android-apz breaks Fennec build.
Assignee: nobody → rbarker
Comment on attachment 8752286 [details] [diff] [review]
0001-Bug-1272732-Specifying-disable-android-apz-breaks-Fennec-build.-r-16051311-989265f.patch

Review of attachment 8752286 [details] [diff] [review]:
-----------------------------------------------------------------

::: gfx/layers/moz.build
@@ +253,5 @@
>          'ipc/GonkNativeHandleUtils.cpp',
>          'ipc/ShadowLayerUtilsGralloc.cpp',
>      ]
>  
> +if CONFIG['MOZ_WIDGET_TOOLKIT'] == 'android' and CONFIG['MOZ_ANDROID_APZ']:

Can we do just the MOZ_ANDROID_APZ check? That implies the toolkit thing already, and would make it consistent with the ifdefs in the code.
Attachment #8752286 - Flags: review?(bugmail.mozilla) → review+
Simplify patch. Carry forward r+ from :kats
Attachment #8752286 - Attachment is obsolete: true
https://hg.mozilla.org/mozilla-central/rev/3cc184cc0bc2
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Randall, please make sure you mark dependencies properly. This needs uplifting to 48.
Comment on attachment 8752299 [details] [diff] [review]
0001-Bug-1272732-Specifying-disable-android-apz-breaks-Fennec-build.-r-16051311-4eceae6.patch

Approval Request Comment
[Feature/regressing bug #]: bug 1229462
[User impact if declined]: harder to turn off APZ on Fennec if we need to (not likely that we will, but just in case)
[Describe test coverage new/current, TreeHerder]: tested locally
[Risks and why]: low risk, just a build config change that doesn't affect the default configuration
[String/UUID change made/needed]: none
Attachment #8752299 - Flags: approval-mozilla-beta?
Comment on attachment 8752299 [details] [diff] [review]
0001-Bug-1272732-Specifying-disable-android-apz-breaks-Fennec-build.-r-16051311-4eceae6.patch

Let's do it in case we disable it!
should be in 48 beta 5
Attachment #8752299 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.