Extract i18n behavior from settings.

Extract i18n behavior from settings.

The i18n behavior previously defined for md-settings seems to be usefull
for other CLs (e.g. https://codereview.chromium.org/1371073003/).
Therefore, it is moved to resources/{js,html}.

BUG=134849
Committed: https://crrev.com/9a63e1a18a28fa065085c359c323c8887f0d38ed
Cr-Commit-Position: refs/heads/master@{#354464}

[fhorschig, 2015-10-13 10:07:47]

Patchset #1 (id:1) has been deleted

[fhorschig, 2015-10-13 10:08:45]

fhorschig@chromium.org changed reviewers:
+ dbeam@chromium.org

[fhorschig, 2015-10-13 10:08:45]

Hi Dan,
would you please have a short look on the extraction of the I18nBehavior used in
CL 1371073003?

Cheers,
Friedrich

[fhorschig, 2015-10-13 10:09:48]

CC'ed dschuyler@ as original author of this class in case he is interested in
this change.

[Dan Beam, 2015-10-15 03:45:43]

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

https://codereview.chromium.org/1404593002/diff/20001/chrome/browser/resource...
chrome/browser/resources/settings/default_browser_page/default_browser_page.html:4:
<link rel="import" href="chrome://resources/html/polymer_i18n_behavior.html">
why polymer_?

[fhorschig, 2015-10-15 15:10:35]

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

https://codereview.chromium.org/1404593002/diff/20001/chrome/browser/resource...
chrome/browser/resources/settings/default_browser_page/default_browser_page.html:4:
<link rel="import" href="chrome://resources/html/polymer_i18n_behavior.html">
On 2015/10/15 03:45:43, Dan Beam wrote:
> why polymer_?

To state its relation to polymer, similar to polymer_config. 
I dropped the prefix as your question made me think that it might be
unnecessary.

[fhorschig, 2015-10-15 15:34:14]

Patchset #2 (id:40001) has been deleted

[Dan Beam, 2015-10-15 16:50:05]

lgtm

[stevenjb, 2015-10-15 21:33:16]

On 2015/10/15 16:50:05, Dan Beam wrote:
> lgtm

I would definitely prefer to see the behavior html and js in the same directory
as we have been doing for other Polymer elements. Any reason not to just put
these in webui/resources/cr_elements/?

(We should remove the webui/resources/cr_elements/1_0/ subdirectory now that we
don't have an .8 elements; I can do that)

[stevenjb, 2015-10-15 22:04:22]

On 2015/10/15 21:33:16, stevenjb wrote:
> On 2015/10/15 16:50:05, Dan Beam wrote:
> > lgtm
> 
> I would definitely prefer to see the behavior html and js in the same
directory
> as we have been doing for other Polymer elements. Any reason not to just put
> these in webui/resources/cr_elements/?
> 
> (We should remove the webui/resources/cr_elements/1_0/ subdirectory now that
we
> don't have an .8 elements; I can do that)

https://codereview.chromium.org/1406023003/

[Dan Beam, 2015-10-16 01:20:37]

On 2015/10/15 22:04:22, stevenjb wrote:
> On 2015/10/15 21:33:16, stevenjb wrote:
> > On 2015/10/15 16:50:05, Dan Beam wrote:
> > > lgtm
> > 
> > I would definitely prefer to see the behavior html and js in the same
> directory
> > as we have been doing for other Polymer elements. Any reason not to just put
> > these in webui/resources/cr_elements/?
> > 
> > (We should remove the webui/resources/cr_elements/1_0/ subdirectory now that
> we
> > don't have an .8 elements; I can do that)
> 
> https://codereview.chromium.org/1406023003/

re: behaviors in cr_elements/ -> well, are behaviors elements?  not really

[Dan Beam, 2015-10-16 06:40:36]

The CQ bit was checked by dbeam@chromium.org

[commit-bot: I haz the power, 2015-10-16 06:41:01]

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

[commit-bot: I haz the power, 2015-10-16 07:39:30]

Committed patchset #2 (id:60001)

[commit-bot: I haz the power, 2015-10-16 07:40:14]

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