Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(309)

Side by Side Diff: test/cctest/profiler-extension.cc

Issue 2117343006: Introduce v8::CpuProfiler::New and v8::CpuProfiler::Dispose API. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2013 the V8 project authors. All rights reserved. 1 // Copyright 2013 the V8 project authors. All rights reserved.
2 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
(...skipping 15 matching lines...) Expand all
26 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 26 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
27 // 27 //
28 // Tests of profiles generator and utilities. 28 // Tests of profiles generator and utilities.
29 29
30 #include "test/cctest/profiler-extension.h" 30 #include "test/cctest/profiler-extension.h"
31 #include "test/cctest/cctest.h" 31 #include "test/cctest/cctest.h"
32 32
33 namespace v8 { 33 namespace v8 {
34 namespace internal { 34 namespace internal {
35 35
36 v8::CpuProfile* ProfilerExtension::last_profile = NULL; 36 ProfilerExtension::Scope::Scope(v8::Isolate* isolate) {
37 DCHECK(!profiler_);
38 profiler_ = v8::CpuProfiler::New(isolate);
39 }
40
41 ProfilerExtension::Scope::~Scope() {
42 profiler_->Dispose();
43 profiler_ = nullptr;
44 }
45
46 v8::CpuProfiler* ProfilerExtension::Scope::profiler_ = nullptr;
47 v8::CpuProfile* ProfilerExtension::last_profile = nullptr;
37 const char* ProfilerExtension::kSource = 48 const char* ProfilerExtension::kSource =
38 "native function startProfiling();" 49 "native function startProfiling();"
39 "native function stopProfiling();" 50 "native function stopProfiling();"
40 "native function collectSample();"; 51 "native function collectSample();";
41 52
42 v8::Local<v8::FunctionTemplate> ProfilerExtension::GetNativeFunctionTemplate( 53 v8::Local<v8::FunctionTemplate> ProfilerExtension::GetNativeFunctionTemplate(
43 v8::Isolate* isolate, v8::Local<v8::String> name) { 54 v8::Isolate* isolate, v8::Local<v8::String> name) {
44 v8::Local<v8::Context> context = isolate->GetCurrentContext(); 55 v8::Local<v8::Context> context = isolate->GetCurrentContext();
45 if (name->Equals(context, v8_str(isolate, "startProfiling")).FromJust()) { 56 if (name->Equals(context, v8_str(isolate, "startProfiling")).FromJust()) {
46 return v8::FunctionTemplate::New(isolate, 57 return v8::FunctionTemplate::New(isolate,
47 ProfilerExtension::StartProfiling); 58 ProfilerExtension::StartProfiling);
48 } 59 }
49 if (name->Equals(context, v8_str(isolate, "stopProfiling")).FromJust()) { 60 if (name->Equals(context, v8_str(isolate, "stopProfiling")).FromJust()) {
50 return v8::FunctionTemplate::New(isolate, ProfilerExtension::StopProfiling); 61 return v8::FunctionTemplate::New(isolate, ProfilerExtension::StopProfiling);
51 } 62 }
52 if (name->Equals(context, v8_str(isolate, "collectSample")).FromJust()) { 63 if (name->Equals(context, v8_str(isolate, "collectSample")).FromJust()) {
53 return v8::FunctionTemplate::New(isolate, ProfilerExtension::CollectSample); 64 return v8::FunctionTemplate::New(isolate, ProfilerExtension::CollectSample);
54 } 65 }
55 CHECK(false); 66 CHECK(false);
56 return v8::Local<v8::FunctionTemplate>(); 67 return v8::Local<v8::FunctionTemplate>();
57 } 68 }
58 69
59 void ProfilerExtension::StartProfiling( 70 void ProfilerExtension::StartProfiling(
60 const v8::FunctionCallbackInfo<v8::Value>& args) { 71 const v8::FunctionCallbackInfo<v8::Value>& args) {
61 last_profile = NULL; 72 last_profile = nullptr;
62 v8::CpuProfiler* cpu_profiler = args.GetIsolate()->GetCpuProfiler(); 73 Scope::profiler_->StartProfiling((args.Length() > 0)
63 cpu_profiler->StartProfiling((args.Length() > 0) 74 ? args[0].As<v8::String>()
64 ? args[0].As<v8::String>() 75 : v8::String::Empty(args.GetIsolate()));
65 : v8::String::Empty(args.GetIsolate()));
66 } 76 }
67 77
68 void ProfilerExtension::StopProfiling( 78 void ProfilerExtension::StopProfiling(
69 const v8::FunctionCallbackInfo<v8::Value>& args) { 79 const v8::FunctionCallbackInfo<v8::Value>& args) {
70 v8::CpuProfiler* cpu_profiler = args.GetIsolate()->GetCpuProfiler(); 80 last_profile = Scope::profiler_->StopProfiling(
71 last_profile = cpu_profiler->StopProfiling((args.Length() > 0) 81 (args.Length() > 0) ? args[0].As<v8::String>()
72 ? args[0].As<v8::String>() 82 : v8::String::Empty(args.GetIsolate()));
73 : v8::String::Empty(args.GetIsolate()));
74 } 83 }
75 84
76 void ProfilerExtension::CollectSample( 85 void ProfilerExtension::CollectSample(
77 const v8::FunctionCallbackInfo<v8::Value>& args) { 86 const v8::FunctionCallbackInfo<v8::Value>& args) {
78 args.GetIsolate()->GetCpuProfiler()->CollectSample(); 87 Scope::profiler_->CollectSample();
79 } 88 }
80 89
81 } // namespace internal 90 } // namespace internal
82 } // namespace v8 91 } // namespace v8
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698