Bug 56806

Summary: Web Inspector: protocol cleanup. ConsoleAgent.
Product: WebKit Reporter: Ilya Tikhonovsky <loislo>
Component: Web Inspector (Deprecated)Assignee: Ilya Tikhonovsky <loislo>
Status: RESOLVED FIXED    
Severity: Normal CC: apavlov, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 54856    
Attachments:
Description Flags
[patch] initial version yurys: review+

Description Ilya Tikhonovsky 2011-03-22 00:31:20 PDT 
see protocol metabug:
- All events should use notification wording, not command wording: addConsoleMessage->consoleMessage, updateConsoleMessageRepeatCount->consoleMessageRepeatCountUpdates, etc.


rename: void setConsoleMessagesEnabled(in boolean enabled, out boolean newState); => void enable(out long expiredMessagesCount);
new method: disable()
deleted method: [event] void consoleMessageExpiredCountUpdate(out unsigned long count); delete it. The counter will be transfered as out argument of enable command.
Comment 1 Ilya Tikhonovsky 2011-03-22 00:36:58 PDT 
Created attachment 86435 [details]
[patch] initial version
Comment 2 Yury Semikhatsky 2011-03-22 02:13:08 PDT 
Comment on attachment 86435 [details]
[patch] initial version

View in context: https://bugs.webkit.org/attachment.cgi?id=86435&action=review

> Source/WebCore/inspector/InspectorConsoleAgent.cpp:88
> +    unsigned messageCount = m_consoleMessages.size();

unsigned -> size_t ?
Comment 3 Ilya Tikhonovsky 2011-03-22 02:22:15 PDT 
Committed r81650: <http://trac.webkit.org/changeset/81650>