Add flag to enable Material Design for the PDF Viewer.

Add flag to enable Material Design for the PDF Viewer.

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

[Alexandre Carlton, 2015-02-03 23:25:42]

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

[raymes, 2015-02-04 00:33:17]

lg but you'll need to update the histograms xml file.

[Alexandre Carlton, 2015-02-04 02:39:22]

On 2015/02/04 00:33:17, raymes wrote:
> lg but you'll need to update the histograms xml file.

Done, now ./out/Debug/unit_tests
--gtest_filter='AboutFlagsHistogramTest.CheckHistograms' passes.

[raymes, 2015-02-04 02:41:54]

https://codereview.chromium.org/896463005/diff/20001/tools/metrics/histograms...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/896463005/diff/20001/tools/metrics/histograms...
tools/metrics/histograms/histograms.xml:50962: +  <int value="993472930"
label="pdf-material-ui"/>
I don't think we need this

[raymes, 2015-02-04 02:41:54]

https://codereview.chromium.org/896463005/diff/20001/tools/metrics/histograms...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/896463005/diff/20001/tools/metrics/histograms...
tools/metrics/histograms/histograms.xml:50962: +  <int value="993472930"
label="pdf-material-ui"/>
I don't think we need this

[Alexandre Carlton, 2015-02-04 02:48:32]

alexandrec@chromium.org changed reviewers:
+ isherman@chromium.org

[Alexandre Carlton, 2015-02-04 02:48:33]

isherman@chromium.org: Please review changes in 

tools/metrics/histograms/histograms.xml

Thank you.

https://codereview.chromium.org/896463005/diff/20001/tools/metrics/histograms...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/896463005/diff/20001/tools/metrics/histograms...
tools/metrics/histograms/histograms.xml:50962: +  <int value="993472930"
label="pdf-material-ui"/>
On 2015/02/04 02:41:54, raymes wrote:
> I don't think we need this

Done.

[Sam McNally, 2015-02-04 02:51:18]

lgtm

[raymes, 2015-02-04 02:52:19]

lgtm

[benwells, 2015-02-04 17:57:13]

lgtm

[Ilya Sherman, 2015-02-04 23:20:22]

histograms.xml lgtm

[Ilya Sherman, 2015-02-04 23:20:31]

The CQ bit was checked by isherman@chromium.org

[commit-bot: I haz the power, 2015-02-04 23:21:35]

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

[commit-bot: I haz the power, 2015-02-04 23:25:34]

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

[commit-bot: I haz the power, 2015-02-04 23:25:35]

Try jobs failed on following builders:
  ios_dbg_simulator on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_ng on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ng...)
  ios_rel_device_ninja_ng on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Alexandre Carlton, 2015-02-04 23:37:35]

New patchsets have been uploaded after l-g-t-m from
sammc@chromium.org,raymes@chromium.org,benwells@chromium.org,isherman@chromiu...

[Alexandre Carlton, 2015-02-04 23:38:22]

The CQ bit was checked by alexandrec@chromium.org

[commit-bot: I haz the power, 2015-02-04 23:39:53]

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

[commit-bot: I haz the power, 2015-02-05 00:54:02]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-02-05 00:55:34]

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