Convert date_time_page to 0.8.

Convert date_time_page to 0.8.

While I was at it, I also added the page title as a translated string, as this seems to have been forgotten in the first implementation.

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

[Kyle Horimoto, 2015-05-11 19:05:12]

khorimoto@chromium.org changed reviewers:
+ jlklein@chromium.org

[Kyle Horimoto, 2015-05-11 19:05:12]

Hopefully this looks good. Difficult to figure out if I did this correctly since
we can't actually run it/test it yet!

[Jeremy Klein, 2015-05-11 21:20:48]

https://codereview.chromium.org/1137063002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/settings/date_time_page/date_time_page.css
(right):

https://codereview.chromium.org/1137063002/diff/20001/chrome/browser/resource...
chrome/browser/resources/settings/date_time_page/date_time_page.css:5:
core-label {
core-label doesn't exist in 0.8. What should this be?

https://codereview.chromium.org/1137063002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/settings/date_time_page/date_time_page.html
(right):

https://codereview.chromium.org/1137063002/diff/20001/chrome/browser/resource...
chrome/browser/resources/settings/date_time_page/date_time_page.html:7: <link
rel="stylesheet" href="chrome://md-settings/settings_page/settings_page.css">
I think this still needs to be rel="import" type="css"

https://codereview.chromium.org/1137063002/diff/20001/chrome/browser/resource...
chrome/browser/resources/settings/date_time_page/date_time_page.html:9:
<paper-material class="layout veritcal cross-fade">
This probably requires importing classes/iron-flex-layout.html, but it may be
easier to do these in css now.

Side note - is cross-fade also a class now?

https://codereview.chromium.org/1137063002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/settings/date_time_page/date_time_page.js (right):

https://codereview.chromium.org/1137063002/diff/20001/chrome/browser/resource...
chrome/browser/resources/settings/date_time_page/date_time_page.js:29: prefs: {
This may need notify: true.

[Kyle Horimoto, 2015-05-11 21:34:57]

khorimoto@chromium.org changed reviewers:
+ jhawkins@chromium.org

[Kyle Horimoto, 2015-05-11 21:34:58]

+jhakwins for:
chrome/app/settings_strings.grdp
chrome/browser/ui/webui/settings/md_settings_localized_strings_provider.cc

https://codereview.chromium.org/1137063002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/settings/date_time_page/date_time_page.css
(right):

https://codereview.chromium.org/1137063002/diff/20001/chrome/browser/resource...
chrome/browser/resources/settings/date_time_page/date_time_page.css:5:
core-label {
On 2015/05/11 21:20:48, Jeremy Klein wrote:
> core-label doesn't exist in 0.8. What should this be?

Crap, you're right. This was just a wrapper for .time-zone-label, so I just
adjusted that selector to add an extra 10 pixels of margin on all sides.

https://codereview.chromium.org/1137063002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/settings/date_time_page/date_time_page.html
(right):

https://codereview.chromium.org/1137063002/diff/20001/chrome/browser/resource...
chrome/browser/resources/settings/date_time_page/date_time_page.html:7: <link
rel="stylesheet" href="chrome://md-settings/settings_page/settings_page.css">
On 2015/05/11 21:20:48, Jeremy Klein wrote:
> I think this still needs to be rel="import" type="css"

Done.

https://codereview.chromium.org/1137063002/diff/20001/chrome/browser/resource...
chrome/browser/resources/settings/date_time_page/date_time_page.html:9:
<paper-material class="layout veritcal cross-fade">
On 2015/05/11 21:20:48, Jeremy Klein wrote:
> This probably requires importing classes/iron-flex-layout.html, but it may be
> easier to do these in css now.
> 
> Side note - is cross-fade also a class now?

Imported the layout, and removed cross-fade as discussed offline.

https://codereview.chromium.org/1137063002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/settings/date_time_page/date_time_page.js (right):

https://codereview.chromium.org/1137063002/diff/20001/chrome/browser/resource...
chrome/browser/resources/settings/date_time_page/date_time_page.js:29: prefs: {
On 2015/05/11 21:20:48, Jeremy Klein wrote:
> This may need notify: true.

Done.

[Jeremy Klein, 2015-05-11 21:36:39]

lgtm

[James Hawkins, 2015-05-11 21:38:34]

lgtm

[Kyle Horimoto, 2015-05-11 21:38:59]

The CQ bit was checked by khorimoto@chromium.org

[commit-bot: I haz the power, 2015-05-11 21:41:39]

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

[commit-bot: I haz the power, 2015-05-11 23:44:39]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-05-11 23:45:27]

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