Numbers should be "integer" or "number". No "unsigned" or "long" allowed.
Created attachment 86267 [details] Patch
Comment on attachment 86267 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=86267&action=review lgtm > Source/WebCore/inspector/InspectorConsoleAgent.cpp:238 > + int messageCount = m_consoleMessages.size(); > + for (int i = 0; i < messageCount; ++i) why did you change this? > Source/WebCore/inspector/InspectorConsoleAgent.h:97 > - unsigned m_expiredConsoleMessageCount; > - HashMap<String, unsigned> m_counts; > + int m_expiredConsoleMessageCount; > + HashMap<String, int> m_counts; ditto
Comment on attachment 86267 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=86267&action=review Please address comments before landing. > Source/WebCore/inspector/InspectorConsoleAgent.cpp:237 > + int messageCount = m_consoleMessages.size(); If you are changing type of the counter, it should become size_t, not int. >> Source/WebCore/inspector/InspectorConsoleAgent.h:97 >> + HashMap<String, int> m_counts; > > ditto I agree, I'd rather leave unsigned as value type here and cast if needed when formatting protocol messages.
Committed r81572: <http://trac.webkit.org/changeset/81572>