Backed out changeset 2c9fc232f3ca (bug 1210632) for eslint failure on /devtools/server/actors/highlighters/measuring-tool.js:522:66 r=backout on a CLOSED TREE
authorDorel Luca <dluca@mozilla.com>
Wed, 10 Jan 2018 20:28:20 +0200
changeset 450339 8467a81a2b5b3213e00137854ae45f2bd939d77e
parent 450338 49462334e363e8b248396b954be72bffe40c5bd0
child 450340 731b1321ba9566f571aed01387bf34a66390d86f
push id8527
push userCallek@gmail.com
push dateThu, 11 Jan 2018 21:05:50 +0000
treeherdermozilla-beta@95342d212a7a [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbackout
bugs1210632
milestone59.0a1
backs out2c9fc232f3cae5a3c9f6cddac1848eb551167a19
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Backed out changeset 2c9fc232f3ca (bug 1210632) for eslint failure on /devtools/server/actors/highlighters/measuring-tool.js:522:66 r=backout on a CLOSED TREE
devtools/server/actors/highlighters/measuring-tool.js
--- a/devtools/server/actors/highlighters/measuring-tool.js
+++ b/devtools/server/actors/highlighters/measuring-tool.js
@@ -514,18 +514,18 @@ MeasuringToolHighlighter.prototype = {
         break;
       case "mousemove":
         ({ scrollX, scrollY, innerWidth, innerHeight } = this.env.window);
         x = event.clientX + scrollX;
         y = event.clientY + scrollY;
 
         let { coords } = this;
 
-        x = Math.min(innerWidth + scrollX, Math.max(scrollX, x));
-        y = Math.min(innerHeight + scrollY, Math.max(scrollY, y));
+        x = Math.min(innerWidth + scrollX - 1, Math.max(0 + scrollX, x));
+        y = Math.min(innerHeight + scrollY, Math.max(1 + scrollY, y));
 
         this.setSize(x - coords.x, y - coords.y);
 
         let type = this._isDragging ? "size" : "position";
 
         this.showLabel(type);
         break;
       case "mouseleave":