Remove unused blink::FeaturePolicy class.

Remove unused blink::FeaturePolicy class.

This is part 5 of 5 in the effort to move the FeaturePolicy implementation
into the content layer, which will facilitate use of the framework for browser-
based policy decisions.

See the other CLs in this series:
[1/5] https://codereview.chromium.org/2648423002/ (Rename classes)
[2/5] https://codereview.chromium.org/2654873004/ (Centralize content-side code)
[3/5] https://codereview.chromium.org/2655663004/ (Maintain parallel FP in browser)
[4/5] https://codereview.chromium.org/2651883008/ (Use content/ FP in blink)
[5/5] (This CL)                                   (Remove unused blink code.)

BUG=685195
Review-Url: https://codereview.chromium.org/2656533004
Cr-Commit-Position: refs/heads/master@{#455152}
Committed: https://chromium.googlesource.com/chromium/src/+/419a628d898182f01fe1974bca078351ef4660cb

[iclelland, 2017-01-25 15:29:52]

Description was changed from

==========
Remove unused blink::FeaturePolicy class

This is part 5 of 5 in the effort to move the FeaturePolicy implementation
into the content layer, which will facilitate use of the framework for browser-
based policy decisions.

See the other CLs in this series:
[1/5] https://codereview.chromium.org/2648423002/ (Rename classes)
[2/5] https://codereview.chromium.org/2654873004/ (Centralize content-side code)
[3/5] https://codereview.chromium.org/2655663004/ (Maintain parallel FP in
browser)
[4/5] https://codereview.chromium.org/2651883008/ (Use content/ FP in blink)
[5/5] (This CL)                                   (Remove unused blink code.)

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

to

==========
Remove unused blink::FeaturePolicy class.

This is part 5 of 5 in the effort to move the FeaturePolicy implementation
into the content layer, which will facilitate use of the framework for browser-
based policy decisions.

See the other CLs in this series:
[1/5] https://codereview.chromium.org/2648423002/ (Rename classes)
[2/5] https://codereview.chromium.org/2654873004/ (Centralize content-side code)
[3/5] https://codereview.chromium.org/2655663004/ (Maintain parallel FP in
browser)
[4/5] https://codereview.chromium.org/2651883008/ (Use content/ FP in blink)
[5/5] (This CL)                                   (Remove unused blink code.)

BUG=685195
==========

[iclelland, 2017-01-25 15:29:52]

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

[iclelland, 2017-01-25 15:30:01]

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

[commit-bot: I haz the power, 2017-01-25 15:30:15]

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, 2017-01-25 16:29:23]

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

[commit-bot: I haz the power, 2017-01-25 16:29:23]

Dry run: Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[iclelland, 2017-02-10 05:39:42]

+r raymes

This one should be the easiest, I hope :)

[raymes, 2017-02-13 02:42:37]

lgtm

https://codereview.chromium.org/2656533004/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/feature_policy/FeaturePolicy.cpp
(right):

https://codereview.chromium.org/2656533004/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/feature_policy/FeaturePolicy.cpp:9: #include
"platform/weborigin/KURL.h"
nit: can we remove this?

https://codereview.chromium.org/2656533004/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/feature_policy/FeaturePolicy.cpp:12: #include
"wtf/text/StringBuilder.h"
nit: can we remove this?

[iclelland, 2017-02-27 18:55:37]

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

[commit-bot: I haz the power, 2017-02-27 18:56:22]

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, 2017-02-27 19:00:24]

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

[commit-bot: I haz the power, 2017-02-27 19:00:25]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[iclelland, 2017-03-03 15:23:47]

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

[commit-bot: I haz the power, 2017-03-03 15:23:57]

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, 2017-03-03 17:02:19]

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

[commit-bot: I haz the power, 2017-03-03 17:02:20]

Dry run: This issue passed the CQ dry run.

[iclelland, 2017-03-07 05:44:49]

On 2017/02/13 02:42:37, raymes wrote:
> lgtm
> 
>
https://codereview.chromium.org/2656533004/diff/20001/third_party/WebKit/Sour...
> File third_party/WebKit/Source/platform/feature_policy/FeaturePolicy.cpp
> (right):
> 
>
https://codereview.chromium.org/2656533004/diff/20001/third_party/WebKit/Sour...
> third_party/WebKit/Source/platform/feature_policy/FeaturePolicy.cpp:9:
#include
> "platform/weborigin/KURL.h"
> nit: can we remove this?
> 
>
https://codereview.chromium.org/2656533004/diff/20001/third_party/WebKit/Sour...
> third_party/WebKit/Source/platform/feature_policy/FeaturePolicy.cpp:12:
#include
> "wtf/text/StringBuilder.h"
> nit: can we remove this?

Done.

[iclelland, 2017-03-07 05:45:08]

iclelland@chromium.org changed reviewers:
+ haraken@chromium.org

[iclelland, 2017-03-07 05:45:08]

+r haraken, can you PTAL as owner?

[haraken, 2017-03-07 06:29:10]

LGTM

[iclelland, 2017-03-07 15:02:19]

The CQ bit was checked by iclelland@chromium.org

[iclelland, 2017-03-07 15:02:20]

The patchset sent to the CQ was uploaded after l-g-t-m from raymes@chromium.org,
haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/2656533004/#ps80001
(title: "Rebase before commit")

[commit-bot: I haz the power, 2017-03-07 15:03:03]

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, 2017-03-07 18:41:02]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1488898939906290,
"parent_rev": "2f9b30be37b33ac03e135564ceaeb2dc327f2c12", "commit_rev":
"419a628d898182f01fe1974bca078351ef4660cb"}

[commit-bot: I haz the power, 2017-03-07 18:41:38]

Description was changed from

==========
Remove unused blink::FeaturePolicy class.

This is part 5 of 5 in the effort to move the FeaturePolicy implementation
into the content layer, which will facilitate use of the framework for browser-
based policy decisions.

See the other CLs in this series:
[1/5] https://codereview.chromium.org/2648423002/ (Rename classes)
[2/5] https://codereview.chromium.org/2654873004/ (Centralize content-side code)
[3/5] https://codereview.chromium.org/2655663004/ (Maintain parallel FP in
browser)
[4/5] https://codereview.chromium.org/2651883008/ (Use content/ FP in blink)
[5/5] (This CL)                                   (Remove unused blink code.)

BUG=685195
==========

to

==========
Remove unused blink::FeaturePolicy class.

This is part 5 of 5 in the effort to move the FeaturePolicy implementation
into the content layer, which will facilitate use of the framework for browser-
based policy decisions.

See the other CLs in this series:
[1/5] https://codereview.chromium.org/2648423002/ (Rename classes)
[2/5] https://codereview.chromium.org/2654873004/ (Centralize content-side code)
[3/5] https://codereview.chromium.org/2655663004/ (Maintain parallel FP in
browser)
[4/5] https://codereview.chromium.org/2651883008/ (Use content/ FP in blink)
[5/5] (This CL)                                   (Remove unused blink code.)

BUG=685195

Review-Url: https://codereview.chromium.org/2656533004
Cr-Commit-Position: refs/heads/master@{#455152}
Committed:
https://chromium.googlesource.com/chromium/src/+/419a628d898182f01fe1974bca07...
==========

[commit-bot: I haz the power, 2017-03-07 18:41:39]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/419a628d898182f01fe1974bca07...