[UserScript] Do not create SubstitutionMap if not necessary.

[UserScript] Do not create SubstitutionMap if not necessary.

Tiny improvement.
We won't need to call GetLocalizationMessages for most extensions as
they wouldn't be needed if those extensions only have content_script
JS but not CSS.

BUG=622464
Test=None, internal cleanup.

Committed: https://crrev.com/4ff54b3aa230cb7a5f7c9b30181ff4e19d763a46
Cr-Commit-Position: refs/heads/master@{#410810}

[lazyboy, 2016-07-29 07:57:47]

The CQ bit was checked by lazyboy@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-29 07:57:56]

Dry run: 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-07-29 08:58:22]

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

[commit-bot: I haz the power, 2016-07-29 08:58:28]

Dry run: This issue passed the CQ dry run.

[lazyboy, 2016-08-08 20:18:02]

Description was changed from

==========
[UserScript] Do not create SubstitutionMap if not necessary.

Tiny improvement.

BUG=622464
Test=None, internal cleanup.
==========

to

==========
[UserScript] Do not create SubstitutionMap if not necessary.

Tiny improvement.
We won't need to call GetLocalizationMessages for most extensions as
they wouldn't be needed if those extensions only have content_script
JS but not CSS.

BUG=622464
Test=None, internal cleanup.
==========

[lazyboy, 2016-08-08 23:35:26]

lazyboy@chromium.org changed reviewers:
+ asargent@chromium.org

[asargent_no_longer_on_chrome, 2016-08-09 00:01:30]

lgtm

[lazyboy, 2016-08-09 00:41:16]

The CQ bit was checked by lazyboy@chromium.org

[commit-bot: I haz the power, 2016-08-09 00:41:48]

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-09 00:50:50]

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

[commit-bot: I haz the power, 2016-08-09 00:50:52]

Try jobs failed on following builders:
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)

[lazyboy, 2016-08-09 20:30:22]

The CQ bit was checked by lazyboy@chromium.org

[commit-bot: I haz the power, 2016-08-09 20:31:24]

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-09 20:49:31]

Description was changed from

==========
[UserScript] Do not create SubstitutionMap if not necessary.

Tiny improvement.
We won't need to call GetLocalizationMessages for most extensions as
they wouldn't be needed if those extensions only have content_script
JS but not CSS.

BUG=622464
Test=None, internal cleanup.
==========

to

==========
[UserScript] Do not create SubstitutionMap if not necessary.

Tiny improvement.
We won't need to call GetLocalizationMessages for most extensions as
they wouldn't be needed if those extensions only have content_script
JS but not CSS.

BUG=622464
Test=None, internal cleanup.
==========

[commit-bot: I haz the power, 2016-08-09 20:49:33]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-09 20:52:44]

Description was changed from

==========
[UserScript] Do not create SubstitutionMap if not necessary.

Tiny improvement.
We won't need to call GetLocalizationMessages for most extensions as
they wouldn't be needed if those extensions only have content_script
JS but not CSS.

BUG=622464
Test=None, internal cleanup.
==========

to

==========
[UserScript] Do not create SubstitutionMap if not necessary.

Tiny improvement.
We won't need to call GetLocalizationMessages for most extensions as
they wouldn't be needed if those extensions only have content_script
JS but not CSS.

BUG=622464
Test=None, internal cleanup.

Committed: https://crrev.com/4ff54b3aa230cb7a5f7c9b30181ff4e19d763a46
Cr-Commit-Position: refs/heads/master@{#410810}
==========

[commit-bot: I haz the power, 2016-08-09 20:52:45]

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