Bug 31296 - Web Inspector: Should support console.groupCollapsed
Summary: Web Inspector: Should support console.groupCollapsed
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Jessie Berlin
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-11-10 08:11 PST by Keishi Hattori
Modified: 2010-05-28 07:43 PDT (History)
12 users (show)

See Also:


Attachments
Proposed patch to add console.groupCollapsed (9.95 KB, patch)
2010-05-19 12:46 PDT, Jessie Berlin
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.