Remove RefPtr<ScriptState> from PerformanceObserver

third_party/WebKit/Source/bindings/core/v8/custom/V8PerformanceObserverCustom.cpp

Index: third_party/WebKit/Source/bindings/core/v8/custom/V8PerformanceObserverCustom.cpp
diff --git a/third_party/WebKit/Source/bindings/core/v8/custom/V8PerformanceObserverCustom.cpp b/third_party/WebKit/Source/bindings/core/v8/custom/V8PerformanceObserverCustom.cpp
index 11d65aafbdcbe4a66acf0ca522ec966feed79bbc..c3e8deacc3036225d608e15d9f320864826865fa 100644
--- a/third_party/WebKit/Source/bindings/core/v8/custom/V8PerformanceObserverCustom.cpp
+++ b/third_party/WebKit/Source/bindings/core/v8/custom/V8PerformanceObserverCustom.cpp
@@ -54,8 +54,8 @@ void V8PerformanceObserver::constructorCustom(
   PerformanceObserverCallback* callback =
       PerformanceObserverCallback::create(scriptState, v8Callback);
 
-  PerformanceObserver* observer =
-      PerformanceObserver::create(scriptState, performance, callback);
+  PerformanceObserver* observer = PerformanceObserver::create(
+      currentExecutionContext(info.GetIsolate()), performance, callback);

[Yuki, 2016/12/02 07:07:01]

Why don't you use scriptState->getExecutionContext()?

In the constructor (i.e. constructorCustom), the current realm must equal to the
relevant realm of the receiver object, so your code should be okay.  However, I
don't want to mix them.

We've already using scriptState for PerformanceObserverCallback::create() on
line 54-55, so using scriptState->getExecutionContext() is consistent, I think.

 
   // TODO(bashi): Don't set private property (and remove this custom
   // constructor) when we can call setWrapperReference() correctly.