Bug 32897 – Web Inspector: [REGRESSION] console's clear command does not work.

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector (Deprecated)

[Pavel Feldman]

It turned out that console's clear command should be in InjectedScriptHost, not InspectorBackend binding.

[Pavel Feldman]

Created attachment 45427 [details]
[PATCH] Proposed fix

[WebKit Review Bot]

style-queue ran check-webkit-style on attachment 45427 [details] without any errors.

[Timothy Hatcher]

Comment on attachment 45427 [details]
[PATCH] Proposed fix

> +    requestClearConsoleMessages: function()
> +    clearConsoleMessages: function()

Having "Console" in these function names is redundant when they are on the ConsoleView object and always referenced in context.

[Pavel Feldman]

(In reply to comment #3)
> (From update of attachment 45427 [details])
> > +    requestClearConsoleMessages: function()
> > +    clearConsoleMessages: function()
> 
> Having "Console" in these function names is redundant when they are on the
> ConsoleView object and always referenced in context.

Done.

[Pavel Feldman]

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	LayoutTests/ChangeLog
	M	LayoutTests/inspector/console-tests.js
	M	WebCore/ChangeLog
	M	WebCore/inspector/InjectedScriptHost.cpp
	M	WebCore/inspector/InjectedScriptHost.h
	M	WebCore/inspector/InjectedScriptHost.idl
	M	WebCore/inspector/InspectorBackend.cpp
	M	WebCore/inspector/InspectorBackend.h
	M	WebCore/inspector/InspectorBackend.idl
	M	WebCore/inspector/InspectorController.cpp
	M	WebCore/inspector/InspectorController.h
	M	WebCore/inspector/front-end/ConsoleView.js
	M	WebCore/inspector/front-end/InjectedScript.js
	M	WebCore/inspector/front-end/InjectedScriptAccess.js
	M	WebCore/inspector/front-end/inspector.js
Committed r52545