New issue
Advanced search Search tips

Issue 730762 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 403462



Sign in to add a comment

ECKEncryptedMediaTest.Renewal test fails with mojo CDM

Project Member Reported by xhw...@chromium.org, Jun 7 2017

Issue description

[ RUN      ] Mojo/ECKEncryptedMediaTest.Renewal/0
[22223:22223:0607/123352.556442:WARNING:password_store_factory.cc(250)] Using basic (unencrypted) st
ore for password storage. See https://chromium.googlesource.com/chromium/src/+/master/docs/linux_pas
sword_storage.md for more information about password storage options.
[22223:22223:0607/123356.671719:INFO:media_browsertest.cc(46)] 12:33:56.671 Starting HTTP server
[22223:22223:0607/123356.677112:INFO:media_browsertest.cc(62)] 12:33:56.677 Running test URL: http:/
/127.0.0.1:44531/eme_player.html?mediaFile=bear-320x240-v_enc-v.webm&mediaType=video/webm;%20codecs=
%22vp8%22&keySystem=org.chromium.externalclearkey.renewal&useMSE=1
[22223:22332:0607/123400.649074:WARNING:simple_synchronous_entry.cc(1102)] Could not open platform f
iles for entry.
[1:1:0607/123400.746239:ERROR:render_process_impl.cc(171)] WebFrame LEAKED 1 TIMES
[22223:22223:0607/123401.036594:INFO:CONSOLE(270)] "12:34:01.033 -  Using ClearKeyPlayer", source: h
ttp://127.0.0.1:44531/eme_player_js/utils.js (270)
[22223:22223:0607/123401.039948:INFO:CONSOLE(270)] "12:34:01.037 -  Registering video event handlers
.", source: http://127.0.0.1:44531/eme_player_js/utils.js (270)
[22223:22223:0607/123401.041687:INFO:CONSOLE(270)] "12:34:01.038 -  Setting video media keys: org.ch
romium.externalclearkey.renewal", source: http://127.0.0.1:44531/eme_player_js/utils.js (270)
[22223:22315:0607/123401.498716:WARNING:embedded_test_server.cc(219)] Request not handled. Returning
 404: /favicon.ico
[22223:22223:0607/123402.034974:INFO:CONSOLE(270)] "12:34:02.024 -  heartbeat #1", source: http://12
7.0.0.1:44531/eme_player_js/utils.js (270)
[22223:22223:0607/123402.120645:INFO:CONSOLE(270)] "12:34:02.110 -  Loading media using MSE.", sourc
e: http://127.0.0.1:44531/eme_player_js/utils.js (270)
[22223:22223:0607/123402.257523:INFO:CONSOLE(270)] "12:34:02.249 -  video.src has been set to blob:h
ttp://127.0.0.1:44531/a40dcae1-f72e-43e2-a7ea-0d58cf431ca9", source: http://127.0.0.1:44531/eme_play
er_js/utils.js (270)
[22223:22223:0607/123402.263799:INFO:CONSOLE(270)] "12:34:02.255 -  initEMEPlayer() done", source: h
ttp://127.0.0.1:44531/eme_player_js/utils.js (270)
[22223:22223:0607/123402.270452:INFO:CONSOLE(270)] "12:34:02.261 -  Media player created.", source: 
http://127.0.0.1:44531/eme_player_js/utils.js (270)
[22223:22223:0607/123402.278692:INFO:CONSOLE(270)] "12:34:02.269 -  Starting play, hidden: false", s
ource: http://127.0.0.1:44531/eme_player_js/utils.js (270)
[22223:22223:0607/123402.374502:INFO:CONSOLE(270)] "12:34:02.368 -  Event: play", source: http://127
.0.0.1:44531/eme_player_js/utils.js (270)
[22223:22223:0607/123402.379074:INFO:CONSOLE(270)] "12:34:02.373 -  Event: waiting", source: http://
127.0.0.1:44531/eme_player_js/utils.js (270)
[22223:22223:0607/123402.402140:INFO:CONSOLE(270)] "12:34:02.394 -  onSourceOpen [object Event]", so
urce: http://127.0.0.1:44531/eme_player_js/utils.js (270)
[22223:22223:0607/123402.407037:INFO:CONSOLE(270)] "12:34:02.400 -  Creating a source buffer for typ
e video/webm; codecs="vp8"", source: http://127.0.0.1:44531/eme_player_js/utils.js (270)
[22335:22335:0607/123402.695874:ERROR:ffmpeg_cdm_video_decoder.cc(171)] Initialize(): avcodec_find_d
ecoder failed.
[22223:22223:0607/123402.708653:INFO:CONSOLE(270)] "12:34:02.701 -  Creating new media key session f
or initDataType: webm, initData: 30313233343536373839303132333435", source: http://127.0.0.1:44531/eme_player_js/utils.js (270)
[22223:22223:0607/123402.745849:INFO:CONSOLE(270)] "12:34:02.737 -  End of appending buffer from bear-320x240-v_enc-v.webm", source: http://127.0.0.1:44531/eme_player_js/utils.js (270)
[22223:22223:0607/123402.764490:INFO:CONSOLE(270)] "12:34:02.754 -  MediaKeySession onMessage [object MediaKeyMessageEvent]", source: http://127.0.0.1:44531/eme_player_js/utils.js (270)
[22223:22223:0607/123402.777555:INFO:CONSOLE(270)] "12:34:02.769 -  Calling update: {"keys":[{"kty":"oct","alg":"A128KW","kid":"MDEyMzQ1Njc4OTAxMjM0NQ","k":"691i8WgU0nto7xIq_OSuPA"}]}", source: http://127.0.0.1:44531/eme_player_js/utils.js (270)
[22223:22223:0607/123402.792297:INFO:CONSOLE(270)] "12:34:02.783 -  Unexpected expiration:  NaN", source: http://127.0.0.1:44531/eme_player_js/utils.js (270)
[22223:22223:0607/123402.800834:INFO:CONSOLE(145)] "FAIL: ReferenceError error is not defined", source: http://127.0.0.1:44531/eme_player_js/utils.js (145)
../../chrome/browser/media/media_browsertest.cc:57: Failure
      Expected: expected_title
      Which is: "ENDED"
