ContentSecurityPolicy: avoid defining static String singletons.

ContentSecurityPolicy: avoid defining static String singletons.

As CSP is used by multiple threads, we cannot define string literals
in terms of DEFINE_STATIC_LOCAL(). Follow what is done elsewhere
for ContentSecurityPolicy and resort to using plain string literals.

R=
BUG=

Committed: https://crrev.com/6824a5fe98ea6501513973db94700085b6b71f22
Cr-Commit-Position: refs/heads/master@{#431956}

[sof, 2016-11-11 15:46:59]

The CQ bit was checked by sigbjornf@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-11-11 15:47:15]

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

[sof, 2016-11-11 15:48:43]

sigbjornf@opera.com changed reviewers:
+ mkwst@chromium.org

[sof, 2016-11-11 15:48:44]

please take a look.

Noticed that
http/tests/fetch/referrer/origin-when-cross-origin-serviceworker-from-document.html
was flakily failing, e.g.,


https://storage.googleapis.com/chromium-layout-test-archives/linux_chromium_r...

[commit-bot: I haz the power, 2016-11-11 17:30:55]

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

[commit-bot: I haz the power, 2016-11-11 17:30:56]

Dry run: This issue passed the CQ dry run.

[sof, 2016-11-11 19:58:43]

Description was changed from

==========
ContentSecurityPolicy: avoid defining static String singletons.

As CSP is used by multiple threads, we cannot define string literals
in terms of DEFINE_STATIC_LOCAL(). Not that there is a clear motivation
for introducing String singletons of the literal strings residing
in .rodata.

R=
BUG=
==========

to

==========
ContentSecurityPolicy: avoid defining static String singletons.

As CSP is used by multiple threads, we cannot define string literals
in terms of DEFINE_STATIC_LOCAL(). Follow what is done elsewhere
for ContentSecurityPolicy and resort to using plain string literals.

R=
BUG=
==========

[Mike West, 2016-11-14 14:01:14]

LGTM, thanks!

[sof, 2016-11-14 18:37:56]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-11-14 18:39:29]

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-11-14 23:16:53]

Description was changed from

==========
ContentSecurityPolicy: avoid defining static String singletons.

As CSP is used by multiple threads, we cannot define string literals
in terms of DEFINE_STATIC_LOCAL(). Follow what is done elsewhere
for ContentSecurityPolicy and resort to using plain string literals.

R=
BUG=
==========

to

==========
ContentSecurityPolicy: avoid defining static String singletons.

As CSP is used by multiple threads, we cannot define string literals
in terms of DEFINE_STATIC_LOCAL(). Follow what is done elsewhere
for ContentSecurityPolicy and resort to using plain string literals.

R=
BUG=
==========

[commit-bot: I haz the power, 2016-11-14 23:16:55]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-14 23:22:56]

Description was changed from

==========
ContentSecurityPolicy: avoid defining static String singletons.

As CSP is used by multiple threads, we cannot define string literals
in terms of DEFINE_STATIC_LOCAL(). Follow what is done elsewhere
for ContentSecurityPolicy and resort to using plain string literals.

R=
BUG=
==========

to

==========
ContentSecurityPolicy: avoid defining static String singletons.

As CSP is used by multiple threads, we cannot define string literals
in terms of DEFINE_STATIC_LOCAL(). Follow what is done elsewhere
for ContentSecurityPolicy and resort to using plain string literals.

R=
BUG=

Committed: https://crrev.com/6824a5fe98ea6501513973db94700085b6b71f22
Cr-Commit-Position: refs/heads/master@{#431956}
==========

[commit-bot: I haz the power, 2016-11-14 23:22:57]

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