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

Issue 729098 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Sep 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Changing video_MultiplePlayback suit to bvt_perbuild

Project Member Reported by avkodipelli@chromium.org, Jun 2 2017

Issue description

As video_MultiplePlayback is currently in Experimental suit and it is no longer available. We have changed suit to bvt_perbuild using CL https://chromium-review.googlesource.com/c/521783/

Creating this bug for merging this fix into M60 and M59.


 
Owner: vsu...@chromium.org
Labels: Merge-Request-59 Merge-Request-60
Project Member

Comment 3 by sheriffbot@chromium.org, Jun 2 2017

Labels: -Merge-Request-59 Merge-Review-59 Hotlist-Merge-Review
This bug requires manual review: We are only 3 days from stable.
Please contact the milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), gkihumba@(ChromeOS), Abdul Syed@(Desktop)

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

Comment 4 by sheriffbot@chromium.org, Jun 3 2017

Labels: -Merge-Request-60 Hotlist-Merge-Approved Merge-Approved-60
Your change meets the bar and is auto-approved for M60. Please go ahead and merge the CL to branch 3112 manually. Please contact milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), josafat@(ChromeOS), bustamante@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: Merge-Approved-59
Labels: -Merge-Review-59
Status: Assigned (was: Untriaged)
vsuley@, ping for merging this approved issue
Project Member

Comment 7 by sheriffbot@chromium.org, Jun 9 2017

Cc: gkihumba@google.com
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 8 by sheriffbot@chromium.org, Jun 12 2017

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 9 by bugdroid1@chromium.org, Jun 14 2017

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

commit a5894f3a5165496f43e59214a8040c2601619d40
Author: Vinayak Suley <vsuley@chromium.org>
Date: Wed Jun 14 00:44:45 2017

Assigning video_MultiplePlayback to perbuild suite.

The experimental suite has been retired. Moving this test
to run in bvt-perbuild since we want it to run but not block
the build.

BUG= 729098 
TEST=None

Change-Id: I98ac68708285493a48973ba2c86c8a03dbdd3f08
Reviewed-on: https://chromium-review.googlesource.com/521783
Commit-Ready: Vinayak Suley <vsuley@chromium.org>
Tested-by: Vinayak Suley <vsuley@chromium.org>
Reviewed-by: Rohit Makasana <rohitbm@chromium.org>
(cherry picked from commit 34ae222c3d184602d4e4794a4e6a4bac91768a64)
Reviewed-on: https://chromium-review.googlesource.com/531648
Reviewed-by: Vinayak Suley <vsuley@chromium.org>
Commit-Queue: Vinayak Suley <vsuley@chromium.org>
Trybot-Ready: Vinayak Suley <vsuley@chromium.org>

[modify] https://crrev.com/a5894f3a5165496f43e59214a8040c2601619d40/client/site_tests/video_MultiplePlayback/control

Project Member

Comment 10 by bugdroid1@chromium.org, Jun 14 2017

Labels: merge-merged-release-R60-9592.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/4bee1ad9daf3c1d22ea609574699f630135c9d4e

commit 4bee1ad9daf3c1d22ea609574699f630135c9d4e
Author: Vinayak Suley <vsuley@chromium.org>
Date: Wed Jun 14 00:44:54 2017

Assigning video_MultiplePlayback to perbuild suite.

The experimental suite has been retired. Moving this test
to run in bvt-perbuild since we want it to run but not block
the build.

BUG= 729098 
TEST=None

Change-Id: I98ac68708285493a48973ba2c86c8a03dbdd3f08
Reviewed-on: https://chromium-review.googlesource.com/521783
Commit-Ready: Vinayak Suley <vsuley@chromium.org>
Tested-by: Vinayak Suley <vsuley@chromium.org>
Reviewed-by: Rohit Makasana <rohitbm@chromium.org>
(cherry picked from commit 34ae222c3d184602d4e4794a4e6a4bac91768a64)
Reviewed-on: https://chromium-review.googlesource.com/531649
Reviewed-by: Vinayak Suley <vsuley@chromium.org>
Commit-Queue: Vinayak Suley <vsuley@chromium.org>
Trybot-Ready: Vinayak Suley <vsuley@chromium.org>

[modify] https://crrev.com/4bee1ad9daf3c1d22ea609574699f630135c9d4e/client/site_tests/video_MultiplePlayback/control

Labels: -Hotlist-Merge-Review
Project Member

Comment 12 by sheriffbot@chromium.org, Jul 28 2017

Labels: -Merge-Approved-59 -Merge-Approved-60
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: Verified (was: Assigned)
Already merged.

Sign in to add a comment