Make the Material Design PDF Viewer's toolbar thinner

Make the Material Design PDF Viewer's toolbar thinner

The main toolbar of the Material Design PDF Viewer (at the top containing page selection functionality) is set to use the 'core-narrow' class, setting it to the same height as the toolbar in Inbox by Google.

BUG=110020
Committed: https://crrev.com/7e8cd61f389cc099f1d7bf40507094cdc770e5a8
Cr-Commit-Position: refs/heads/master@{#314293}

[Alexandre Carlton, 2015-01-30 07:09:36]

alexandrec@chromium.org changed reviewers:
+ benwells@chromium.org, raymes@chromium.org, sammc@chromium.org

[raymes, 2015-02-02 00:34:34]

lgtm

[Sam McNally, 2015-02-02 05:58:12]

Make which toolbar thinner?

[Alexandre Carlton, 2015-02-02 06:03:09]

Specified which toolbar was made narrower.

[Sam McNally, 2015-02-02 06:07:13]

On 2015/02/02 06:03:09, Alexandre Carlton wrote:
> Specified which toolbar was made narrower.

PDF viewer toolbar?

[benwells, 2015-02-02 09:57:50]

On 2015/02/02 06:07:13, Sam McNally wrote:
> On 2015/02/02 06:03:09, Alexandre Carlton wrote:
> > Specified which toolbar was made narrower.
> 
> PDF viewer toolbar?

As Sam is hinting at, the description of this CL is very vague considering it
goes into a list of CLs affecting all of chrome.

A better description would be 'Make the material design PDF viewer's toolbar
thinner' or something like that to indicate (a) this is the PDF viewer and (b)
it is the new experimental material design version of the PDF viewer.

[benwells, 2015-02-02 09:58:03]

On 2015/02/02 09:57:50, benwells wrote:
> On 2015/02/02 06:07:13, Sam McNally wrote:
> > On 2015/02/02 06:03:09, Alexandre Carlton wrote:
> > > Specified which toolbar was made narrower.
> > 
> > PDF viewer toolbar?
> 
> As Sam is hinting at, the description of this CL is very vague considering it
> goes into a list of CLs affecting all of chrome.
> 
> A better description would be 'Make the material design PDF viewer's toolbar
> thinner' or something like that to indicate (a) this is the PDF viewer and (b)
> it is the new experimental material design version of the PDF viewer.

forgot to add, lgtm with the description updated.

[Sam McNally, 2015-02-03 07:23:43]

lgtm

[Alexandre Carlton, 2015-02-03 07:44:35]

The CQ bit was checked by alexandrec@chromium.org

[commit-bot: I haz the power, 2015-02-03 07:44:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/891563004/40001

[commit-bot: I haz the power, 2015-02-03 08:36:57]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-02-03 08:38:49]

Patchset 3 (id:??) landed as
https://crrev.com/7e8cd61f389cc099f1d7bf40507094cdc770e5a8
Cr-Commit-Position: refs/heads/master@{#314293}