DevTools: fix timer leak.

webkit/glue/devtools/js/profiler_processor.js

 // Copyright (c) 2009 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.
 
 /**
  * @fileoverview Profiler processor is used to process log file produced
  * by V8 and produce an internal profile representation which is used
  * for building profile views in 'Profiles' tab.
  */
 goog.provide('devtools.profiler.Processor');
@@ skipping 210 matching lines @@
    */
   this.profileId_ = 1;
 
   /**
    * Counter for processed ticks.
    * @type {number}
    */
   this.ticksCount_ = 0;
 
   /**
+   * Interval id for updating processing status.
+   * @type {number}
+   */
+  this.processingInterval_ = null;
+
+  /**
    * The current heap snapshot.
    * @type {string}
    */
   this.currentHeapSnapshot_ = null;
 
   /**
    * Next heap snapshot id.
    * @type {number}
    */
   this.heapSnapshotId_ = 1;
@@ skipping 54 matching lines @@
  * @param {function(devtools.profiler.ProfileView)} callback Callback function.
  */
 devtools.profiler.Processor.prototype.setNewProfileCallback = function(
     callback) {
   this.newProfileCallback_ = callback;
 };
 
 
 devtools.profiler.Processor.prototype.processProfiler_ = function(
     state, params) {
-  var processingInterval = null;
   switch (state) {
     case 'resume':
       if (this.currentProfile_ == null) {
         this.currentProfile_ = new devtools.profiler.JsProfile();
         // see the comment for devtools.profiler.Processor.PROGRAM_ENTRY
         this.currentProfile_.addCode(
           'Function', '(program)',
           devtools.profiler.Processor.PROGRAM_ENTRY, 1);
         if (this.startedProfileProcessing_) {
           this.startedProfileProcessing_();
         }
         this.ticksCount_ = 0;
         var self = this;
         if (this.profileProcessingStatus_) {
-          processingInterval = window.setInterval(
+          this.processingInterval_ = window.setInterval(
               function() { self.profileProcessingStatus_(self.ticksCount_); },
               1000);
         }
       }
       break;
     case 'pause':
       if (this.currentProfile_ != null) {
-        window.clearInterval(processingInterval);
+        window.clearInterval(this.processingInterval_);
+        this.processingInterval_ = null;
         if (this.finishedProfileProcessing_) {
           this.finishedProfileProcessing_(this.createProfileForView());
         }
         this.currentProfile_ = null;
       }
       break;
     case 'begin':
       var samplingRate = NaN;
       if (params.length > 0) {
         samplingRate = parseInt(params[0]);
@@ skipping 130 matching lines @@
 /**
  * Creates a profile for further displaying in ProfileView.
  */
 devtools.profiler.Processor.prototype.createProfileForView = function() {
   var profile = this.viewBuilder_.buildView(
       this.currentProfile_.getTopDownProfile());
   profile.uid = this.profileId_++;
   profile.title = UserInitiatedProfileName + '.' + profile.uid;
   return profile;
 };