Issue metadata
Sign in to add a comment
|
video_YouTubeHTML5 is failing |
||||||||||||||||||||||
Issue descriptionvideo_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.
,
Apr 6 2016
Looks like an YouTube API issue which started on 04/01. Can you file a b/ ?
,
Apr 6 2016
Created bug b/28041656.
,
Apr 12 2016
,
May 6 2016
Rohit. YouTube people do not think a change breaks autoplay. Can we disable the tests for now?
,
May 6 2016
Avinash, please remove the suite part from the control files.
,
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
,
May 24 2016
video_YouTubeFlash is being removed. https://chromium-review.googlesource.com/#/c/346755/1 https://chromium-review.googlesource.com/#/c/346756/1
,
May 24 2016
,
May 24 2016
A question: if these tests aren't running, how do they affect?
,
May 31 2016
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?
,
May 31 2016
I discussed with Pawel. Let's remove this test. Owen. Please help removing this.
,
May 31 2016
Rohit said this test was still useful. Assigning the bug to him and removed the milestone.
,
May 31 2016
Thanks. We will check if we can use normal <video> JS for YouTube video testing.
,
Jun 28 2016
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.
,
Jun 28 2016
,
Jun 28 2016
,
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.
,
Jun 29 2016
Also tools/perf/page_sets/data, I think that is the original location.
,
Jun 29 2016
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.
,
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.
,
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
,
Jul 20 2016
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?
,
Jul 20 2016
Your change meets the bar and is auto-approved for M53 (branch: 2785)
,
Jul 20 2016
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
,
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
,
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
,
Sep 7 2016
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
,
Oct 4 2016
,
Oct 28 2016
As per wmatrix results : https://wmatrix.googleplex.com/unfiltered?hide_missing=True&tests=video_YouTubeHTML5 |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by avkodipelli@chromium.org
, Apr 5 2016