Bug 121336 – Replace ScriptState with JSC::ExecState

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Sam Weinig]

Replace ScriptState with JSC::ExecState

[Sam Weinig]

Created attachment 211609 [details]
Patch

[WebKit Commit Bot]

Attachment 211609 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCore/Modules/indexeddb/IDBCursor.h', u'Source/WebCore/Modules/indexeddb/IDBObjectStore.h', u'Source/WebCore/bindings/js/JSInjectedScriptHostCustom.cpp', u'Source/WebCore/bindings/js/JSInjectedScriptManager.cpp', u'Source/WebCore/bindings/js/JSNodeCustom.cpp', u'Source/WebCore/bindings/js/JSNodeFilterCondition.h', u'Source/WebCore/bindings/js/ScriptController.cpp', u'Source/WebCore/bindings/js/ScriptDebugServer.cpp', u'Source/WebCore/bindings/js/ScriptDebugServer.h', u'Source/WebCore/bindings/js/ScriptEventListener.cpp', u'Source/WebCore/bindings/js/ScriptEventListener.h', u'Source/WebCore/bindings/js/ScriptFunctionCall.cpp', u'Source/WebCore/bindings/js/ScriptFunctionCall.h', u'Source/WebCore/bindings/js/ScriptObject.cpp', u'Source/WebCore/bindings/js/ScriptObject.h', u'Source/WebCore/bindings/js/ScriptProfiler.cpp', u'Source/WebCore/bindings/js/ScriptProfiler.h', u'Source/WebCore/bindings/js/ScriptState.cpp', u'Source/WebCore/bindings/js/ScriptState.h', u'Source/WebCore/bindings/js/ScriptValue.cpp', u'Source/WebCore/bindings/js/ScriptValue.h', u'Source/WebCore/bindings/js/SerializedScriptValue.cpp', u'Source/WebCore/bindings/js/SerializedScriptValue.h', u'Source/WebCore/dom/Document.cpp', u'Source/WebCore/dom/Document.h', u'Source/WebCore/dom/NodeFilter.cpp', u'Source/WebCore/dom/NodeFilter.h', u'Source/WebCore/dom/NodeFilterCondition.cpp', u'Source/WebCore/dom/NodeFilterCondition.h', u'Source/WebCore/dom/NodeIterator.cpp', u'Source/WebCore/dom/NodeIterator.h', u'Source/WebCore/dom/ScriptExecutionContext.cpp', u'Source/WebCore/dom/ScriptExecutionContext.h', u'Source/WebCore/dom/Traversal.cpp', u'Source/WebCore/dom/Traversal.h', u'Source/WebCore/dom/TreeWalker.cpp', u'Source/WebCore/dom/TreeWalker.h', u'Source/WebCore/inspector/ConsoleMessage.cpp', u'Source/WebCore/inspector/ConsoleMessage.h', u'Source/WebCore/inspector/InjectedScript.h', u'Source/WebCore/inspector/InjectedScriptBase.cpp', u'Source/WebCore/inspector/InjectedScriptBase.h', u'Source/WebCore/inspector/InjectedScriptCanvasModule.cpp', u'Source/WebCore/inspector/InjectedScriptCanvasModule.h', u'Source/WebCore/inspector/InjectedScriptHost.cpp', u'Source/WebCore/inspector/InjectedScriptHost.h', u'Source/WebCore/inspector/InjectedScriptManager.cpp', u'Source/WebCore/inspector/InjectedScriptManager.h', u'Source/WebCore/inspector/InjectedScriptModule.cpp', u'Source/WebCore/inspector/InjectedScriptModule.h', u'Source/WebCore/inspector/InspectorAgent.cpp', u'Source/WebCore/inspector/InspectorCanvasAgent.cpp', u'Source/WebCore/inspector/InspectorCanvasAgent.h', u'Source/WebCore/inspector/InspectorConsoleAgent.cpp', u'Source/WebCore/inspector/InspectorConsoleAgent.h', u'Source/WebCore/inspector/InspectorConsoleInstrumentation.h', u'Source/WebCore/inspector/InspectorDOMAgent.cpp', u'Source/WebCore/inspector/InspectorDebuggerAgent.cpp', u'Source/WebCore/inspector/InspectorDebuggerAgent.h', u'Source/WebCore/inspector/InspectorFrontendClientLocal.cpp', u'Source/WebCore/inspector/InspectorFrontendHost.cpp', u'Source/WebCore/inspector/InspectorIndexedDBAgent.cpp', u'Source/WebCore/inspector/InspectorInstrumentation.cpp', u'Source/WebCore/inspector/InspectorInstrumentation.h', u'Source/WebCore/inspector/PageConsoleAgent.cpp', u'Source/WebCore/inspector/PageDebuggerAgent.cpp', u'Source/WebCore/inspector/PageRuntimeAgent.cpp', u'Source/WebCore/inspector/PageRuntimeAgent.h', u'Source/WebCore/inspector/ScriptArguments.cpp', u'Source/WebCore/inspector/ScriptArguments.h', u'Source/WebCore/inspector/ScriptDebugListener.h', u'Source/WebCore/inspector/WorkerDebuggerAgent.cpp', u'Source/WebCore/inspector/WorkerRuntimeAgent.cpp', u'Source/WebCore/page/Console.cpp', u'Source/WebCore/page/Console.h', u'Source/WebCore/page/ContentSecurityPolicy.cpp', u'Source/WebCore/page/ContentSecurityPolicy.h', u'Source/WebCore/page/PageConsole.cpp', u'Source/WebCore/page/PageConsole.h', u'Source/WebCore/workers/WorkerGlobalScope.cpp', u'Source/WebCore/workers/WorkerGlobalScope.h']" exit_code: 1
Source/WebCore/bindings/js/ScriptProfiler.h:62:  The parameter name "state" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/bindings/js/ScriptProfiler.h:67:  The parameter name "state" adds no information, so it should be removed.  [readability/parameter_name] [5]
Total errors found: 2 in 82 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Anders Carlsson]

