Last Comment Bug 307088 - tab not responding if append to parent with hidden attribute
: tab not responding if append to parent with hidden attribute
Status: NEW
: testcase
Product: Core
Classification: Components
Component: XBL (show other bugs)
: Trunk
: x86 Linux
: -- normal with 2 votes (vote)
: ---
Assigned To: Nobody; OK to take it and work on it
:
Mentors:
Depends on: 265086
Blocks:
  Show dependency treegraph
 
Reported: 2005-09-04 23:44 PDT by jooaun.saw
Modified: 2009-08-23 18:02 PDT (History)
9 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
test case (2.31 KB, application/vnd.mozilla.xul+xml)
2005-09-04 23:45 PDT, jooaun.saw
no flags Details
testcase2 (2.20 KB, application/vnd.mozilla.xul+xml)
2005-09-06 03:13 PDT, Martijn Wargers [:mwargers] (not working for Mozilla)
no flags Details

Description jooaun.saw 2005-09-04 23:44:19 PDT
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.9) Gecko/20050711 Firefox/1.0.5
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.9) Gecko/20050711 Firefox/1.0.5

Tab does not respond to clicks, so not able to switch tabs.

Reproducible: Always

Steps to Reproduce:
1. Set parent hidden attribute to true.
2. Append tabbox with tabs to parent element.
3. Set parent hidden attribute to false.

Actual Results:  
tab does not respond to mouse clicks.

Expected Results:  
Tab should respond to mouse clicks, and switch tab. If programmer not allowed to
append element to a hidden element, should throw exception.
Comment 1 jooaun.saw 2005-09-04 23:45:07 PDT
Created attachment 194889 [details]
test case
Comment 2 Martijn Wargers [:mwargers] (not working for Mozilla) 2005-09-05 01:51:38 PDT
I think this probably suffers from bug 233639.
And possibly also from bug 266590.
Comment 3 Martijn Wargers [:mwargers] (not working for Mozilla) 2005-09-05 02:38:28 PDT
Hmm, actually this might depend on bug 307098.
And possibly also from bug 266590.
Comment 4 Martijn Wargers [:mwargers] (not working for Mozilla) 2005-09-05 03:21:26 PDT
Testcase2 from bug 307085 also suffers from this bug, and that testcase doesn't
use any hidden=true, so I guess this could have nothing to do with bug 307098.
I get a js error, by the way, when clicking on any of the tabs:
Error: uncaught exception: Permission denied to create wrapper for object of
class UnnamedClass
Comment 5 Boris Zbarsky [:bz] (Out June 25-July 6) 2005-09-05 20:31:21 PDT
Does this start working if you clone an existing (empty?) tabbox instead of
creating a new one with createElement?
Comment 6 Martijn Wargers [:mwargers] (not working for Mozilla) 2005-09-06 03:13:51 PDT
Created attachment 194996 [details]
testcase2

(In reply to comment #5)
> Does this start working if you clone an existing (empty?) tabbox instead of
> creating a new one with createElement?
No.
Comment 7 Alex Vincent [:WeirdAl] 2005-10-19 19:35:56 PDT
I believe this bug to cover the same issue as bug 209701. Unlike this bug,
however, bug 209701 has at least an attempt at a patch. I am therefore resolving
this bug as a duplicate of bug 209701. Feel free to reopen if you disagree.

*** This bug has been marked as a duplicate of 209701 ***
Comment 8 Boris Zbarsky [:bz] (Out June 25-July 6) 2005-11-06 19:01:06 PST
I see nothing here suggesting this is related to bug 209701.  What made you think it is?
Comment 9 Alex Vincent [:WeirdAl] 2005-11-06 19:12:21 PST
The error message was identical when I looked.  Also, both cases involved remote XUL, which is known to have some difficulties in the XUL widget accessing some of its XBL-ized properties.
Comment 10 Boris Zbarsky [:bz] (Out June 25-July 6) 2005-11-06 19:22:36 PST
Ah, I see what the problem is.  Setting selectedIndex to 0 does nothing since  that node doesn't have an XBL binding (see bug 265086 and the fact that we apply bindings on frame construction).  Then when you click nothing is selected, and tabbox can't deal with that.

You could fix this by cloning the tabs element, I guess...
Comment 11 Boris Zbarsky [:bz] (Out June 25-July 6) 2005-11-06 19:24:12 PST
And the error message in this case comes when tabbox sees nothing is selected and tries to throw Components.results.NS_ERROR_FAILURE.  Wrapping _that_ is what fails (due to bug 209701).  But by that point it's way too late anyway -- we're already throwing.

Note You need to log in before you can comment on or make changes to this bug.