Closed Bug 1372221 Opened 7 years ago Closed 7 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)

Product:
Core
Component:
Disability Access APIs
Type:
enhancement
Priority:
Not set
Severity:
blocker

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla56
Tracking Flags:
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: 7 years ago
status-firefox56: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.