Comment on attachment 211609 [details]
Patch

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

>> Source/WebCore/bindings/js/ScriptProfiler.h:62
>> +    static void start(JSC::ExecState* state, const String& title);
> 
> The parameter name "state" adds no information, so it should be removed.  [readability/parameter_name] [5]

Yes please.

>> Source/WebCore/bindings/js/ScriptProfiler.h:67
>> +    static PassRefPtr<ScriptProfile> stop(JSC::ExecState* state, const String& title);
> 
> The parameter name "state" adds no information, so it should be removed.  [readability/parameter_name] [5]

Ditto.

[Sam Weinig]

Created attachment 211610 [details]
Patch

[Darin Adler]

Comment on attachment 211610 [details]
Patch

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

> Source/WebCore/dom/Document.cpp:862
> +PassRefPtr<CustomElementConstructor> Document::registerElement(WebCore::JSC::ExecState* state, const AtomicString& name, ExceptionCode& ec)

WebCore::JSC? No!

> Source/WebCore/dom/Document.cpp:867
> +PassRefPtr<CustomElementConstructor> Document::registerElement(WebCore::JSC::ExecState* state, const AtomicString& name, const Dictionary& options, ExceptionCode& ec)

Ditto.

> Source/WebCore/dom/Document.h:1142
> +    PassRefPtr<CustomElementConstructor> registerElement(WebCore::JSC::ExecState*, const AtomicString& name, ExceptionCode&);
> +    PassRefPtr<CustomElementConstructor> registerElement(WebCore::JSC::ExecState*, const AtomicString& name, const Dictionary& options, ExceptionCode&);

WebCore::JSC? No!

[Sam Weinig]

(In reply to comment #5)
> (From update of attachment 211610 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=211610&action=review
> 
> > Source/WebCore/dom/Document.cpp:862
> > +PassRefPtr<CustomElementConstructor> Document::registerElement(WebCore::JSC::ExecState* state, const AtomicString& name, ExceptionCode& ec)
> 
> WebCore::JSC? No!
> 
> > Source/WebCore/dom/Document.cpp:867
> > +PassRefPtr<CustomElementConstructor> Document::registerElement(WebCore::JSC::ExecState* state, const AtomicString& name, const Dictionary& options, ExceptionCode& ec)
> 
> Ditto.
> 
> > Source/WebCore/dom/Document.h:1142
> > +    PassRefPtr<CustomElementConstructor> registerElement(WebCore::JSC::ExecState*, const AtomicString& name, ExceptionCode&);
> > +    PassRefPtr<CustomElementConstructor> registerElement(WebCore::JSC::ExecState*, const AtomicString& name, const Dictionary& options, ExceptionCode&);
> 
> WebCore::JSC? No!

I wonder if anyone compiles with ENABLE(CUSTOM_ELEMENTS) on.

[Sam Weinig]

Committed r155736: <http://trac.webkit.org/changeset/155736>