Material PDF: Style audit and cleanup

Material PDF: Style audit and cleanup

This CL modifies styles across the PDF viewer to make them match the design
mocks more closely, and cleans up and simplifies styles wherever possible.

BUG=439114
TBR=estade@chromium.org

Committed: https://crrev.com/f693f2575ec0a37917b1c8c429537d8634919c45
Cr-Commit-Position: refs/heads/master@{#345800}

[tsergeant, 2015-08-26 06:35:23]

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

[tsergeant, 2015-08-26 06:35:23]

PTAL! This is a bit of a mixed bag of changes, so let me know if you're not sure
what different things are changing.

[raymes, 2015-08-27 00:43:46]

lgtm

https://codereview.chromium.org/1312373002/diff/40001/chrome/browser/resource...
File
chrome/browser/resources/pdf/elements/viewer-page-selector/viewer-page-selector.html
(right):

https://codereview.chromium.org/1312373002/diff/40001/chrome/browser/resource...
chrome/browser/resources/pdf/elements/viewer-page-selector/viewer-page-selector.html:13:
/ <span id="pagelength-inner">{{docLength}}</span>
Maybe have 2 sibling spans for the slash and the length

[tsergeant, 2015-08-27 02:51:59]

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

[tsergeant, 2015-08-27 02:52:48]

Adding estade@ as TBR for trivial change to component_extension_resources.grd

https://codereview.chromium.org/1312373002/diff/40001/chrome/browser/resource...
File
chrome/browser/resources/pdf/elements/viewer-page-selector/viewer-page-selector.html
(right):

https://codereview.chromium.org/1312373002/diff/40001/chrome/browser/resource...
chrome/browser/resources/pdf/elements/viewer-page-selector/viewer-page-selector.html:13:
/ <span id="pagelength-inner">{{docLength}}</span>
On 2015/08/27 at 00:43:46, raymes wrote:
> Maybe have 2 sibling spans for the slash and the length

Done. It still uses an extra parent element for the length to help with spacing,
but it's a nicer solution overall.

[tsergeant, 2015-08-27 02:53:36]

The CQ bit was checked by tsergeant@chromium.org

[tsergeant, 2015-08-27 02:53:36]

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/1312373002/#ps60001
(title: "Page selector tweaks")

[commit-bot: I haz the power, 2015-08-27 02:53:42]

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

[commit-bot: I haz the power, 2015-08-27 03:24:50]

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

[commit-bot: I haz the power, 2015-08-27 03:24:51]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[tsergeant, 2015-08-27 03:33:15]

The CQ bit was checked by tsergeant@chromium.org

[tsergeant, 2015-08-27 03:33:15]

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/1312373002/#ps80001
(title: "Fix tests for page selector")

[commit-bot: I haz the power, 2015-08-27 03:33:23]

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

[commit-bot: I haz the power, 2015-08-27 05:09:25]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2015-08-27 05:09:58]

Patchset 5 (id:??) landed as
https://crrev.com/f693f2575ec0a37917b1c8c429537d8634919c45
Cr-Commit-Position: refs/heads/master@{#345800}