New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 600757 link

Starred by 4 users

Issue metadata

Status: Verified
Owner:
Closed: Oct 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug-Regression



Sign in to add a comment

video_YouTubeHTML5 is failing

Project Member Reported by avkodipelli@chromium.org, Apr 5 2016

Issue description

video_YouTubeHTML5 and video_YouTubeFlash are failing  are failing on TOT.

https://wmatrix.googleplex.com/unfiltered?hide_missing=True&tests=video_YouTubeHTML5

https://wmatrix.googleplex.com/unfiltered?hide_missing=True&tests=video_YouTubeFlash 

With error : Video failed to load. Player expected status: Playing and current status: Unstarted.

 
Observed issue on M50 and M49 also. 
Labels: -Pri-1 Pri-2
Looks like an YouTube API issue which started on 04/01. Can you file a b/ ?
Created bug b/28041656.
Status: WontFix (was: Untriaged)
YT bug tracked in b/28041656. close here.
Owner: rohi...@chromium.org
Status: Assigned (was: WontFix)
Rohit. YouTube people do not think a change breaks autoplay. Can we disable the tests for now?
Owner: avkodipelli@chromium.org
Avinash, please remove the suite part from the control files.
Project Member

Comment 7 by bugdroid1@chromium.org, May 11 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/5cd5f6607c66b9ad01c1405ba40a733860c76a26

commit 5cd5f6607c66b9ad01c1405ba40a733860c76a26
Author: avinash kodipelli <avkodipelli@chromium.org>
Date: Mon May 09 21:49:40 2016

Removing bvt suit attribute.

Removing bvt-perbuild suit attribute as there few issues while running the test.

BUG= chromium:600757 
TEST=manual

Change-Id: I9a52619e387ea529e9be0f0c6da3a291c6d63368
Reviewed-on: https://chromium-review.googlesource.com/343535
Reviewed-by: Rohit Makasana <rohitbm@chromium.org>
Tested-by: Avinash Kodipelli <avkodipelli@chromium.org>
Commit-Queue: Avinash Kodipelli <avkodipelli@chromium.org>

[modify] https://crrev.com/5cd5f6607c66b9ad01c1405ba40a733860c76a26/client/site_tests/video_YouTubeFlash/control
[modify] https://crrev.com/5cd5f6607c66b9ad01c1405ba40a733860c76a26/client/site_tests/video_YouTubeHTML5/control

Labels: VideoShortList
A question: if these tests aren't running, how do they affect?
Labels: -M-51 M-53
Summary: video_YouTubeHTML5 are failing (was: video_YouTubeHTML5 and video_YouTubeFlash are failing )
Not much impact. video_VideoDecodeAccelerator and video_VideoEncodeAccelerator are more important. It looks like video_YouTubeHTML5 was disabled and not run. Any concern removing it, too?
Owner: owenlin@chromium.org
I discussed with Pawel. Let's remove this test. Owen. Please help removing this.
Labels: -VideoShortList -M-53
Owner: rohi...@chromium.org
Rohit said this test was still useful. Assigning the bug to him and removed the milestone.
Thanks. We will check if we can use normal <video> JS for YouTube video testing.
Labels: M-54
Owner: vsu...@chromium.org
Summary: video_YouTubeHTML5 is failing (was: video_YouTubeHTML5 are failing )
 Issue 623470  would have been caught if we would have this test running.

Vin, YouTube player uses HTML5 video. Could you check if the player can be controlled using JS so we don't need to rely on YouTube APIs which often breaks.
Cc: -mu...@chromium.org
Components: -Internals>Media>Video OS>Kernel>Video

Comment 18 by ihf@chromium.org, Jun 29 2016

I think what we want is invest in using webpage replay. In this case a wpr of youtube. Can you investigate how to do this
https://www.chromium.org/developers/telemetry/record_a_page_set

There may actually be already some captures, you need an internal Chrome checkout
chrome/test/chromeos/autotest/files/client/deps/telemetry_dep/test_src/tools/perf/page_sets/data/
but I don't see youtube right now. Please check the discussion on issue 618698.


Comment 19 by ihf@chromium.org, Jun 29 2016

