Bug 1350217 - Remove redundant shared module dependencies r?honza draft
authorRicky Chien <ricky060709@gmail.com>
Wed, 29 Mar 2017 15:21:25 +0800
changeset 553511 5357f2ed22b5281bd467edfdc3b97c68a39ab854
parent 553413 60d7a0496a3673450ddbc37ec387525148c32604
child 553512 167dab8655cc94493d4e6083212fb871660ebcf9
child 553517 4422f8307313d52a967b24a56703c02a28a1f4d6
child 553527 9e000f4d9a41ab7a8e27e55724a056578fd25b3a
child 553548 7d43d605895ee33e722f5b89b8345d04ea100f49
child 553550 f5a8f82bc74e7200661c181c3532cb624c1dfac3
push id51659
push userbmo:rchien@mozilla.com
push dateThu, 30 Mar 2017 06:27:45 +0000
reviewershonza
bugs1350217
milestone55.0a1
Bug 1350217 - Remove redundant shared module dependencies r?honza MozReview-Commit-ID: GhQn2RKfDgq
devtools/client/netmonitor/src/components/request-list-content.js
devtools/client/netmonitor/src/components/request-list-header.js
--- a/devtools/client/netmonitor/src/components/request-list-content.js
+++ b/devtools/client/netmonitor/src/components/request-list-content.js
@@ -1,15 +1,14 @@
 /* This Source Code Form is subject to the terms of the Mozilla Public
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 "use strict";
 
-const { KeyCodes } = require("devtools/client/shared/keycodes");
 const {
   createClass,
   createFactory,
   DOM,
   PropTypes,
 } = require("devtools/client/shared/vendor/react");
 const { connect } = require("devtools/client/shared/vendor/react-redux");
 const { HTMLTooltip } = require("devtools/client/shared/widgets/tooltip/HTMLTooltip");
@@ -171,46 +170,46 @@ const RequestListContent = createClass({
   onScroll() {
     this.tooltip.hide();
   },
 
   /**
    * Handler for keyboard events. For arrow up/down, page up/down, home/end,
    * move the selection up or down.
    */
-  onKeyDown(e) {
+  onKeyDown(evt) {
     let delta;
 
-    switch (e.keyCode) {
-      case KeyCodes.DOM_VK_UP:
-      case KeyCodes.DOM_VK_LEFT:
+    switch (evt.key) {
+      case "ArrowUp":
+      case "ArrowLeft":
         delta = -1;
         break;
-      case KeyCodes.DOM_VK_DOWN:
-      case KeyCodes.DOM_VK_RIGHT:
+      case "ArrowDown":
+      case "ArrowRight":
         delta = +1;
         break;
-      case KeyCodes.DOM_VK_PAGE_UP:
+      case "PageUp":
         delta = "PAGE_UP";
         break;
-      case KeyCodes.DOM_VK_PAGE_DOWN:
+      case "PageDown":
         delta = "PAGE_DOWN";
         break;
-      case KeyCodes.DOM_VK_HOME:
+      case "Home":
         delta = -Infinity;
         break;
-      case KeyCodes.DOM_VK_END:
+      case "End":
         delta = +Infinity;
         break;
     }
 
     if (delta) {
       // Prevent scrolling when pressing navigation keys.
-      e.preventDefault();
-      e.stopPropagation();
+      evt.preventDefault();
+      evt.stopPropagation();
       this.props.onSelectDelta(delta);
     }
   },
 
   onContextMenu(evt) {
     evt.preventDefault();
     this.contextMenu.open(evt);
   },
--- a/devtools/client/netmonitor/src/components/request-list-header.js
+++ b/devtools/client/netmonitor/src/components/request-list-header.js
@@ -5,17 +5,16 @@
 "use strict";
 
 const {
   createClass,
   PropTypes,
   DOM,
 } = require("devtools/client/shared/vendor/react");
 const { connect } = require("devtools/client/shared/vendor/react-redux");
-const { setNamedTimeout } = require("devtools/client/shared/widgets/view-helpers");
 const Actions = require("../actions/index");
 const { getWaterfallScale } = require("../selectors/index");
 const { getFormattedTime } = require("../utils/format-utils");
 const { L10N } = require("../utils/l10n");
 const WaterfallBackground = require("../waterfall-background");
 
 const { div, button } = DOM;
 
@@ -66,20 +65,20 @@ const RequestListHeader = createClass({
     this.background.destroy();
     this.background = null;
     window.removeEventListener("resize", this.resizeWaterfall);
   },
 
   resizeWaterfall() {
     // Measure its width and update the 'waterfallWidth' property in the store.
     // The 'waterfallWidth' will be further updated on every window resize.
-    setNamedTimeout("resize-events", 50, () => {
-      const { width } = this.refs.header.getBoundingClientRect();
+    setTimeout(() => {
+      let { width } = this.refs.header.getBoundingClientRect();
       this.props.resizeWaterfall(width);
-    });
+    }, 50);
   },
 
   render() {
     const { sort, scale, waterfallWidth, onHeaderClick } = this.props;
 
     return div(
       { className: "devtools-toolbar requests-list-toolbar" },
       div({ className: "toolbar-labels" },