Closed Bug 1372221 Opened 2 years ago Closed 2 years ago

"accessible/base/NotificationController.cpp:659:17: error: unused variable 'containerElm' [-Werror=unused-variable]" bustage on Beta 55

Categories

(Core :: Disability Access APIs, enhancement, blocker)

enhancement
Not set
blocker

Tracking

()

RESOLVED FIXED
mozilla56
Tracking Status
firefox-esr52 --- unaffected
firefox53 --- unaffected
firefox54 --- unaffected
firefox55 + fixed
firefox56 --- fixed

People

(Reporter: RyanVM, Assigned: RyanVM)

References

Details

[Tracking Requested - why for this release]:

Looks like bug 1353094 removed the one usage of containerElm outside of A11Y_LOG ifdefs.

https://hg.mozilla.org/mozilla-central/rev/7636849e6f58

Per IRC discussion with dbolter, we can just ifdef the setting of containerElm to fix this.
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/dd923bb97533
Set containerElm only if A11Y_LOG is defined. r=davidb
https://hg.mozilla.org/mozilla-central/rev/dd923bb97533
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.