Add deprecation warning to requestMediaKeySystemAccess on insecure origins.

Add deprecation warning to requestMediaKeySystemAccess on insecure origins.

For more details, see https://goo.gl/rStTGz and
https://w3c.github.io/encrypted-media/#requestMediaKeySystemAccess.

BUG=481604
TEST=Updated layout test; using requestMediaKeySystemAccess() on HTTP and HTTPS URLs.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=195062

[ddorwin, 2015-05-06 00:00:58]

ddorwin@chromium.org changed reviewers:
+ jrummell@chromium.org, jww@chromium.org, philipj@opera.com

[ddorwin, 2015-05-06 00:00:59]

https://codereview.chromium.org/1125573003/

jww: Please review everything.
jrummell: Please review modules/encryptedmedia.
philipj: Please OWNERS review core/

[ddorwin, 2015-05-06 00:02:17]

That should have said this will land after
https://codereview.chromium.org/1125573003/.

[philipj_slow, 2015-05-06 07:39:44]

core/ LGTM and comments in modules/encryptedmedia/

https://codereview.chromium.org/1125903003/diff/20001/Source/modules/encrypte...
File Source/modules/encryptedmedia/NavigatorRequestMediaKeySystemAccess.cpp
(right):

https://codereview.chromium.org/1125903003/diff/20001/Source/modules/encrypte...
Source/modules/encryptedmedia/NavigatorRequestMediaKeySystemAccess.cpp:181: if
(document->isPrivilegedContext(errorMessage)) {
isPriviledgedContext is on ExecutionContext, so the toDocument() could wait
until step 6 as before.

https://codereview.chromium.org/1125903003/diff/20001/Source/modules/encrypte...
Source/modules/encryptedmedia/NavigatorRequestMediaKeySystemAccess.cpp:185: //
FIXME: Implement the following:
The style recently changed, should now be TODO(ddorwin)

[ddorwin, 2015-05-06 20:41:36]

https://codereview.chromium.org/1125903003/diff/20001/Source/modules/encrypte...
File Source/modules/encryptedmedia/NavigatorRequestMediaKeySystemAccess.cpp
(right):

https://codereview.chromium.org/1125903003/diff/20001/Source/modules/encrypte...
Source/modules/encryptedmedia/NavigatorRequestMediaKeySystemAccess.cpp:181: if
(document->isPrivilegedContext(errorMessage)) {
On 2015/05/06 07:39:43, philipj_UTC2 wrote:
> isPriviledgedContext is on ExecutionContext, so the toDocument() could wait
> until step 6 as before.

Done.

https://codereview.chromium.org/1125903003/diff/20001/Source/modules/encrypte...
Source/modules/encryptedmedia/NavigatorRequestMediaKeySystemAccess.cpp:185: //
FIXME: Implement the following:
On 2015/05/06 07:39:43, philipj_UTC2 wrote:
> The style recently changed, should now be TODO(ddorwin)

Done. Thanks.

[philipj_slow, 2015-05-06 20:53:17]

Changes look good, LGTM for everything.

[jrummell, 2015-05-06 20:58:43]

lgtm

[jww, 2015-05-07 03:05:54]

David, lgtm, although you should try to run
tools/metrics/histograms/update_use_counter_feature_enum.py to update
histograms.xml.

However, there's a good chance you won't be able to commit histograms.xml
because there was a problem with the use counter enums that needs to be fixed.
Thus, if running update_use_counter_feature_enum.py moves any of the other use
counters around, feel free to leave it be, and I'll take care of it in the next
few days.

[ddorwin, 2015-05-07 17:12:43]

On 2015/05/07 03:05:54, jww wrote:
> David, lgtm, although you should try to run
> tools/metrics/histograms/update_use_counter_feature_enum.py to update
> histograms.xml.
> 
> However, there's a good chance you won't be able to commit histograms.xml
> because there was a problem with the use counter enums that needs to be fixed.
> Thus, if running update_use_counter_feature_enum.py moves any of the other use
> counters around, feel free to leave it be, and I'll take care of it in the
next
> few days.

I ran it successfully (I think). I'll upload a CL for it.

[ddorwin, 2015-05-07 17:12:47]

The CQ bit was checked by ddorwin@chromium.org

[ddorwin, 2015-05-07 17:12:48]

The patchset sent to the CQ was uploaded after l-g-t-m from
jrummell@chromium.org, philipj@opera.com
Link to the patchset: https://codereview.chromium.org/1125903003/#ps60001
(title: "rebase")

[commit-bot: I haz the power, 2015-05-07 17:13:20]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1125903003/60001

[commit-bot: I haz the power, 2015-05-07 17:17:38]

Committed patchset #4 (id:60001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=195062