888067 - Defect - Combined bar shouldnt appear on the js error console panel

Status: VERIFIED FIXED

(Firefox for Metro Graveyard :: General, defect, P2)

Description

[Allison Naaktgeboren :ally]

Attachment: combined url/appbar on top of error console

was not on yesterday's build.

also of note: the bar won't dismiss.

Comment 1

[Jim Mathies [:jimm]]

This only happens if the startpage is visible when you invoke the console. If you open from a web site it falls away. This due to a left over "startpage".

Comment 2

[Jim Mathies [:jimm]]

Attachment: fix

Comment 3

[Jonathan Wilde [:jwilde]]

Comment on attachment 768883 [details] [diff] [review]
fix

Review of attachment 768883 [details] [diff] [review]:
-----------------------------------------------------------------

We really should just move the position of the panel UI in terms of z-index rather than adding extra logic to PanelUI. I'll go make the second patch for this. :D

Comment 4

[Jonathan Wilde [:jwilde]]

Attachment: fix v2

updating the dom z-order instead of updating panelui

Comment 5

[Jonathan Wilde [:jwilde]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/3029d3574f2e

Comment 6

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/3029d3574f2e

Comment 7

[Cornel Ionce [:noni] [Hubs QA]]

User Agent : Mozilla/5.0 (Windows NT 6.2; WOW64; rv:25.0) Gecko/20130712 Firefox/25.0

Verified as fixed on latest nightly build.
The combined url/appbar is not shown anymore on the js error console panel.

Comment 8

[Samvedana (:Samvedana)]

User Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:26.0) Gecko/20100101 Firefox/26.0
Build ID: 20130816030205
Built from http://hg.mozilla.org/mozilla-central/rev/1ed5a88cd4d0

WFM
Tested on windows 8 using latest nightly for iteration-12. The combined url/appbar is not shown anymore on the js error console panel.