Implement "go to page" functionality in out of process PDF

Implement "go to page" functionality in out of process PDF

The page can receive messages from the plugin that instruct it to scroll to a particular page.

BUG=303491
R=arv@chromium.org, koz@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=252178

[koz (OOO until 15th September), 2014-02-18 07:13:24]

lgtm

[raymes, 2014-02-19 00:23:52]

+arv for OWNERS

[arv (Not doing code reviews), 2014-02-19 02:18:08]

LGTM with nit

https://codereview.chromium.org/165813004/diff/1/chrome/browser/resources/pdf...
File chrome/browser/resources/pdf/pdf.js (right):

https://codereview.chromium.org/165813004/diff/1/chrome/browser/resources/pdf...
chrome/browser/resources/pdf/pdf.js:66: } else if (message.data.type ==
'goToPage') {
Maybe use a switch instead?

[raymes, 2014-02-19 03:24:35]

https://codereview.chromium.org/165813004/diff/1/chrome/browser/resources/pdf...
File chrome/browser/resources/pdf/pdf.js (right):

https://codereview.chromium.org/165813004/diff/1/chrome/browser/resources/pdf...
chrome/browser/resources/pdf/pdf.js:66: } else if (message.data.type ==
'goToPage') {
On 2014/02/19 02:18:08, arv wrote:
> Maybe use a switch instead?

Done.

[raymes, 2014-02-19 23:49:19]

The CQ bit was checked by raymes@chromium.org

[commit-bot: I haz the power, 2014-02-20 01:18:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/raymes@chromium.org/165813004/130001

[commit-bot: I haz the power, 2014-02-20 01:18:48]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-02-20 01:18:49]

Failed to apply patch for chrome/browser/resources/pdf/index.html:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file chrome/browser/resources/pdf/index.html
  Hunk #1 FAILED at 173.
  Hunk #2 FAILED at 186.
  Hunk #3 FAILED at 226.
  3 out of 3 hunks FAILED -- saving rejects to file
chrome/browser/resources/pdf/index.html.rej

Patch:       chrome/browser/resources/pdf/index.html
Index: chrome/browser/resources/pdf/index.html
diff --git a/chrome/browser/resources/pdf/index.html
b/chrome/browser/resources/pdf/index.html
index
c7876c948207cee13ffedd3ef4bfbeed191739de..71b557d0954b4dfe42a8060eac35c8fffbf3baf1
100644
--- a/chrome/browser/resources/pdf/index.html
+++ b/chrome/browser/resources/pdf/index.html
@@ -173,7 +173,7 @@ license that can be found in the LICENSE file.
   
 </polymer-element>
 
-<polymer-element name="viewer-toolbar" attributes="fadingIn"
assetpath="html_office/elements/viewer-toolbar/">
+<polymer-element name="viewer-toolbar" attributes="fadingIn"
on-mouseover="{{fadeIn}}" on-mousemove="{{fadeIn}}" on-mouseout="{{fadeOut}}"
assetpath="html_office/elements/viewer-toolbar/">
 <template>
   <style>/* Copyright 2013 The Chromium Authors. All rights reserved.
  * Use of this source code is governed by a BSD-style license that can be
@@ -186,7 +186,6 @@ license that can be found in the LICENSE file.
   font-size: 0;
   opacity: 1;
   padding: 30px 30px 15px 30vw;
-  pointer-events: none;
   position: fixed;
   right: 0;
 }
@@ -226,7 +225,6 @@ license that can be found in the LICENSE file.
   display: inline-block;
   height: 36px;
   margin: 0;
-  pointer-events: all;
   width: 43px;
 }

[raymes, 2014-02-20 04:36:14]

Committed patchset #5 manually as r252178 (presubmit successful).