Add probe for V8.Complie

third_party/WebKit/Source/bindings/core/v8/V8ScriptRunnerTest.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "bindings/core/v8/V8ScriptRunner.h"
 
 #include "bindings/core/v8/V8BindingForCore.h"
 #include "bindings/core/v8/V8BindingForTesting.h"
 #include "core/loader/resource/ScriptResource.h"
 #include "platform/heap/Handle.h"
@@ skipping 42 matching lines @@
   }
   unsigned TagForCodeCache(CachedMetadataHandler* cache_handler) const {
     return V8ScriptRunner::TagForCodeCache(cache_handler);
   }
   void SetCacheTimeStamp(CachedMetadataHandler* cache_handler) {
     V8ScriptRunner::SetCacheTimeStamp(cache_handler);
   }
 
   bool CompileScript(v8::Isolate* isolate, V8CacheOptions cache_options) {
     return !V8ScriptRunner::CompileScript(
+                ScriptState::Current(isolate),

[haraken, 2017/07/19 05:51:19]

Using ScriptState::Current is not nice unless you're pretty sure that you're in
a correct ScriptState.

Alternately can you pass in the ScriptState to CompileScript?

[Liquan (Max) Gu, 2017/07/19 15:04:44]

Done.

+
                 V8String(isolate, Code()), Filename(), String(),
-                WTF::TextPosition(), isolate, resource_.Get(), nullptr,
+                WTF::TextPosition(), resource_.Get(), nullptr,
                 resource_.Get() ? resource_->CacheHandler() : nullptr,
                 kNotSharableCrossOrigin, cache_options)
                 .IsEmpty();
   }
 
   void SetEmptyResource() {
     resource_ = ScriptResource::CreateForTest(NullURL(), UTF8Encoding());
   }
 
   void SetResource() {
@@ skipping 51 matching lines @@
   // The cached data is associated with the encoding.
   ScriptResource* another_resource =
       ScriptResource::CreateForTest(Url(), UTF16LittleEndianEncoding());
   EXPECT_FALSE(CacheHandler()->GetCachedMetadata(
       TagForCodeCache(another_resource->CacheHandler())));
 }
 
 }  // namespace
 
 }  // namespace blink