Also tools/perf/page_sets/data, I think that is the original location.
We already have a Vimeo video test using wpr and it was having a similar pain in maintaining the test as wpr is in maintenance mode and it was hard get someone even to look at the problem on priority.

YouTube tests are high priority tests and if they are failing for a reason, we can't wait for another 2-3 weeks for someone to take a look at failures.

Our experience says that We never had problem a problem with the online YouTube video for testing.

Comment 21 by ihf@chromium.org, Jun 29 2016

Didn't know that wpr is not really supported anymore. That would be sad. Well, I guess with youtube it is somewhat reasonable to use them directly.
Project Member

Comment 22 by bugdroid1@chromium.org, Jul 12 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/c5996fbae3c6c743b1c22000c210c509cadf653b

commit c5996fbae3c6c743b1c22000c210c509cadf653b
Author: Vinayak Suley <vsuley@chromium.org>
Date: Mon Jul 11 22:30:41 2016

Enable YouTube HTML5 test.

It seems like the YouTube HTML5 test is working now. We don't know what
change made the scenario pass again (likely external), but we want to
start running this test in the BVT-perbuild suite so we can start
monitoring it.

BUG= chromium:600757 
TEST=None

Change-Id: I45b8ec376b59c4ec54b75cfc8155c7769b08619e
Reviewed-on: https://chromium-review.googlesource.com/359940
Reviewed-by: Rohit Makasana <rohitbm@chromium.org>
Commit-Queue: Vinayak Suley <vsuley@chromium.org>
Tested-by: Vinayak Suley <vsuley@chromium.org>

[modify] https://crrev.com/c5996fbae3c6c743b1c22000c210c509cadf653b/client/site_tests/video_YouTubeHTML5/control

Cc: keta...@chromium.org avkodipelli@chromium.org
Labels: M-53 Merge-Request-53
We've been running this test in the lab for a little while now and it's doing quite well. Merging this back into M-53 would help cut down some manual testing, may I have branch TPM approval please? 

Comment 24 by dimu@google.com, Jul 20 2016

Labels: -Merge-Request-53 Merge-Approved-53 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M53 (branch: 2785)
Project Member

Comment 25 by bugdroid1@chromium.org, Jul 20 2016

Labels: merge-merged-release-R53-8530.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/e4019de7b5674c2eaea4e3a6cad1d303dce76980

commit e4019de7b5674c2eaea4e3a6cad1d303dce76980
Author: Vinayak Suley <vsuley@chromium.org>
Date: Mon Jul 11 22:30:41 2016

Enable YouTube HTML5 test.

It seems like the YouTube HTML5 test is working now. We don't know what
change made the scenario pass again (likely external), but we want to
start running this test in the BVT-perbuild suite so we can start
monitoring it.

BUG= chromium:600757 
TEST=None

Change-Id: I45b8ec376b59c4ec54b75cfc8155c7769b08619e
Previous-Reviewed-on: https://chromium-review.googlesource.com/359940
(cherry picked from commit 1980f9d3562ecc5e01c15f32f5d3359fbc5fddff)
Reviewed-on: https://chromium-review.googlesource.com/361961
Reviewed-by: Vinayak Suley <vsuley@chromium.org>
Commit-Queue: Vinayak Suley <vsuley@chromium.org>
Tested-by: Vinayak Suley <vsuley@chromium.org>
Reviewed-by: Rohit Makasana <rohitbm@chromium.org>

[modify] https://crrev.com/e4019de7b5674c2eaea4e3a6cad1d303dce76980/client/site_tests/video_YouTubeHTML5/control

Project Member

Comment 26 by sheriffbot@chromium.org, Jul 23 2016

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 27 by sheriffbot@chromium.org, Jul 26 2016

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 28 by sheriffbot@chromium.org, Sep 7 2016

Labels: -Merge-Approved-53
This issue hasn't been updated in the last 6 weeks, so removing its merge approval label. Please re-request a merge if needed.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: Fixed (was: Assigned)
Status: Verified (was: Fixed)
As per wmatrix results : https://wmatrix.googleplex.com/unfiltered?hide_missing=True&tests=video_YouTubeHTML5

Sign in to add a comment