Closed Bug 412931 Opened 17 years ago Closed 17 years ago

Tabs not exposed unless 3 of them

Categories

(Core :: Disability Access APIs, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
mozilla1.9beta3

People

(Reporter: steve, Assigned: aaronlev)

References

Details

(Keywords: access, regression)

Attachments

(2 files, 1 obsolete file)

User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9b3pre) Gecko/2008011704 Minefield/3.0b3pre Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9b3pre) Gecko/2008011704 Minefield/3.0b3pre The ROLE_PAGE_TAB_LIST for the web page tabs does not appear unless there are 3 tabs. In addition the child change events are not always generated as tabs come and go Reproducible: Always Steps to Reproduce: 1.setup minefield with a single page open (no tabs) 2. use accerciser to look for page tab (hint ctrl-alt-? or browse 0.16.2) 3. Opne and tab a look again 4 open a tab and look again 5. close the tab and look again 6 refresh accerciser and look again Actual Results: The tab list and tabs only appear when third tab opened Accerciser does not know the tab is closed. Expected Results: Tabs always correctly exposed in a11y as well as events raised.
Marco, can you confirm?
Attached file python to prints page tabs (deleted) —
Marco, hope this helps, it prints the number of tabs and their names or an error message. run as 'python test.py'
Not reproducible on my box.
In addition, the behaviour has changed in the meantime: Only the inactive tab is now exposed. So, if I have 2 tabs open, and am on tab 2, only tab 1 is now exposed, but tab 2 isn't.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: blocking1.9?
Marco, were you able to duplicate this on Windows?
(In reply to comment #7) > Marco, were you able to duplicate this on Windows? Yes! If you look at the hierarchy in AccExplorer, you'll see that the active tab is missing from the tabs node under the nameless grouping node. In the January 12 and successive builds, the tabs node and its children were missing alltogether.
Keywords: access, regression
OS: Linux → All
Hardware: PC → All
Target Milestone: --- → mozilla1.9beta3
Aaron: how serious is this in terms of a regression over b2? does it make the beta entirely useless for accessibility testing? Just wondering if it can wait for beta4
(In reply to comment #10) > Created an attachment (id=300643) [details] > Easy fix, but I need to investigate why 1st parent invalidatation for > DOM_CREATE didn't take Doesn't this revert part of the fix for bug 408997, and would thus reintroduce that problem?
It was an optimization that's being removed, so no. But I still am investigating because I don't understand why we need to remove that optimization.
Moving out to beta4, feel free to renom if you disagree.
Flags: blocking1.9? → blocking1.9+
Target Milestone: mozilla1.9beta3 → mozilla1.9beta4
This may fix other problems as well. It's a flaw in our logic that we've had ever since we added the eCoalesceFromSameSubtree capability to simplify the events we fire.
Attachment #300643 - Attachment is obsolete: true
Attachment #300779 - Flags: review?(surkov.alexander)
Comment on attachment 300779 [details] [diff] [review] Asynch show events are removed from the queue when an ancestor is also shown, in order to avoid redundancy. However, we still need to invalidate children in the parents of the shown nodes for those re >+ if (eventType == nsIAccessibleEvent::EVENT_ASYNCH_SHOW) { >+ if (accessible) { Is it possible accessible is null?
> Is it possible accessible is null? Yes, absolutely. We fire the delayed event for a dom node, which could be any kind of node like a span etc. When there is no node we will end up calling FireShowHideEvents() which will recurse back here for each first line accessible in that inaccessible node's subtree.
I ran with this patch most of the day, and consider it to be solid. If we get review from Alexander I'd like to request approval for beta 3.
I'm not sure if Surkov watches you so we need to CC him 1st.
(In reply to comment #18) > I'm not sure if Surkov watches you so we need to CC him 1st. > I'm watching for you :)
Attachment #300779 - Flags: review?(surkov.alexander) → review+
Attachment #300779 - Flags: approval1.9b3?
Marco ran with this all day and wants to get it in. This fixes a cache corruption bug -- the type of bug that leads to instability.
Target Milestone: mozilla1.9beta4 → mozilla1.9beta3
Attachment #300779 - Flags: approval1.9b3? → approval1.9b3+
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Verified using Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9b3) Gecko/2008020514 Firefox/3.0b3
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: