Closed Bug 615841 Opened 12 years ago Closed 12 years ago

ourA11yNotification is only used in nsFrameManager::ReResolveStyleContext ifdef ACCESSIBILITY

Categories

(Core :: Layout, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla2.0b8

People

(Reporter: timeless, Assigned: timeless)

Details

Attachments

(1 file)

#ifdef ACCESSIBILITY
      A11yNotificationType ourA11yNotification = eDontNotify;


layout/base/nsFrameManager.cpp:
 In member function ‘nsChangeHint nsFrameManager::ReResolveStyleContext(nsPresContext*, nsIFrame*, nsIContent*, nsStyleChangeList*, nsChangeHint, nsRestyleHint, mozilla::css::RestyleTracker&, nsFrameManager::DesiredA11yNotifications, nsTArray<nsIContent*, nsTArrayDefaultAllocator>&)’:
1416: warning: unused variable ‘ourA11yNotification’
Attached patch move into ifdefSplinter Review
Assignee: nobody → timeless
Status: NEW → ASSIGNED
Attachment #494354 - Flags: review?(bzbarsky)
Attachment #494354 - Flags: approval2.0?
Attachment #494354 - Flags: review?(bzbarsky)
Attachment #494354 - Flags: review+
Attachment #494354 - Flags: approval2.0?
Attachment #494354 - Flags: approval2.0+
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/6c885a625cb9
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Keywords: checkin-needed
OS: Mac OS X → All
Hardware: x86 → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
You need to log in before you can comment on or make changes to this bug.