Bug 987871 - Enable flag to fix double event firing on checkbox toggle

dothq · Apr 1, 2014 · 12f06d4 · 12f06d4
1 parent 5f1c71a
commit 12f06d4
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 10 deletions.
diff --git a/accessible/src/generic/Accessible.h b/accessible/src/generic/Accessible.h
@@ -873,7 +873,7 @@ class Accessible : public nsIAccessible,
     eGroupInfoDirty = 1 << 5, // accessible needs to update group info
     eIgnoreDOMUIEvent = 1 << 6, // don't process DOM UI events for a11y events
 
-    eLastStateFlag = eGroupInfoDirty
+    eLastStateFlag = eIgnoreDOMUIEvent
   };
 
   /**
@@ -987,7 +987,7 @@ class Accessible : public nsIAccessible,
   int32_t mIndexInParent;
 
   static const uint8_t kChildrenFlagsBits = 2;
-  static const uint8_t kStateFlagsBits = 6;
+  static const uint8_t kStateFlagsBits = 7;
   static const uint8_t kContextFlagsBits = 1;
   static const uint8_t kTypeBits = 6;
   static const uint8_t kGenericTypesBits = 12;

diff --git a/accessible/tests/mochitest/jsat/test_content_integration.html b/accessible/tests/mochitest/jsat/test_content_integration.html
@@ -42,10 +42,6 @@
           [ContentMessages.activateCurrent(), {
             speak: 'checked'
           }],
-          [null, {
-            speak: 'checked',
-            speak_checkFunc: 'todo_isnot'
-          }],
           [ContentMessages.simpleMoveNext, {
             speak: 'much range label'
           }],
@@ -74,10 +70,6 @@
           [ContentMessages.activateCurrent(), {
             speak: 'unchecked'
           }],
-          [null, {
-            speak: 'unchecked',
-            speak_checkFunc: 'todo_isnot'
-          }],
           [ContentMessages.simpleMovePrevious, {
             speak: 'wow heading level 1'
           }],