Add SLOW expectation for virtual/spv2/fast/overflow/lots-of-sibling-inline-boxes.html

Add SLOW expectation for virtual/spv2/fast/overflow/lots-of-sibling-inline-boxes.html

BUG=594189
NOTRY=true

Committed: https://crrev.com/658061b938c58c9ea432bbf293b3a9d34a1c5457
Cr-Commit-Position: refs/heads/master@{#380697}

[alexmos, 2016-03-11 18:32:18]

alexmos@chromium.org changed reviewers:
+ lukasza@chromium.org

[alexmos, 2016-03-11 18:33:15]

Description was changed from

==========
Add SLOW expectation for
virtual/spv2/fast/overflow/lots-of-sibling-inline-boxes.html

BUG=594189
==========

to

==========
Add SLOW expectation for
virtual/spv2/fast/overflow/lots-of-sibling-inline-boxes.html

BUG=594189
NOTRY=true
==========

[Łukasz Anforowicz, 2016-03-11 19:11:41]

LGTM

I was a bit surprised that non-virtual annotations have no impact on virtual
test suites, but I see that similarly duplicated expectations are already
present in other cases - i.e.:

crbug.com/420008 inspector/tracing/ [ Slow ]
crbug.com/420008 virtual/threaded/inspector/tracing/ [ Slow ]

Maybe it would be worth opening a separate bug to ask around if the virtual
suites should automatically inherit expectations from their non-virtual
counterparts?

[alexmos, 2016-03-11 19:19:53]

On 2016/03/11 19:11:41, Łukasz Anforowicz wrote:
> LGTM
> 
> I was a bit surprised that non-virtual annotations have no impact on virtual
> test suites, but I see that similarly duplicated expectations are already
> present in other cases - i.e.:
> 
> crbug.com/420008 inspector/tracing/ [ Slow ]
> crbug.com/420008 virtual/threaded/inspector/tracing/ [ Slow ]
> 
> Maybe it would be worth opening a separate bug to ask around if the virtual
> suites should automatically inherit expectations from their non-virtual
> counterparts?

Thanks!  Yes, that sounds worth doing - I'll follow up and file a bug.

[alexmos, 2016-03-11 19:19:58]

The CQ bit was checked by alexmos@chromium.org

[commit-bot: I haz the power, 2016-03-11 19:20:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1786763002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1786763002/1

[commit-bot: I haz the power, 2016-03-11 19:29:10]

Committed patchset #1 (id:1)

[alexmos, 2016-03-11 19:30:45]

On 2016/03/11 19:19:53, alexmos wrote:
> On 2016/03/11 19:11:41, Łukasz Anforowicz wrote:
> > LGTM
> > 
> > I was a bit surprised that non-virtual annotations have no impact on virtual
> > test suites, but I see that similarly duplicated expectations are already
> > present in other cases - i.e.:
> > 
> > crbug.com/420008 inspector/tracing/ [ Slow ]
> > crbug.com/420008 virtual/threaded/inspector/tracing/ [ Slow ]
> > 
> > Maybe it would be worth opening a separate bug to ask around if the virtual
> > suites should automatically inherit expectations from their non-virtual
> > counterparts?
> 
> Thanks!  Yes, that sounds worth doing - I'll follow up and file a bug.

Filed https://crbug.com/594216

[commit-bot: I haz the power, 2016-03-11 19:31:14]

Description was changed from

==========
Add SLOW expectation for
virtual/spv2/fast/overflow/lots-of-sibling-inline-boxes.html

BUG=594189
NOTRY=true
==========

to

==========
Add SLOW expectation for
virtual/spv2/fast/overflow/lots-of-sibling-inline-boxes.html

BUG=594189
NOTRY=true

Committed: https://crrev.com/658061b938c58c9ea432bbf293b3a9d34a1c5457
Cr-Commit-Position: refs/heads/master@{#380697}
==========

[commit-bot: I haz the power, 2016-03-11 19:31:14]

Patchset 1 (id:??) landed as
https://crrev.com/658061b938c58c9ea432bbf293b3a9d34a1c5457
Cr-Commit-Position: refs/heads/master@{#380697}