TestGtkEmbedNotebook background tab broken

RESOLVED FIXED in mozilla1.9.3a1

Status

()

RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: yaneti, Assigned: karlt)

Tracking

(Depends on: 1 bug)

Trunk
mozilla1.9.3a1
x86_64
Linux
Points:
---
Dependency tree / graph
Bug Flags:
blocking1.9.2 +

Firefox Tracking Flags

(status1.9.2 beta5-fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

9 years ago
Fedora rawhide x86_64

In TestGtkEmbedNotebook built against xulrunner-1.9.2.1-0.4.b3.fc13.x86_64 the tab thats initially created in the background no longer works.

Reverting 
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/f212daadb16b
fixes it.
(Assignee)

Updated

9 years ago
Blocks: 509278
(Assignee)

Comment 1

9 years ago
Created attachment 414791 [details] [diff] [review]
check toplevel window state in hierarchy_changed_cb

This was meant to check the window state of the toplevel not the container widget.  This problem didn't show up for XUL apps because for them the toplevel is in the same nsIWidget as the container and so hasn't been shown at this point, but this is a problem for embedding clients.
Assignee: nobody → karlt
Status: NEW → ASSIGNED
Attachment #414791 - Flags: review?(roc)
(Assignee)

Updated

9 years ago
Flags: blocking1.9.2?
Flags: blocking1.9.2? → blocking1.9.2+
(Assignee)

Updated

9 years ago
Depends on: 531336
(Assignee)

Comment 2

9 years ago
Created attachment 414792 [details] [diff] [review]
check toplevel window state in hierarchy_changed_cb
Attachment #414792 - Flags: review?(roc)
(Assignee)

Updated

9 years ago
Attachment #414791 - Attachment is obsolete: true
(Assignee)

Comment 3

9 years ago
http://hg.mozilla.org/mozilla-central/rev/abd0a4aa3b68
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Whiteboard: [needs 192 landing]
Target Milestone: --- → mozilla1.9.3a1
Version: 1.9.2 Branch → Trunk
(Assignee)

Comment 4

9 years ago
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/e4085b47c5be
status1.9.2: --- → final-fixed
Whiteboard: [needs 192 landing]
You need to log in before you can comment on or make changes to this bug.