Preprocess icons.html files for <if>s

Preprocess settings/icons.html and cr_elements/icons.html for <if>s

BUG=605821
R=dbeam@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[michaelpg, 2016-08-16 00:46:50]

Description was changed from

==========
Flatten settings/icons.html for <if>s

BUG=605821
R=dbeam@chromium.org
==========

to

==========
Flatten settings/icons.html for <if>s

BUG=605821
R=dbeam@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[michaelpg, 2016-08-16 00:47:54]

oops

[Dan Beam, 2016-08-16 00:52:41]

can you please use preprocess=true instead if you only need <if>?

[Dan Beam, 2016-08-16 00:52:53]

On 2016/08/16 00:52:41, Dan Beam wrote:
> can you please use preprocess=true instead if you only need <if>?

preprocess="true"**

[michaelpg, 2016-08-16 00:57:42]

Description was changed from

==========
Flatten settings/icons.html for <if>s

BUG=605821
R=dbeam@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Preprocess settings/icons.html and cr_elements/icons.html for <if>s

BUG=605821
R=dbeam@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[michaelpg, 2016-08-16 00:58:26]

ptal

preprocess="true" doesn't seem to work for md_history/history.html --
flattenhtml does, though. maybe because it's BINDATA?

[Dan Beam, 2016-08-16 01:27:08]

On 2016/08/16 00:58:26, michaelpg wrote:
> ptal
> 
> preprocess="true" doesn't seem to work for md_history/history.html --
> flattenhtml does, though. maybe because it's BINDATA?

yes, i'm pretty sure the grit code that looks for these attributes only cares
when type="chrome_html"

[Dan Beam, 2016-08-16 01:27:22]

lgtm

[michaelpg, 2016-08-16 02:57:38]

The CQ bit was checked by michaelpg@chromium.org

[commit-bot: I haz the power, 2016-08-16 02:58:10]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-16 03:41:10]

Description was changed from

==========
Preprocess settings/icons.html and cr_elements/icons.html for <if>s

BUG=605821
R=dbeam@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Preprocess settings/icons.html and cr_elements/icons.html for <if>s

BUG=605821
R=dbeam@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[commit-bot: I haz the power, 2016-08-16 03:41:11]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-16 03:43:01]

Description was changed from

==========
Preprocess settings/icons.html and cr_elements/icons.html for <if>s

BUG=605821
R=dbeam@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Preprocess settings/icons.html and cr_elements/icons.html for <if>s

BUG=605821
R=dbeam@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/e4e61b6bb3a602adecf92d4e0a41804e5997895f
Cr-Commit-Position: refs/heads/master@{#412159}
==========

[commit-bot: I haz the power, 2016-08-16 03:43:02]

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