[Webkit-unassigned] [Bug 20919] Constrain the number of messages the inspector shows

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=20919


Darin Adler <darin at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #45242|review?                     |review-
               Flag|                            |




--- Comment #5 from Darin Adler <darin at apple.com>  2009-12-19 11:29:47 PST ---
(From update of attachment 45242)
> +        m_expiredConsoleMessageCount += 100;
> +        for (size_t i = 0; i < 100; ++i)
> +            delete m_consoleMessages[i];
> +        m_consoleMessages.remove(0, 100);

Should use a named constant for 100 instead of repeating it 3 times. In fact, I
think it would be good to put that constant up at the top of the file along
with maximumConsoleMessages.

> +void InspectorFrontend::updateConsoleMessageExpiredCount(const int count)

The const here should be left out. All it does is prevent the function from
modifying its local copy of the argument internally. This is not something we
should do in WebKit code.

> +        void updateConsoleMessageExpiredCount(const int count);

Ditto.

review- so you can fix these minor quibbles

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.