Reviewers: Vyacheslav Egorov,

Description:
Revert inadvertent changes to the spinning balls tool.

Please review this at http://codereview.chromium.org/8230010/

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

Affected files:
  M     benchmarks/spinning-balls/v.js


Index: benchmarks/spinning-balls/v.js
===================================================================
--- benchmarks/spinning-balls/v.js      (revision 9578)
+++ benchmarks/spinning-balls/v.js      (working copy)
@@ -42,9 +42,6 @@
   } )();
 }

-
-
-
 var kNPoints = 8000;
 var kNModifications = 20;
 var kNVisiblePoints = 200;
@@ -57,23 +54,11 @@
 var dyingPoints = void 0;
 var scene = void 0;
 var renderingStartTime = void 0;
-var omitFrameDueToVisibilityChange = false;
 var scene = void 0;
 var pausePlot = void 0;
 var splayTree = void 0;

-handleVisibilityChange();

-function handleVisibilityChange() {
-  console.log("visibility change");
-  omitFrameDueToVisibilityChange = true;
- document.addEventListener("webkitvisibilitychange", handleVisibilityChange);
-  document.addEventListener("msvisibilitychange", handleVisibilityChange);
-  document.addEventListener("mozvisibilitychange", handleVisibilityChange);
-  document.addEventListener("visibilitychange", handleVisibilityChange);
-}
-
-
 function Point(x, y, z, payload) {
   this.x = x;
   this.y = y;
@@ -368,12 +353,9 @@
   scene.draw();

   var renderingEndTime = Date.now();
-  if (!omitFrameDueToVisibilityChange) {
-    var pause = renderingEndTime - renderingStartTime;
-    pausePlot.addPause(pause);
-  }
+  var pause = renderingEndTime - renderingStartTime;
+  pausePlot.addPause(pause);
   renderingStartTime = renderingEndTime;
-  omitFrameDueToVisibilityChange = false;

   pausePlot.draw();



--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to