Teach i18nTemplate.process() to handle <link rel=import> and <template>

Teach i18nTemplate.process() to handle <link rel=import> and <template>

This is required to support Polymerized web UI until we figure out a
more flicker-resistant solution (see: crbug.com/506009)

This CL was previously reverted here because it broke OOBE login:
https://codereview.chromium.org/1234753002/

BUG=425626, 509087
R=arv@chromium.org,dzhioev@chromium.org
TBR=yoshiki@chromium.org

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

Committed: https://crrev.com/3e0fadfeaad12f8ef176d8ed6571f109b9479985
Cr-Commit-Position: refs/heads/master@{#338758}

[Dan Beam, 2015-07-08 00:39:31]

Patchset #1 (id:1) has been deleted

[Dan Beam, 2015-07-08 02:13:12]

Patchset #1 (id:20001) has been deleted

[Dan Beam, 2015-07-08 02:16:32]

Patchset #1 (id:40001) has been deleted

[Dan Beam, 2015-07-08 02:16:52]

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

[arv (Not doing code reviews), 2015-07-08 22:29:16]

LGTM

Do you need to update the gn build files too?

https://codereview.chromium.org/1229573003/diff/80001/chrome/test/data/webui/...
File chrome/test/data/webui/i18n_process_test.html (right):

https://codereview.chromium.org/1229573003/diff/80001/chrome/test/data/webui/...
chrome/test/data/webui/i18n_process_test.html:6: <link rel=import
href=&quot;data:text/html,
Can you use ' instead?

[Dan Beam, 2015-07-08 23:25:50]

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

[Dan Beam, 2015-07-08 23:25:51]

TBR=yoshiki@ for ui/file_manager and components/chrome_apps/webstore_widget

btw, had to switch back to `var` (instead of `let`) because of iOS compatibility

On 2015/07/08 22:29:16, arv wrote:
> Do you need to update the gn build files too?

nah, this GYP runs separately from Chrome's GYP and has no GN equivalent
(currently)

https://codereview.chromium.org/1229573003/diff/80001/chrome/test/data/webui/...
File chrome/test/data/webui/i18n_process_test.html (right):

https://codereview.chromium.org/1229573003/diff/80001/chrome/test/data/webui/...
chrome/test/data/webui/i18n_process_test.html:6: <link rel=import
href=&quot;data:text/html,
On 2015/07/08 22:29:16, arv wrote:
> Can you use ' instead?

Done.

[Dan Beam, 2015-07-09 00:30:05]

Patchset #4 (id:120001) has been deleted

[Dan Beam, 2015-07-09 00:30:14]

Patchset #3 (id:100001) has been deleted

[Dan Beam, 2015-07-09 01:19:16]

The CQ bit was checked by dbeam@chromium.org

[Dan Beam, 2015-07-09 01:19:17]

The patchset sent to the CQ was uploaded after l-g-t-m from arv@chromium.org
Link to the patchset: https://codereview.chromium.org/1229573003/#ps140001
(title: "test fixes")

[commit-bot: I haz the power, 2015-07-09 01:20:53]

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

[commit-bot: I haz the power, 2015-07-09 03:33:21]

Committed patchset #3 (id:140001)

[commit-bot: I haz the power, 2015-07-09 03:34:04]

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

[Dan Beam, 2015-07-14 01:30:04]

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

[Dan Beam, 2015-07-14 01:30:05]

this was reverted because it broke OOBE login

+dzhioev@ for cycle detection logic

will test on OOBE now

[dzhioev (left Google), 2015-07-14 02:43:23]

https://codereview.chromium.org/1229573003/diff/220001/chrome/test/data/webui...
File chrome/test/data/webui/i18n_process_test.html (right):

https://codereview.chromium.org/1229573003/diff/220001/chrome/test/data/webui...
chrome/test/data/webui/i18n_process_test.html:51: }
I believe we need tests for actual custom elements. Two cases needs to be
tested:
a) a custom element that presented in the DOM tree when process() was called
b) a custom element that was inserted into the DOM after the process() was
called.

Also we need a test checking that calling process() for the second time with a
different values works.

