Fix for Multipage selection by dragging mouse in OOP case in PDF.

chrome/browser/resources/pdf/pdf.js

 // Copyright 2013 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.
 
 'use strict';
 
 /**
  * @return {number} Width of a scrollbar in pixels
  */
 function getScrollbarWidth() {
@@ skipping 135 matching lines @@
       // when we initiate the zoom.
       if ((zoomDelta > MIN_ZOOM_DELTA) && !this.setZoomInProgress_)
         this.viewport_.setZoom(zoomChangeInfo.newZoomFactor);
     }.bind(this));
   }
 
   // Parse open pdf parameters.
   this.paramsParser_ = new OpenPDFParamsParser();
   this.navigator_ = new Navigator(this.streamDetails_.originalUrl,
       this.viewport_, this.paramsParser_);
+  this.isSelecting_ = false;
+  this.mousemoveCallback_ = null;
+
 }
 
 PDFViewer.prototype = {
   /**
    * @private
    * Handle key events. These may come from the user directly or via the
    * scripting API.
    * @param {KeyboardEvent} e the event to handle.
    */
   handleKeyEvent_: function(e) {
@@ skipping 262 matching lines @@
         if (!this.isPrintPreview_)
           this.progressBar_.style.visibility = 'visible';
         this.errorScreen_.text = message.data.loadFailedString;
         break;
       case 'cancelStreamUrl':
         chrome.mimeHandlerPrivate.abortStream();
         break;
       case 'bookmarks':
         this.bookmarks_ = message.data.bookmarks;
         break;
+      case 'setIsSelecting': {

[Sam McNally, 2015/01/28 02:43:51]

Match the other cases: no {}.

[Deepak, 2015/01/28 11:59:26]

Done.

+        if (message.data.isSelecting == this.isSelecting_)

[raymes, 2015/01/28 03:33:08]

Does this ever happen?

[Deepak, 2015/01/28 11:59:26]

This is not needed as we are sending message when selecting_ is getting changed.

+            break;

[Sam McNally, 2015/01/28 02:43:51]

2 space indent.

[Deepak, 2015/01/28 11:59:26]

Done.

+        this.isSelecting_ = message.data.isSelecting;
+        if (this.isSelecting_) {
+            this.mousemoveCallback_ = this.selectionDragListener_.bind(this);
+            this.plugin_.addEventListener('mousemove',
+                this.mousemoveCallback_, false);

[Sam McNally, 2015/01/28 02:43:51]

Line up with the opening ( or break before 'mousemove'.

[Deepak, 2015/01/28 11:59:26]

If I bring this.mousemoveCallback_ to start of '(' before 'mousemove' then it
goes more than 80 chars.
Is the rule of breaking line will go 4 spaces more than the parent line does not
works for js.
does not found much at
http://google-styleguide.googlecode.com/svn/trunk/javascriptguide.xml.
Please correct me if I missed something.

[Sam McNally, 2015/02/04 02:56:01]

See code formatting > function arguments.

+        } else {
+            this.plugin_.removeEventListener('mousemove',
+                this.mousemoveCallback_, false);
+        }
+      }
+      break;
     }
   },
 
   /**
+   * @private
+   * Helper function that check weather point is in inside rect or not.
+   * @param {Object} point The point that need to be checked.
+   * @param {Object} rect The rect with top left point and bottom right point.
+   * @return {boolean} returns true if point is inside rect else returns false.
+   */
+  isPointInsideRect_: function(point, rect) {

[Sam McNally, 2015/01/28 02:43:51]

This doesn't need to be a method.

[Deepak, 2015/01/28 11:59:26]

Done.

+    var x1 = Math.min(rect.x1, rect.x2);

[Sam McNally, 2015/01/28 02:43:51]

Why are these here?

[Deepak, 2015/01/28 11:59:26]

Done.

+    var x2 = Math.max(rect.x1, rect.x2);
+    var y1 = Math.min(rect.y1, rect.y2);
+    var y2 = Math.max(rect.y1, rect.y2);
+    return ((x1 <= point.x && point.x <= x2) &&
+        (y1 <= point.y && point.y <= y2));
+  },
+
+  /**
+   * @private
+   * Handles mousemove events. Scrolls page when mouse move happen reacting to
+   * the direction scroll.
+   * @param {Object} event The mouse move event.
+   */
+  selectionDragListener_: function(event) {
+    var position = this.viewport_.position;
+    var viewportRect = {
+      x1: position.x / this.viewport_.zoom,
+      y1: position.y / this.viewport_.zoom,
+      x2: (position.x + this.viewport_.size.width) / this.viewport_.zoom,
+      y2: (position.y + this.viewport_.size.height) / this.viewport_.zoom
+    };
+    var point = {
+      x: event.pageX / this.viewport_.zoom,

[Sam McNally, 2015/01/28 02:43:51]

Check whether screenX or screenY is between 0 and this.viewport_.size.width or
this.viewport_.size.height, respectively.

[Deepak, 2015/01/28 11:59:26]

Done.

+      y: event.pageY / this.viewport_.zoom
+    };
+    var isPointInside = this.isPointInsideRect_(point, viewportRect);
+    if (isPointInside)
+      return;
+
+    if (this.viewport_.documentHasScrollbars().vertical) {
+      if (point.y < viewportRect.y1 && viewportRect.y1 > 0) {
+        position.y -= (Viewport.SCROLL_INCREMENT / this.viewport_.zoom);

[Sam McNally, 2015/01/28 02:43:51]

This will only scroll when the mouse moves, and always at a fixed speed.

What this should do is set the scroll velocity based on how far the mouse is
outside the viewport and use a timer to increment the position by the velocity.

[Deepak, 2015/01/28 11:59:26]

yes, Currently it is scrolling when their is mouse move and at the fixed speed.
Locally I tried to increase speed based on how distance mouse is outside the
view port. it works well.
**In adobe reader scrolling happen with same speed irrespective of mouse
position from viewport.

But when I tried to use 
this.timerVal_= setInterval(this.fun.bind(this),timeinms);

then when I keep mouse still then based on timer it is calling function and
updating scrolling. But 2 issue happen:
1) While scrolling with timer selecting is not updating.
2) while clearing interval by 
        if (this.timerVal_ != null) {
            clearInterval(this.timerVal_);
            this.timerVal_ = null;
        }
It is calling clearInterval() but still calls are going to the 'fun' function
after every timeinms.
can you please suggest clue to address these.
Thanks.

+      } else if ((point.y > viewportRect.y2) &&
+        (event.pageY < (this.viewport_.documentDimensions_.height *
+            this.viewport_.zoom))) {
+        position.y += (Viewport.SCROLL_INCREMENT / this.viewport_.zoom);
+      }
+    }
+    if (this.viewport_.documentHasScrollbars().horizontal) {
+      if (point.x < viewportRect.x1 && viewportRect.x1 > 0) {
+        position.x -= (Viewport.SCROLL_INCREMENT / this.viewport_.zoom);
+      } else if ((point.x > viewportRect.x2) &&
+        (event.pageX < this.viewport_.documentDimensions_.width *
+            this.viewport_.zoom)) {
+        position.x += (Viewport.SCROLL_INCREMENT / this.viewport_.zoom);
+      }
+    }
+    this.viewport_.position = position;
+  },
+
+  /**
    * @private
    * A callback that's called before the zoom changes. Notify the plugin to stop
    * reacting to scroll events while zoom is taking place to avoid flickering.
    */
   beforeZoom_: function() {
     this.plugin_.postMessage({
       type: 'stopScrolling'
     });
   },
 
@@ skipping 213 matching lines @@
               this.streamDetails_.tabId != -1);
   },
 
   /**
    * @type {Viewport} the viewport of the PDF viewer.
    */
   get viewport() {
     return this.viewport_;
   }
 };