Material PDF: Variety of minor UI bugfixes

Material PDF: Variety of minor UI bugfixes

* Fix regression where dropdowns would not be sized correctly on first
  open.
* Make page selector only accept numerical inputs
* Position top toolbar z-level above zoom toolbar
* Ellipsize long bookmark names

BUG=527776, 527798, 526981
Committed: https://crrev.com/2667644160e695c5170d64ec0ee9759112e9b386
Cr-Commit-Position: refs/heads/master@{#347597}

[tsergeant, 2015-09-04 03:53:11]

tsergeant@chromium.org changed reviewers:
+ raymes@chromium.org

[tsergeant, 2015-09-04 03:53:12]

~~~PTAL~~~

[raymes, 2015-09-05 01:41:33]

nit in description: * Position top toolbar z-level about zoom toolbar
about->above

Looks good, just one thing I don't follow :)

https://codereview.chromium.org/1310353003/diff/1/chrome/browser/resources/pd...
File
chrome/browser/resources/pdf/elements/viewer-page-selector/viewer-page-selector.css
(right):

https://codereview.chromium.org/1310353003/diff/1/chrome/browser/resources/pd...
chrome/browser/resources/pdf/elements/viewer-page-selector/viewer-page-selector.css:39:
margin: 0;
What was this change for?

[tsergeant, 2015-09-07 00:05:11]

https://codereview.chromium.org/1310353003/diff/1/chrome/browser/resources/pd...
File
chrome/browser/resources/pdf/elements/viewer-page-selector/viewer-page-selector.css
(right):

https://codereview.chromium.org/1310353003/diff/1/chrome/browser/resources/pd...
chrome/browser/resources/pdf/elements/viewer-page-selector/viewer-page-selector.css:39:
margin: 0;
On 2015/09/05 at 01:41:33, raymes wrote:
> What was this change for?

I changed the <input> to type="number", so that you can't enter anything that's
not a number. This CSS hides the spinner buttons that appear for number inputs.

[raymes, 2015-09-07 00:06:48]

lgtm

[tsergeant, 2015-09-07 00:12:23]

The CQ bit was checked by tsergeant@chromium.org

[commit-bot: I haz the power, 2015-09-07 00:12:29]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1310353003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1310353003/1

[commit-bot: I haz the power, 2015-09-07 00:49:39]

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

[commit-bot: I haz the power, 2015-09-07 00:49:39]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[tsergeant, 2015-09-07 00:59:59]

The CQ bit was checked by tsergeant@chromium.org

[tsergeant, 2015-09-07 00:59:59]

The patchset sent to the CQ was uploaded after l-g-t-m from raymes@chromium.org
Link to the patchset: https://codereview.chromium.org/1310353003/#ps20001
(title: "Fix tests")

[commit-bot: I haz the power, 2015-09-07 01:00:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1310353003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1310353003/20001

[commit-bot: I haz the power, 2015-09-07 02:21:14]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-09-07 02:21:54]

Patchset 2 (id:??) landed as
https://crrev.com/2667644160e695c5170d64ec0ee9759112e9b386
Cr-Commit-Position: refs/heads/master@{#347597}