Delete several deprecated methods on v8::CpuProfiler

test/cctest/profiler-extension.cc

 // Copyright 2013 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
 //       with the distribution.
 //     * Neither the name of Google Inc. nor the names of its
 //       contributors may be used to endorse or promote products derived
 //       from this software without specific prior written permission.
 //
 // THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
 // "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
 // LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
 // A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
 // OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
 // SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
 // LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
 // DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
 // THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+//
+// Tests of profiles generator and utilities.
 
-#include "trigger-failure-extension.h"
-#include "v8.h"
+#include "profiler-extension.h"
 
-namespace v8 {
-namespace internal {
+#include "cctest.h"
 
+const v8::CpuProfile* ProfilerExtension::last_profile = NULL;
+const char* ProfilerExtension::kSource =
+    "native function startProfiling();"
+    "native function stopProfiling();";
 
-const char* const TriggerFailureExtension::kSource =
-    "native function triggerCheckFalse();"
-    "native function triggerAssertFalse();"
-    "native function triggerSlowAssertFalse();";
-
-
-v8::Handle<v8::FunctionTemplate>
-TriggerFailureExtension::GetNativeFunctionTemplate(
-    v8::Isolate* isolate,
-    v8::Handle<v8::String> str) {
-  if (strcmp(*v8::String::Utf8Value(str), "triggerCheckFalse") == 0) {
-    return v8::FunctionTemplate::New(
-        TriggerFailureExtension::TriggerCheckFalse);
-  } else if (strcmp(*v8::String::Utf8Value(str), "triggerAssertFalse") == 0) {
-    return v8::FunctionTemplate::New(
-        TriggerFailureExtension::TriggerAssertFalse);
+v8::Handle<v8::FunctionTemplate> ProfilerExtension::GetNativeFunctionTemplate(
+    v8::Isolate* isolate, v8::Handle<v8::String> name) {
+  if (name->Equals(v8::String::NewFromUtf8(isolate, "startProfiling"))) {
+    return v8::FunctionTemplate::New(ProfilerExtension::StartProfiling);
+  } else if (name->Equals(v8::String::NewFromUtf8(isolate, "stopProfiling"))) {
+    return v8::FunctionTemplate::New(ProfilerExtension::StopProfiling);
   } else {
-    CHECK_EQ(0, strcmp(*v8::String::Utf8Value(str), "triggerSlowAssertFalse"));
-    return v8::FunctionTemplate::New(
-        TriggerFailureExtension::TriggerSlowAssertFalse);
+    CHECK(false);
+    return v8::Handle<v8::FunctionTemplate>();
   }
 }
 
 
-void TriggerFailureExtension::TriggerCheckFalse(
+void ProfilerExtension::StartProfiling(
     const v8::FunctionCallbackInfo<v8::Value>& args) {
-  CHECK(false);
+  last_profile = NULL;
+  v8::CpuProfiler* cpu_profiler = args.GetIsolate()->GetCpuProfiler();
+  if (args.Length() > 0)
+    cpu_profiler->StartCpuProfiling(args[0].As<v8::String>());
+  else
+    cpu_profiler->StartCpuProfiling(
+        v8::String::NewFromUtf8(args.GetIsolate(), ""));

[alph, 2013/12/17 16:20:17]

Will v8::String::Empty() work here?

[Sven Panne, 2013/12/18 06:48:21]

Heads up: Although this is not explicitly deprecated, we're about to remove
*all* deprecated stuff + some more things. Empty() without an Isolate* parameter
is about to die, too. In general, use an Isolate version if there is one.

[yurys, 2013/12/18 07:26:19]

Done.

[yurys, 2013/12/18 07:26:19]

Switched to the method with Isolate* parameter.

 }
 
 
-void TriggerFailureExtension::TriggerAssertFalse(
+void ProfilerExtension::StopProfiling(
     const v8::FunctionCallbackInfo<v8::Value>& args) {
-  ASSERT(false);
+  v8::CpuProfiler* cpu_profiler = args.GetIsolate()->GetCpuProfiler();
+  if (args.Length() > 0)
+    last_profile = cpu_profiler->StopCpuProfiling(args[0].As<v8::String>());
+  else
+    last_profile = cpu_profiler->StopCpuProfiling(
+        v8::String::NewFromUtf8(args.GetIsolate(), ""));

[alph, 2013/12/17 16:20:17]

ditto

[yurys, 2013/12/18 07:26:19]

Done.

 }
 
 
-void TriggerFailureExtension::TriggerSlowAssertFalse(
-    const v8::FunctionCallbackInfo<v8::Value>& args) {
-  SLOW_ASSERT(false);
-}
-
-
-void TriggerFailureExtension::Register() {
-  static TriggerFailureExtension trigger_failure_extension;
-  static v8::DeclareExtension declaration(&trigger_failure_extension);
-}
-
-} }  // namespace v8::internal
+static ProfilerExtension kProfilerExtension;
+v8::DeclareExtension kProfilerExtensionDeclaration(&kProfilerExtension);