bug 1324019 - remove TreeMutation::mEventTree field r=davidb

It causes warnings if the logging define is undefined as it is on aurora,
   because the field is then unused.  Since the only uses can easily be
   replaced with the mQueueEvents field we can just do that and remove the
   useless field.
This commit is contained in:
Trevor Saunders 2016-12-16 11:34:23 -05:00
parent c975fbc136
commit fc68eda3fc
2 changed files with 2 additions and 4 deletions

View File

@ -25,7 +25,6 @@ EventTree* const TreeMutation::kNoEventTree = reinterpret_cast<EventTree*>(-1);
TreeMutation::TreeMutation(Accessible* aParent, bool aNoEvents) :
mParent(aParent), mStartIdx(UINT32_MAX),
mStateFlagsCopy(mParent->mStateFlags),
mEventTree(aNoEvents ? kNoEventTree : nullptr),
mQueueEvents(!aNoEvents)
{
#ifdef DEBUG
@ -33,7 +32,7 @@ TreeMutation::TreeMutation(Accessible* aParent, bool aNoEvents) :
#endif
#ifdef A11Y_LOG
if (mEventTree != kNoEventTree && logging::IsEnabled(logging::eEventTree)) {
if (mQueueEvents && logging::IsEnabled(logging::eEventTree)) {
logging::MsgBegin("EVENTS_TREE", "reordering tree before");
logging::AccessibleInfo("reordering for", mParent);
Controller()->RootEventTree().Log();
@ -119,7 +118,7 @@ TreeMutation::Done()
#endif
#ifdef A11Y_LOG
if (mEventTree != kNoEventTree && logging::IsEnabled(logging::eEventTree)) {
if (mQueueEvents && logging::IsEnabled(logging::eEventTree)) {
logging::MsgBegin("EVENTS_TREE", "reordering tree after");
logging::AccessibleInfo("reordering for", mParent);
Controller()->RootEventTree().Log();

View File

@ -47,7 +47,6 @@ private:
Accessible* mParent;
uint32_t mStartIdx;
uint32_t mStateFlagsCopy;
EventTree* mEventTree;
/*
* True if mutation events should be queued.