|Summary:||Web Inspector: Should support console.groupCollapsed|
|Product:||WebKit||Reporter:||Keishi Hattori <keishi>|
|Component:||Web Inspector (Deprecated)||Assignee:||Jessie Berlin <jberlin>|
|Severity:||Normal||CC:||abarth, bweinstein, eric, jberlin, joepeck, keishi, ossy, pfeldman, pmuellr, rik, timothy, webkit.review.bot|
|Version:||528+ (Nightly build)|
Description Keishi Hattori 2009-11-10 08:11:38 PST
From Firebug documentation console.groupCollapsed(object[, object, ...]) Like console.group(), but the block is initially collapsed.
Comment 1 Jessie Berlin 2010-05-19 12:46:22 PDT
Created attachment 56512 [details] Proposed patch to add console.groupCollapsed
Comment 2 Pavel Feldman 2010-05-24 12:27:35 PDT
Comment on attachment 56512 [details] Proposed patch to add console.groupCollapsed Looks good. Thanks for putting it together!
Comment 3 Jessie Berlin 2010-05-26 06:46:07 PDT
Comment on attachment 56512 [details] Proposed patch to add console.groupCollapsed Thanks for the review :) Committed in r60234: http://trac.webkit.org/changeset/60234
Comment 4 WebKit Review Bot 2010-05-26 07:27:35 PDT
http://trac.webkit.org/changeset/60234 might have broken Qt Linux Release
Comment 5 Jessie Berlin 2010-05-26 08:18:33 PDT
(In reply to comment #4) > http://trac.webkit.org/changeset/60234 might have broken Qt Linux Release It doesn't appear that the Qt Linux Release is broken. There appear to be some compiler warnings that cropped up with this change (though they look unrelated, so I am not sure how that happened and would appreciate if someone could explain it to me), but they disappear after the 60235 revision.
Comment 6 Timothy Hatcher 2010-05-26 09:21:27 PDT
The WebKit Review Bot always seems to bitch about the Qt bot for innocuous inspector changes. It is getting annoying…
Comment 7 Eric Seidel (no email) 2010-05-26 12:34:14 PDT
We should change the name to SherriffBot, because that's what the WebKit Review Bot is. He's bitching because the Qt bots keep losing their network conneciton mid-build and failing builds. It's happening frequently for all builds. We may just have to remove the Qt builders from the core set until they can get a stable network.
Comment 8 Csaba Osztrogonác 2010-05-26 15:29:22 PDT
Many apologies for this false positive alarm. It was a flakey test I skipped after this fail: http://trac.webkit.org/changeset/60236
Comment 9 Patrick Mueller 2010-05-28 07:43:15 PDT
I noticed that the MessageType enum in Console.h does not match WebInspector.ConsoleMessage.MessageType in ConsoleView.js, and the mis-match seems to happen with this bug fix. Are we still keeping these aligned? It appears to me that we should be keeping them aligned. Since I'm adding a new message type for Bug 38664, I thought I'd go ahead fix MessageType in ConsoleView.js in that patch. Since no one uses the Result MessageType from the C++ code, there's no problem with the current code, today, but it may cause problems down the road if we don't keep them aligned.