Rebaselining Win10 with download button.

Rebaselining Win10 with media controls download button.

This CL is a follow up to https://codereview.chromium.org/2337013005

NOTRY=true
NOTREECHECKS=true

BUG=601247
Committed: https://crrev.com/e34a6457a83b0d60dda1ce4776e84eeee2e97aa6
Cr-Commit-Position: refs/heads/master@{#419181}

[kdsilva, 2016-09-16 14:49:18]

The CQ bit was checked by kdsilva@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-09-16 14:49:45]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-16 14:52:45]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-16 14:52:46]

Dry run: Try jobs failed on following builders:
  android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  chromeos_x86-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_x86-ge...)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[kdsilva, 2016-09-16 14:54:16]

The CQ bit was checked by kdsilva@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-09-16 14:54:45]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[kdsilva, 2016-09-16 14:56:04]

Description was changed from

==========
Rebaselining Win10 with download button.

BUG=601247
==========

to

==========
Rebaselining Win10 with media controls download button.

This CL is a follow up to https://codereview.chromium.org/2337013005

BUG=601247
==========

[kdsilva, 2016-09-16 15:00:05]

kdsilva@google.com changed reviewers:
+ avayvod@chromium.org, iclelland@chromium.org

[kdsilva, 2016-09-16 15:00:06]

PTAL :)

[kdsilva, 2016-09-16 15:00:43]

Description was changed from

==========
Rebaselining Win10 with media controls download button.

This CL is a follow up to https://codereview.chromium.org/2337013005

BUG=601247
==========

to

==========
Rebaselining Win10 with media controls download button.

This CL is a follow up to https://codereview.chromium.org/2337013005

NOTRY=true
NOTREECHECKS=true

BUG=601247
==========

[kdsilva, 2016-09-16 15:01:00]

The CQ bit was unchecked by kdsilva@google.com

[kdsilva, 2016-09-16 15:01:35]

The CQ bit was checked by kdsilva@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-09-16 15:02:03]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-16 15:06:09]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-16 15:06:10]

Dry run: This issue passed the CQ dry run.

[whywhat, 2016-09-16 15:09:18]

Ian, it seems from the sheriff-o-matic that we need to add some more bots for
this test and another test that's failing?

WebKit Linux
WebKit Linux (dbg)

WebKit Mac10.10
WebKit Mac10.11

virtual/rootlayerscrolls/scrollbars/custom-scrollbar-reconstruction-on-inserting-style-sheet.html

WebKit Win10

And

win10_blink_rel for both tests?

Not familiar with rebaselining and sheriff-o-matic for these tests to know for
sure :)

Otherwise, lgtm so Krittika could submit at her convenience.

[iclelland, 2016-09-16 15:18:43]

On 2016/09/16 15:09:18, whywhat wrote:
> Ian, it seems from the sheriff-o-matic that we need to add some more bots for
> this test and another test that's failing?
> 
> WebKit Linux
> WebKit Linux (dbg)
> 
> WebKit Mac10.10
> WebKit Mac10.11
> 
>
virtual/rootlayerscrolls/scrollbars/custom-scrollbar-reconstruction-on-inserting-style-sheet.html
> 
> WebKit Win10

Yes, I think that one should just have the specific platforms removed from the
expectation. It looks like it was a merge conflict with
https://chromium.googlesource.com/chromium/src/+/1fd6bb0620f889ba8dd8a9b620b8...
from yesterday.

> 
> And
> 
> win10_blink_rel for both tests?
> 
> Not familiar with rebaselining and sheriff-o-matic for these tests to know for
> sure :)
> 
> Otherwise, lgtm so Krittika could submit at her convenience.

I think just "Win" should be sufficient to match any and all windows builders
for media-zoom-controls.

Thanks!

[kdsilva, 2016-09-16 15:48:22]

The CQ bit was checked by kdsilva@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-09-16 15:48:46]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-16 15:53:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-16 15:53:07]

Dry run: This issue passed the CQ dry run.

[kdsilva, 2016-09-16 15:55:59]

The CQ bit was checked by kdsilva@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-09-16 15:56:17]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-16 16:00:15]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-16 16:00:16]

Dry run: This issue passed the CQ dry run.

[kdsilva, 2016-09-16 16:03:22]

The CQ bit was checked by kdsilva@google.com

[kdsilva, 2016-09-16 16:03:22]

The patchset sent to the CQ was uploaded after l-g-t-m from avayvod@chromium.org
Link to the patchset: https://codereview.chromium.org/2343133002/#ps80001
(title: "rebased")

[commit-bot: I haz the power, 2016-09-16 16:03:57]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-16 16:08:39]

Description was changed from

==========
Rebaselining Win10 with media controls download button.

This CL is a follow up to https://codereview.chromium.org/2337013005

NOTRY=true
NOTREECHECKS=true

BUG=601247
==========

to

==========
Rebaselining Win10 with media controls download button.

This CL is a follow up to https://codereview.chromium.org/2337013005

NOTRY=true
NOTREECHECKS=true

BUG=601247
==========

[commit-bot: I haz the power, 2016-09-16 16:08:40]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-09-16 16:12:21]

Description was changed from

==========
Rebaselining Win10 with media controls download button.

This CL is a follow up to https://codereview.chromium.org/2337013005

NOTRY=true
NOTREECHECKS=true

BUG=601247
==========

to

==========
Rebaselining Win10 with media controls download button.

This CL is a follow up to https://codereview.chromium.org/2337013005

NOTRY=true
NOTREECHECKS=true

BUG=601247

Committed: https://crrev.com/e34a6457a83b0d60dda1ce4776e84eeee2e97aa6
Cr-Commit-Position: refs/heads/master@{#419181}
==========

[commit-bot: I haz the power, 2016-09-16 16:12:22]

Patchset 5 (id:??) landed as
https://crrev.com/e34a6457a83b0d60dda1ce4776e84eeee2e97aa6
Cr-Commit-Position: refs/heads/master@{#419181}