To be equal to: final_title
      Which is: "EME_UPDATE_FAILED"
[  FAILED  ] Mojo/ECKEncryptedMediaTest.Renewal/0, where GetParam() = 4-byte object <01-00 00-00> (13888 ms)
[----------] 1 test from Mojo/ECKEncryptedMediaTest (13889 ms total)

[----------] Global test environment tear-down
[==========] 1 test from 1 test case ran. (13891 ms total)
[  PASSED  ] 0 tests.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] Mojo/ECKEncryptedMediaTest.Renewal/0, where GetParam() = 4-byte object <01-00 00-00>

 1 FAILED TEST

 
Blocking: 403462
Labels: Proj-Sodo
I guess the root cause is the expiration time conversion:

[22223:22223:0607/123402.792297:INFO:CONSOLE(270)] "12:34:02.783 -  Unexpected expiration:  NaN", source: 
Project Member

Comment 2 by bugdroid1@chromium.org, Jul 12 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b3db7e610c7fc02f50b011182621271beaae3464

commit b3db7e610c7fc02f50b011182621271beaae3464
Author: Xiaohan Wang <xhwang@chromium.org>
Date: Wed Jul 12 22:24:51 2017

media: Enable external clear key renewal test using mojo CDM

According to the EME spec, during update(), expiration should be updated
before resolving the update() promise.

BUG= 730762 
TEST=Fixes and enables a test case.

Change-Id: I145f707aee8cbc53413417654282dafe6edc9f28
Reviewed-on: https://chromium-review.googlesource.com/567873
Reviewed-by: John Rummell <jrummell@chromium.org>
Commit-Queue: Xiaohan Wang <xhwang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#486137}
[modify] https://crrev.com/b3db7e610c7fc02f50b011182621271beaae3464/chrome/browser/media/encrypted_media_browsertest.cc
[modify] https://crrev.com/b3db7e610c7fc02f50b011182621271beaae3464/media/cdm/ppapi/external_clear_key/clear_key_cdm.cc
[modify] https://crrev.com/b3db7e610c7fc02f50b011182621271beaae3464/media/mojo/clients/mojo_decryptor.cc
[modify] https://crrev.com/b3db7e610c7fc02f50b011182621271beaae3464/media/mojo/services/mojo_cdm_service.cc
[modify] https://crrev.com/b3db7e610c7fc02f50b011182621271beaae3464/media/mojo/services/mojo_decryptor_service.cc

Comment 3 by xhw...@chromium.org, Jul 12 2017

Status: Fixed (was: Assigned)

Sign in to add a comment