Bug 44027 - Web Inspector: [REGRESSION] console's clear command does not work.
Summary: Web Inspector: [REGRESSION] console's clear command does not work.
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: Pavel Feldman
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-15 05:15 PDT by Pavel Feldman
Modified: 2010-08-16 05:17 PDT (History)
9 users (show)

See Also:


Attachments
[PATCH] Proposed fix. (7.93 KB, patch)
2010-08-15 05:27 PDT, Pavel Feldman
no flags Details | Formatted Diff | Diff
[PATCH] Same with proper test title. (7.23 KB, patch)
2010-08-15 05:34 PDT, Pavel Feldman
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pavel Feldman 2010-08-15 05:15:55 PDT
Patch to follow.
Comment 1 Pavel Feldman 2010-08-15 05:16:20 PDT
Regresed in r63891.
Comment 2 Pavel Feldman 2010-08-15 05:27:14 PDT
Created attachment 64443 [details]
[PATCH] Proposed fix.
Comment 3 Pavel Feldman 2010-08-15 05:34:32 PDT
Created attachment 64444 [details]
[PATCH] Same with proper test title.
Comment 4 Joseph Pecoraro 2010-08-15 10:46:26 PDT
Any particular reason we wait for a callback to clear the UI? I don't see a purpose
to the consoleMessagesCleared notification. I'd rather the UI clear instantly. Am
I missing something, or are we just being consistent and letting every backend
message have some kind of callback? Looks good to me, I just have this
question.
Comment 5 Pavel Feldman 2010-08-15 11:15:09 PDT
This bug should have stated more clearly that it is about clear() command in the console not working.

We initiate clear from injected script (backend) in case of command line api use. So we need to send a direct message to the frontend. Easy to forget - same happened to Yury. So I added a test.
Comment 6 Joseph Pecoraro 2010-08-15 13:11:23 PDT
Comment on attachment 64444 [details]
[PATCH] Same with proper test title.

(In reply to comment #5)
> This bug should have stated more clearly that it is about clear() command
> in the console not working.

My fault. Its entirely clear from the bug's title, I just overlooked it!


> We initiate clear from injected script (backend) in case of command line
> api use. So we need to send a direct message to the frontend. Easy to forget
> - same happened to Yury. So I added a test.

Haha, you're right its easy to forget =). Good fix and nice test.

r=me
Comment 7 Pavel Feldman 2010-08-16 05:16:53 PDT
Comment on attachment 64444 [details]
[PATCH] Same with proper test title.

Clearing flags on attachment: 64444

Committed r65412: <http://trac.webkit.org/changeset/65412>
Comment 8 Pavel Feldman 2010-08-16 05:17:14 PDT
All reviewed patches have been landed.  Closing bug.