Use GURLs instead of patterns for SetContentSetting in plugins

Use GURLs instead of patterns for SetContentSetting in plugins

Migrate to use SetContentSettingDefaultScope() which takes GURLs only in plugins.
For special test or specific patterns, remain to use SetContentSetting().

BUG=551747
Committed: https://crrev.com/38ac96539ffa5fc11696885b375c5a49d80fecae
Cr-Commit-Position: refs/heads/master@{#381609}

[lshang, 2016-03-15 04:57:07]

Description was changed from

==========
Use GURLs instead of patterns for SetContentSetting in plugins

Merge branch 'master' into scope_SetContentSetting_in_plugins


changes in plugin_info_message_filter_unittest

BUG=
==========

to

==========
Use GURLs instead of patterns for SetContentSetting in plugins

Migrate to use SetContentSettingDefaultScope() which takes GURLs only in
plugins.
For special test or specific patterns, remain to use SetContentSetting().

BUG=551747
==========

[lshang, 2016-03-15 05:01:32]

lshang@chromium.org changed reviewers:
+ raymes@chromium.org

[lshang, 2016-03-15 05:01:33]

Hi raymes, PTAL thanks!
SetContentSetting() on line 230 is unchanged since it's setting "foo" on all
sites.

[raymes, 2016-03-16 03:27:28]

lgtm

[lshang, 2016-03-16 04:36:07]

lshang@chromium.org changed reviewers:
+ bauerb@chromium.org

[lshang, 2016-03-16 04:36:07]

+bauerb for OWNER review, thanks!

[Bernhard Bauer, 2016-03-16 10:23:00]

lgtm

[lshang, 2016-03-16 23:59:39]

The CQ bit was checked by lshang@chromium.org

[commit-bot: I haz the power, 2016-03-17 00:00:11]

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

[commit-bot: I haz the power, 2016-03-17 00:41:28]

Description was changed from

==========
Use GURLs instead of patterns for SetContentSetting in plugins

Migrate to use SetContentSettingDefaultScope() which takes GURLs only in
plugins.
For special test or specific patterns, remain to use SetContentSetting().

BUG=551747
==========

to

==========
Use GURLs instead of patterns for SetContentSetting in plugins

Migrate to use SetContentSettingDefaultScope() which takes GURLs only in
plugins.
For special test or specific patterns, remain to use SetContentSetting().

BUG=551747
==========

[commit-bot: I haz the power, 2016-03-17 00:41:30]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-17 00:42:24]

Description was changed from

==========
Use GURLs instead of patterns for SetContentSetting in plugins

Migrate to use SetContentSettingDefaultScope() which takes GURLs only in
plugins.
For special test or specific patterns, remain to use SetContentSetting().

BUG=551747
==========

to

==========
Use GURLs instead of patterns for SetContentSetting in plugins

Migrate to use SetContentSettingDefaultScope() which takes GURLs only in
plugins.
For special test or specific patterns, remain to use SetContentSetting().

BUG=551747

Committed: https://crrev.com/38ac96539ffa5fc11696885b375c5a49d80fecae
Cr-Commit-Position: refs/heads/master@{#381609}
==========

[commit-bot: I haz the power, 2016-03-17 00:42:24]

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