Fixed some review comments.

Author: hzr

src/profiler/profiler_view.js

@@ -174,7 +174,6 @@ var ProfilerView = function(id, name, container_class, html, default_handler)
     var aggregated_event_list = this._get_aggregated_event_list(event_list);
     var width = this._container.clientWidth - AGGREGATED_EVENTS_WIDTH;
 
-    // TODO: Check if these are already appended
     var frag = document.createDocumentFragment();
     frag.appendChild(this._legend_ele);
     frag.appendChild(this._zoomer_times_ele);
@@ -195,8 +194,6 @@ var ProfilerView = function(id, name, container_class, html, default_handler)
     this._zoomer.set_zoomer_element(this._zoomer_ele);
     if (this._set_zoomer)
       this._zoomer.set_current_area();
-
-    // TODO: if _set_zoomer == false, reset()
   };
 
   this.createView = function(container)

src/ui-scripts/zoomer/zoomer.js

@@ -1,7 +1,5 @@
 "use strict";
 
-// TODO: when width is 1px and mousewheeling, don't move
-
 /**
  * @constructor
  */
@@ -12,6 +10,9 @@ var Zoomer = function(model, zoomer_ele)
 
 var ZoomerPrototype = function()
 {
+  var MOUSEWHEEL_DIFF = 5;
+  var KEYDOWN_DIFF = 1;
+
   this._init = function(model, zoomer_ele)
   {
     /**
@@ -151,7 +152,7 @@ var ZoomerPrototype = function()
   this._zoomer_ele_onmousewheel = function(event)
   {
     var mouse_x = event.clientX - this._zoomer_ele_left;
-    var diff = (mouse_x < this._overlay_left) ? 5 : -5;
+    var diff = (mouse_x < this._overlay_left) ? MOUSEWHEEL_DIFF : -MOUSEWHEEL_DIFF;
     diff *= (event.wheelDelta > 0) ? 1 : -1;
     this.move_overlay(diff);
     event.stopPropagation();
@@ -197,7 +198,7 @@ var ZoomerPrototype = function()
 
   this._overlay_ele_onmousewheel = function(event)
   {
-    var diff = (event.wheelDelta < 0) ? 5 : -5;
+    var diff = (event.wheelDelta < 0) ? MOUSEWHEEL_DIFF : -MOUSEWHEEL_DIFF;
     this.change_overlay_size(diff, -diff);
     event.stopPropagation();
   };
@@ -208,8 +209,8 @@ var ZoomerPrototype = function()
     var diff = {
       33: width,  // Page up
       34: -width, // Page down
-      37: -1,     // Arrow left
-      39: 1       // Arrow right
+      37: -KEYDOWN_DIFF, // Arrow left
+      39: KEYDOWN_DIFF   // Arrow right
     }[event.which];
     if (!diff)
       return;
@@ -279,8 +280,8 @@ var ZoomerPrototype = function()
   this._handle_ele_onkeydown = function(event)
   {
     var diff = {
-      37: -1, // Arrow left
-      39: 1   // Arrow right
+      37: -KEYDOWN_DIFF, // Arrow left
+      39: KEYDOWN_DIFF   // Arrow right
     }[event.which];
     if (!diff)
       return;