https://codereview.chromium.org/1229573003/diff/220001/ui/webui/resources/css...
File ui/webui/resources/css/i18n_process.css (right):

https://codereview.chromium.org/1229573003/diff/220001/ui/webui/resources/css...
ui/webui/resources/css/i18n_process.css:14: [i18n-processed]
[i18n-values*=".innerHTML:"]::before {
Why these selectors differ from the selectors above?

[Dan Beam, 2015-07-14 05:31:17]

I can more tests soon, though I'd be happy to do in followup as well.

https://codereview.chromium.org/1229573003/diff/220001/ui/webui/resources/css...
File ui/webui/resources/css/i18n_process.css (right):

https://codereview.chromium.org/1229573003/diff/220001/ui/webui/resources/css...
ui/webui/resources/css/i18n_process.css:14: [i18n-processed]
[i18n-values*=".innerHTML:"]::before {
On 2015/07/14 02:43:23, dzhioev wrote:
> Why these selectors differ from the selectors above?

The ones with spaces are descendants. But this rule should work for roots as
well.

[Dan Beam, 2015-07-14 16:28:20]

https://codereview.chromium.org/1229573003/diff/220001/chrome/test/data/webui...
File chrome/test/data/webui/i18n_process_test.html (right):

https://codereview.chromium.org/1229573003/diff/220001/chrome/test/data/webui...
chrome/test/data/webui/i18n_process_test.html:51: }
On 2015/07/14 02:43:22, dzhioev wrote:
> I believe we need tests for actual custom elements. Two cases needs to be
> tested:
> a) a custom element that presented in the DOM tree when process() was called
> b) a custom element that was inserted into the DOM after the process() was
> called.

I tried the typical <link rel=import> (which didn't work because this file is
loaded from file://) as well as:

AddLibrary(IDR_WEBUI_JS_POLYMER_CONFIG);
AddLibrary(IDR_POLYMER_1_0_POLYMER_POLYMER_MICRO_EXTRACTED_JS);
AddLibrary(IDR_POLYMER_1_0_POLYMER_POLYMER_MINI_EXTRACTED_JS);
AddLibrary(IDR_POLYMER_1_0_POLYMER_POLYMER_EXTRACTED_JS);

in the .cc test setup, which also fails because Polymer assumes there's an
ownerDocument when injected (which isn't the case for us:
https://code.google.com/p/chromium/codesearch#chromium/src/content/public/tes...).

I'm fine to attempt to figure this out later or maybe open a chrome:// page to
load things more normally (rather than this weird file:// page) or just disable
restrictions about loading chrome:// pages from other origins, but I don't think
it's a high priority.

> 
> Also we need a test checking that calling process() for the second time with a
> different values works. 

Done.

[Dan Beam, 2015-07-14 17:36:35]

The CQ bit was checked by dbeam@chromium.org

[Dan Beam, 2015-07-14 17:36:36]

The patchset sent to the CQ was uploaded after l-g-t-m from arv@chromium.org
Link to the patchset: https://codereview.chromium.org/1229573003/#ps240001
(title: "add more tests")

[commit-bot: I haz the power, 2015-07-14 17:38:28]

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

[commit-bot: I haz the power, 2015-07-14 19:47:50]

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

[commit-bot: I haz the power, 2015-07-14 19:47: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_...)

[Dan Beam, 2015-07-14 19:55:42]

The CQ bit was checked by dbeam@chromium.org

[commit-bot: I haz the power, 2015-07-14 19:56:32]

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

[Dan Beam, 2015-07-14 21:08:45]

The CQ bit was unchecked by dbeam@chromium.org

[Dan Beam, 2015-07-14 21:10:34]

The CQ bit was checked by dbeam@chromium.org

[Dan Beam, 2015-07-14 21:10:35]

The patchset sent to the CQ was uploaded after l-g-t-m from arv@chromium.org
Link to the patchset: https://codereview.chromium.org/1229573003/#ps260001
(title: "videoplayer test fixes")

[commit-bot: I haz the power, 2015-07-14 21:11:40]

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

[commit-bot: I haz the power, 2015-07-14 22:30:19]

Committed patchset #9 (id:260001)

[commit-bot: I haz the power, 2015-07-14 22:31:33]

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