Style cleaning for WorkerContextExecutionProxy.
Created attachment 34092 [details] Proposed Patch
Comment on attachment 34092 [details] Proposed Patch More ChangeLog information would help me understand what all you're changing here.
Created attachment 34116 [details] Add info to ChangeLog
Comment on attachment 34116 [details] Add info to ChangeLog This looks good overall. A couple of comments: > diff --git a/WebCore/bindings/v8/ScheduledAction.cpp b/WebCore/bindings/v8/ScheduledAction.cpp > index 298f6b1..ce4ecf1 100644 > --- a/WebCore/bindings/v8/ScheduledAction.cpp > +++ b/WebCore/bindings/v8/ScheduledAction.cpp > @@ -134,7 +134,7 @@ void ScheduledAction::execute(WorkerContext* workerContext) > > if (!m_function.IsEmpty() && m_function->IsFunction()) { > v8::HandleScope handleScope; > - v8::Local<v8::Context> v8Context = scriptController->proxy()->GetContext(); > + v8::Local<v8::Context> v8Context = scriptController->proxy()->context(); Be careful, abarth is working on change to remove V8Proxy::context(), talk w/him. > +v8::Local<v8::Function> V8DOMWrapper::getConstructor(V8ClassIndex::V8WrapperType type, v8::Handle<v8::Context> context) > +{ > + // Enter the scope for this context to get the correct constructor. > + v8::Context::Scope scope(context); > + > + return getConstructor(type, V8Proxy::getHiddenObjectPrototype(context)); > +} Can we have a better, more specific name for this method and perhaps make it private? It shouldn't be used outside of this class.
(In reply to comment #4) > > - v8::Local<v8::Context> v8Context = scriptController->proxy()->GetContext(); > > + v8::Local<v8::Context> v8Context = scriptController->proxy()->context(); V8Proxy::context() must die. It's not worth renaming it at this point.
Created attachment 34221 [details] Proposed Patch
(In reply to comment #4) > (From update of attachment 34116 [details]) > This looks good overall. A couple of comments: > > > diff --git a/WebCore/bindings/v8/ScheduledAction.cpp b/WebCore/bindings/v8/ScheduledAction.cpp > > index 298f6b1..ce4ecf1 100644 > > --- a/WebCore/bindings/v8/ScheduledAction.cpp > > +++ b/WebCore/bindings/v8/ScheduledAction.cpp > > @@ -134,7 +134,7 @@ void ScheduledAction::execute(WorkerContext* workerContext) > > > > if (!m_function.IsEmpty() && m_function->IsFunction()) { > > v8::HandleScope handleScope; > > - v8::Local<v8::Context> v8Context = scriptController->proxy()->GetContext(); > > + v8::Local<v8::Context> v8Context = scriptController->proxy()->context(); > > Be careful, abarth is working on change to remove V8Proxy::context(), talk > w/him. > Per discussion, we keep it as context() since this is for WorkerContextExecutionProxy. > > > +v8::Local<v8::Function> V8DOMWrapper::getConstructor(V8ClassIndex::V8WrapperType type, v8::Handle<v8::Context> context) > > +{ > > + // Enter the scope for this context to get the correct constructor. > > + v8::Context::Scope scope(context); > > + > > + return getConstructor(type, V8Proxy::getHiddenObjectPrototype(context)); > > +} > > Can we have a better, more specific name for this method and perhaps make it > private? It shouldn't be used outside of this class. Fixed.
Comment on attachment 34221 [details] Proposed Patch r=me.
Committed as http://trac.webkit.org/changeset/46859.