Closed
Bug 773466
Opened 12 years ago
Closed 12 years ago
Console will stop displaying output
Categories
(DevTools :: Console, defect, P2)
DevTools
Console
Tracking
(firefox15+ fixed, firefox16+ verified, firefox17 verified)
VERIFIED
FIXED
Firefox 17
People
(Reporter: ianbicking, Assigned: msucan)
Details
Attachments
(1 file, 1 obsolete file)
5.12 KB,
patch
|
akeybl
:
approval-mozilla-aurora+
lsblakk
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
I'm having a hard time figuring out when this happens exactly, but sometimes the console stops updating with new content. Opening and closing the console (e.g., with Cmd-Alt-K) sometimes fixes it, or at least causes some output to appear that was hidden. This seems to reproduce: > console.dir(function () {}); > console.log('hey'); The "hey" won't display. console.dir({a: 1}) won't cause this, but trying to use it on some other objects (besides simple objects) also causes problems.
Comment 1•12 years ago
|
||
I think it actually prints "hey", but doesn't scroll up the content.
So you have to scroll to reach the "hey" string. But, when the mouse
is over the dir-box, you can't scroll, for some reason.
So 2 problems here:
1) we should ensure that the second log is visible by scrolling to the end of the rhichlistbox. I think we already do that, but for some reason, it doesn't work with the dir.
2) we can't scroll the richlistbox when the mouse is over the dir-box
My STR:
> console.dir(document.body)
> console.log("foo")
Comment 2•12 years ago
|
||
(In reply to Paul Rouget [:paul] from comment #1) > 2) we can't scroll the richlistbox when the mouse is over the dir-box IIRC that's because the dir-box can overflow and get its own scrollbars.
Assignee | ||
Comment 3•12 years ago
|
||
Yep, Panos is correct. The UX for console.dir() is quite poor - it causes problems.
Assignee | ||
Comment 4•12 years ago
|
||
Ian: this seems to be a duplicate of bug 712553. Can you confirm?
Reporter | ||
Comment 5•12 years ago
|
||
I don't think it's a dup of bug 712553. It doesn't seem like a scrolling issue at all. If I open a console on bugzilla.mozilla.org (chosen just for convenience): [11:18:34.911] < console.log('test1') [11:18:34.913] > undefined [11:18:34.914] test1 @ Web Console:1 [11:18:40.566] < console.dir(function () {}) [11:18:40.569] > undefined At this point console.dir doesn't write anything, and it's not close to needing to scroll – there's lots of blank space left in the console. When I enter "console.log('test2')" I don't get any output, neither "< console.log('test2')" nor "test2 @ Web Console:1". If I close and reopen the console I see only "test1" and if I enter "console.log('test3')" I see no output or changes to the console. This is on Nightly, 16.0a1 (2012-07-09), reproduced with two different profiles.
Assignee | ||
Comment 6•12 years ago
|
||
Thanks for your report. I will investigate the issue ASAP.
Assignee | ||
Comment 7•12 years ago
|
||
Problem identified. I have a fix ready, just need to write a test.
Assignee: nobody → mihai.sucan
Status: NEW → ASSIGNED
Priority: -- → P2
Version: 16 Branch → Trunk
Assignee | ||
Comment 8•12 years ago
|
||
Proposed patch with fix and a test. Problem was that HUDService-content.js did not ensure we send arguments either as strings or as arrays - where arrays are meant to be the "remote object represenation". This broke HUDService.jsm's code which assumed that. Also, in HUDService.jsm we now only pass the remoteObject to the tree view if it is an array - making sure the bug doesn't occur in HUDService.jsm either. The test does exactly what Ian suggested we try. Please let me know if further changes are needed. Thanks!
Attachment #641954 -
Flags: review?(rcampbell)
Assignee | ||
Comment 9•12 years ago
|
||
Pushed to try: https://tbpl.mozilla.org/?tree=Try&rev=feac638f8c6c
Comment 10•12 years ago
|
||
Comment on attachment 641954 [details] [diff] [review] proposed patch nice analysis of this problem. Sneaky bug! Thanks for working through this with us, Ian.
Attachment #641954 -
Flags: review?(rcampbell) → review+
Assignee | ||
Comment 11•12 years ago
|
||
Previous try push was orange due to an error in my test. Test fixed now. Another try push: https://tbpl.mozilla.org/?tree=Try&rev=a2833b1a84d6 (hopefully this one will be green) Thanks for the r+!
Attachment #641954 -
Attachment is obsolete: true
Assignee | ||
Comment 12•12 years ago
|
||
Comment on attachment 642910 [details] [diff] [review] [in-fx-team] test fix Landed: https://hg.mozilla.org/integration/fx-team/rev/fdb072732814
Attachment #642910 -
Attachment description: test fix → [in-fx-team] test fix
Assignee | ||
Updated•12 years ago
|
Whiteboard: [fixed-in-fx-team]
Comment 13•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/fdb072732814
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 17
Assignee | ||
Comment 14•12 years ago
|
||
Comment on attachment 642910 [details] [diff] [review] [in-fx-team] test fix [Approval Request Comment] Bug caused by (feature/regressing bug #): bug 673148 - I omitted special-cases like the one reported in this bug. User impact if declined: in certain cases when the dev tries to log objects that are not inspectable, the web console output breaks. Testing completed (on m-c, etc.): landed in fx-team, then m-c and had green try runs. Risk to taking this patch (and alternatives if risky): no known risk. String or UUID changes made by this patch: none.
Attachment #642910 -
Flags: approval-mozilla-aurora?
Assignee | ||
Comment 15•12 years ago
|
||
Firefox 16: affected in the same way as reported. Firefox 15: unaffected. console.dir(function(){}) still causes an error to show in the error console, but output continues to work - it does not stop displaying output. So, technically the bug still exists, but it has far less impact. It seems to me that the latest perf changes in how output works made the bug worse. I am not sure how to proceed for the beta. If it is considered we need to fix the bug for beta (firefox 15) as well, I can do that. Please let me know. Thank you!
Comment 16•12 years ago
|
||
I'm assuming you meant to mark FF15 as affected from your comment. FF15 is a developer-centric release, so I think it would be fantastic to prepare a patch for that branch as well. Approving for Aurora 16 in the meantime.
Updated•12 years ago
|
Attachment #642910 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 17•12 years ago
|
||
Comment on attachment 642910 [details] [diff] [review] [in-fx-team] test fix Indeed, I mixed version numbers. I meant that beta wasn't affected as badly as aurora and nightlies. Thanks for the aurora approval! Landed in aurora: https://hg.mozilla.org/releases/mozilla-aurora/rev/ad98fe0fb57e
Assignee | ||
Updated•12 years ago
|
Assignee | ||
Comment 18•12 years ago
|
||
Comment on attachment 642910 [details] [diff] [review] [in-fx-team] test fix [Approval Request Comment] Bug caused by (feature/regressing bug #): bug 673148 - I omitted special-cases like the one reported in this bug. User impact if declined: in certain cases when the dev tries to log objects that are not inspectable, an exception will occur in the web console. this is visible in the error console. output continues to work after that. Testing completed (on m-c, etc.): landed in fx-team, m-c, aurora and had green try runs. Risk to taking this patch (and alternatives if risky): no known risk. String or UUID changes made by this patch: none.
Attachment #642910 -
Flags: approval-mozilla-beta?
Comment 19•12 years ago
|
||
Comment on attachment 642910 [details] [diff] [review] [in-fx-team] test fix Approving for Beta since we're very early in the cycle. Please land in time for beta 2 (before Tues. AM PT) and then verify with the subsequent build so we can be sure that the Beta (15) landing accomplished what was intended.
Attachment #642910 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Assignee | ||
Comment 20•12 years ago
|
||
Comment on attachment 642910 [details] [diff] [review] [in-fx-team] test fix Thanks for the beta approval! Landed: https://hg.mozilla.org/releases/mozilla-beta/rev/952b175db4ec Please let me know if further action is needed on my behalf.
Assignee | ||
Updated•12 years ago
|
Comment 21•12 years ago
|
||
Reproduced the issue on Nightly 2012-07-17. (In reply to Ian Bicking (:ianb) from comment #0) > I'm having a hard time figuring out when this happens exactly, but sometimes > the console stops updating with new content. Opening and closing the > console (e.g., with Cmd-Alt-K) sometimes fixes it, or at least causes some > output to appear that was hidden. > > This seems to reproduce: > > > console.dir(function () {}); > > console.log('hey'); > > The "hey" won't display. console.dir({a: 1}) won't cause this, but trying > to use it on some other objects (besides simple objects) also causes > problems. "hey" displayed (In reply to Paul Rouget [:paul] from comment #1) > I think it actually prints "hey", but doesn't scroll up the content. > So you have to scroll to reach the "hey" string. But, when the mouse > is over the dir-box, you can't scroll, for some reason. > > So 2 problems here: > 1) we should ensure that the second log is visible by scrolling to the end > of the rhichlistbox. I think we already do that, but for some reason, it > doesn't work with the dir. > 2) we can't scroll the richlistbox when the mouse is over the dir-box > > My STR: > > console.dir(document.body) > > console.log("foo") "foo" displayed Verified fixed on FF 16b4 on Win 7 x64, Ubuntu 12.04 and Mac OS X 10.7.4
Comment 22•12 years ago
|
||
Verified fixed on FF 17b1 on Win 7 x64, Ubuntu 12.04 and Mac OS X 10.6.8
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•