Reviewers: yurys, loislo, Benedikt Meurer,

Description:
Make several cpu-profile tests more stable

10 samples they collected were not enough to ensure the expected
call tree is covered. Increase the samples count to 100.

Please review this at https://codereview.chromium.org/306443003/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files (+6, -6 lines):
  M test/cctest/test-cpu-profiler.cc


Index: test/cctest/test-cpu-profiler.cc
diff --git a/test/cctest/test-cpu-profiler.cc b/test/cctest/test-cpu-profiler.cc index ce422b5b6ba179f257743a2d169ee5b81c1ee97e..bdccd1ee1060281f382a42bdedbfe5d8ccdcddac 100644
--- a/test/cctest/test-cpu-profiler.cc
+++ b/test/cctest/test-cpu-profiler.cc
@@ -1301,12 +1301,12 @@ TEST(JsNativeJsSample) {
   v8::Local<v8::Function> function = v8::Local<v8::Function>::Cast(
env->Global()->Get(v8::String::NewFromUtf8(env->GetIsolate(), "start")));

-  int32_t duration_ms = 20;
+  int32_t duration_ms = 200;
   v8::Handle<v8::Value> args[] = {
     v8::Integer::New(env->GetIsolate(), duration_ms)
   };
   v8::CpuProfile* profile =
-      RunProfiler(env, function, args, ARRAY_SIZE(args), 10);
+      RunProfiler(env, function, args, ARRAY_SIZE(args), 100);

   const v8::CpuProfileNode* root = profile->GetTopDownRoot();
   {
@@ -1386,12 +1386,12 @@ TEST(JsNativeJsRuntimeJsSample) {
   v8::Local<v8::Function> function = v8::Local<v8::Function>::Cast(
env->Global()->Get(v8::String::NewFromUtf8(env->GetIsolate(), "start")));

-  int32_t duration_ms = 20;
+  int32_t duration_ms = 200;
   v8::Handle<v8::Value> args[] = {
     v8::Integer::New(env->GetIsolate(), duration_ms)
   };
   v8::CpuProfile* profile =
-      RunProfiler(env, function, args, ARRAY_SIZE(args), 10);
+      RunProfiler(env, function, args, ARRAY_SIZE(args), 100);

   const v8::CpuProfileNode* root = profile->GetTopDownRoot();
   ScopedVector<v8::Handle<v8::String> > names(3);
@@ -1484,12 +1484,12 @@ TEST(JsNative1JsNative2JsSample) {
   v8::Local<v8::Function> function = v8::Local<v8::Function>::Cast(
env->Global()->Get(v8::String::NewFromUtf8(env->GetIsolate(), "start")));

-  int32_t duration_ms = 20;
+  int32_t duration_ms = 200;
   v8::Handle<v8::Value> args[] = {
     v8::Integer::New(env->GetIsolate(), duration_ms)
   };
   v8::CpuProfile* profile =
-      RunProfiler(env, function, args, ARRAY_SIZE(args), 10);
+      RunProfiler(env, function, args, ARRAY_SIZE(args), 100);

   const v8::CpuProfileNode* root = profile->GetTopDownRoot();
   ScopedVector<v8::Handle<v8::String> > names(3);


--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to