Bug 151287

Summary: Add a test for media control dropoff
Product: WebKit Reporter: Jon Lee <jonlee>
Component: MediaAssignee: Eric Carlson <eric.carlson>
Status: RESOLVED FIXED    
Severity: Normal CC: buildbot, commit-queue, dino, eric.carlson, rniwa, ryanhaddad, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=162494
Bug Depends on: 151446, 159514, 159522    
Bug Blocks: 151067    
Attachments:
Description Flags
Patch
none
Archive of layout-test-results from ews116 for mac-yosemite
none
Archive of layout-test-results from ews104 for mac-mavericks-wk2
none
Archive of layout-test-results from ews102 for mac-mavericks
none
Patch
none
Patch
none
Patch
none
Patch
none
Archive of layout-test-results from ews101 for mac-mavericks
none
Archive of layout-test-results from ews106 for mac-mavericks-wk2
none
Archive of layout-test-results from ews116 for mac-yosemite
none
Patch
none
Patch
none
Patch
none
Proposed patch.
none
Archive of layout-test-results from ews102 for mac-yosemite
none
Archive of layout-test-results from ews104 for mac-yosemite-wk2
none
Archive of layout-test-results from ews112 for mac-yosemite
none
Updated patch for the bots.
buildbot: commit-queue-
Archive of layout-test-results from ews101 for mac-yosemite
none
Archive of layout-test-results from ews107 for mac-yosemite-wk2
none
Archive of layout-test-results from ews112 for mac-yosemite
none
Another updated patch for the bots.
buildbot: commit-queue-
Archive of layout-test-results from ews102 for mac-yosemite
none
Archive of layout-test-results from ews107 for mac-yosemite-wk2
none
Archive of layout-test-results from ews117 for mac-yosemite
none
Another updated patch for the bots.
buildbot: commit-queue-
Archive of layout-test-results from ews102 for mac-yosemite
none
Archive of layout-test-results from ews106 for mac-yosemite-wk2
none
One more time for the bots!
none
Updated patch.
none
Updated. none

Description Jon Lee 2015-11-13 17:40:42 PST
Add a test for keeping track of when media controls drop off based on the width of the media element.
Comment 1 Radar WebKit Bug Importer 2015-11-13 17:41:08 PST
<rdar://problem/23544666>
Comment 2 Jon Lee 2015-11-13 17:53:28 PST
Created attachment 265521 [details]
Patch
Comment 3 Build Bot 2015-11-13 18:38:43 PST
Comment on attachment 265521 [details]
Patch

Attachment 265521 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/425840

New failing tests:
media/controls/inline-elements-dropoff-order.html
Comment 4 Build Bot 2015-11-13 18:38:45 PST
Created attachment 265526 [details]
Archive of layout-test-results from ews116 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews116  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 5 Build Bot 2015-11-13 20:35:17 PST
Comment on attachment 265521 [details]
Patch

Attachment 265521 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/426660

New failing tests:
media/controls/inline-elements-dropoff-order.html
Comment 6 Build Bot 2015-11-13 20:35:19 PST
Created attachment 265531 [details]
Archive of layout-test-results from ews104 for mac-mavericks-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews104  Port: mac-mavericks-wk2  Platform: Mac OS X 10.9.5
Comment 7 Build Bot 2015-11-13 20:50:35 PST
Comment on attachment 265521 [details]
Patch

Attachment 265521 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/426718

New failing tests:
media/controls/inline-elements-dropoff-order.html
Comment 8 Build Bot 2015-11-13 20:50:37 PST
Created attachment 265532 [details]
Archive of layout-test-results from ews102 for mac-mavericks

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-mavericks  Platform: Mac OS X 10.9.5
Comment 9 Dean Jackson 2015-11-13 21:14:16 PST
Comment on attachment 265521 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=265521&action=review

I didn't check why the tests are failing.

> LayoutTests/media/controls/inline-elements-dropoff-order.html:65
> +                testsFinished = function() {
> +                    runAirPlayTests();
> +                };

Do you declare testsFinished as a global variable somewhere? This works as is, but might be nicer if we make it a global var.

> LayoutTests/media/controls/inline-elements-dropoff-order.html:112
> +                internals.setMockMediaPlaybackTargetPickerState("Sleepy TV", "DeviceAvailable");

Have we standardized on Sleepy TV? :)
Comment 10 Jon Lee 2015-11-14 18:51:41 PST
Need to adjust the test for OS's with no wireless playback support.
Comment 11 Jon Lee 2015-11-14 18:53:03 PST
(In reply to comment #9)
> Comment on attachment 265521 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=265521&action=review
> 
> I didn't check why the tests are failing.

Older Mac OSes don't support wireless playback.

> 
> > LayoutTests/media/controls/inline-elements-dropoff-order.html:65
> > +                testsFinished = function() {
> > +                    runAirPlayTests();
> > +                };
> 
> Do you declare testsFinished as a global variable somewhere? This works as
> is, but might be nicer if we make it a global var.
> 
> > LayoutTests/media/controls/inline-elements-dropoff-order.html:112
> > +                internals.setMockMediaPlaybackTargetPickerState("Sleepy TV", "DeviceAvailable");
> 
> Have we standardized on Sleepy TV? :)

Why not? :) I'll change it to something more professional
Comment 12 Jon Lee 2015-11-14 18:59:53 PST
(In reply to comment #9)
> Comment on attachment 265521 [details]
> Patch

> > LayoutTests/media/controls/inline-elements-dropoff-order.html:65
> > +                testsFinished = function() {
> > +                    runAirPlayTests();
> > +                };
> 
> Do you declare testsFinished as a global variable somewhere? This works as
> is, but might be nicer if we make it a global var.

I'll pull the global vars out and declare them for clarity.
Comment 13 Jon Lee 2015-11-14 21:00:41 PST
Created attachment 265550 [details]
Patch
Comment 14 Jon Lee 2015-11-14 21:29:41 PST
Created attachment 265551 [details]
Patch
Comment 15 Jon Lee 2015-11-14 21:50:40 PST
Created attachment 265552 [details]
Patch
Comment 16 Jon Lee 2015-11-14 21:52:18 PST
Created attachment 265553 [details]
Patch
Comment 17 Build Bot 2015-11-14 22:38:42 PST
Comment on attachment 265553 [details]
Patch

Attachment 265553 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/431330

New failing tests:
media/controls/inline-elements-dropoff-order.html
Comment 18 Build Bot 2015-11-14 22:38:45 PST
Created attachment 265554 [details]
Archive of layout-test-results from ews101 for mac-mavericks

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews101  Port: mac-mavericks  Platform: Mac OS X 10.9.5
Comment 19 Build Bot 2015-11-14 22:42:13 PST
Comment on attachment 265553 [details]
Patch

Attachment 265553 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/431332

New failing tests:
media/controls/inline-elements-dropoff-order.html
Comment 20 Build Bot 2015-11-14 22:42:17 PST
Created attachment 265555 [details]
Archive of layout-test-results from ews106 for mac-mavericks-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-mavericks-wk2  Platform: Mac OS X 10.9.5
Comment 21 Build Bot 2015-11-14 22:43:42 PST
Comment on attachment 265553 [details]
Patch

Attachment 265553 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/431329

New failing tests:
media/controls/inline-elements-dropoff-order.html
Comment 22 Build Bot 2015-11-14 22:43:45 PST
Created attachment 265556 [details]
Archive of layout-test-results from ews116 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews116  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 23 Jon Lee 2015-11-14 23:17:06 PST
Created attachment 265557 [details]
Patch
Comment 24 Eric Carlson 2015-11-15 09:59:49 PST
Comment on attachment 265557 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=265557&action=review

> LayoutTests/media/controls/inline-elements-dropoff-order.html:21
> +                if (supportsWirelessTargets)
> +                    internals.setMockMediaPlaybackTargetPickerEnabled(false);

Won't this register the native platform picker, and therefore break your test if there are any real ATVs within range of the machine running the test? 

It seems like you might want to change Internals::setMockMediaPlaybackTargetPickerState so you can leave the mock picker registered but configure it to not advertise any devices. For example:

    if (equalIgnoringCase(deviceState, "DeviceAvailable"))
        state = MediaPlaybackTargetContext::OutputDeviceAvailable;
    else if (equalIgnoringCase(deviceState, "DeviceNotAvailable"))
        state = MediaPlaybackTargetContext:: Unavailable;
    else {
        ec = INVALID_ACCESS_ERR;
        return;
    }

and then:

    internals.setMockMediaPlaybackTargetPickerState("Sleepy TV", "DeviceNotAvailable");
Comment 25 Jon Lee 2015-11-16 14:38:57 PST
Comment on attachment 265557 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=265557&action=review

>> LayoutTests/media/controls/inline-elements-dropoff-order.html:21
>> +                    internals.setMockMediaPlaybackTargetPickerEnabled(false);
> 
> Won't this register the native platform picker, and therefore break your test if there are any real ATVs within range of the machine running the test? 
> 
> It seems like you might want to change Internals::setMockMediaPlaybackTargetPickerState so you can leave the mock picker registered but configure it to not advertise any devices. For example:
> 
>     if (equalIgnoringCase(deviceState, "DeviceAvailable"))
>         state = MediaPlaybackTargetContext::OutputDeviceAvailable;
>     else if (equalIgnoringCase(deviceState, "DeviceNotAvailable"))
>         state = MediaPlaybackTargetContext:: Unavailable;
>     else {
>         ec = INVALID_ACCESS_ERR;
>         return;
>     }
> 
> and then:
> 
>     internals.setMockMediaPlaybackTargetPickerState("Sleepy TV", "DeviceNotAvailable");

I thought that being unavailable was the default state.

However, I see the value in being explicit, so I'll add this.
Comment 26 Jon Lee 2015-11-16 14:48:14 PST
Created attachment 265624 [details]
Patch
Comment 27 Dean Jackson 2015-11-16 15:09:18 PST
Comment on attachment 265624 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=265624&action=review

> LayoutTests/media/controls/inline-elements-dropoff-order.html:23
> +                if (supportsWirelessTargets) {
> +                    internals.setMockMediaPlaybackTargetPickerEnabled(false);
> +                    internals.setMockMediaPlaybackTargetPickerState("Wireless playback target", "DeviceNotAvailable");
> +                }

If you check out Internals::Internals() you'll see this:

#if ENABLE(WIRELESS_PLAYBACK_TARGET)
    if (document && document->page())
        document->page()->setMockMediaPlaybackTargetPickerEnabled(true);
#endif

... which means the mock picker is always enabled inside test runs.

But this is actually what we want in this case - setting it to false will mean we'll get the real platform picker.

So I think you can just test for internals, and then setMockMediaPlaybackTargetPickerState("Wireless playback target", "DeviceNotAvailable");

I think that should mean the button will exist, but doesn't have anything to actually pick from. (Since we never select the target, I guess we didn't really need the addition of the DeviceNotAvailable state, but that will be nice to have in the future anyway)
Comment 28 Jon Lee 2015-11-16 15:53:35 PST
Created attachment 265631 [details]
Patch
Comment 29 Jon Lee 2015-11-16 15:55:55 PST
(In reply to comment #27)
> Comment on attachment 265624 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=265624&action=review
> 
> > LayoutTests/media/controls/inline-elements-dropoff-order.html:23
> > +                if (supportsWirelessTargets) {
> > +                    internals.setMockMediaPlaybackTargetPickerEnabled(false);
> > +                    internals.setMockMediaPlaybackTargetPickerState("Wireless playback target", "DeviceNotAvailable");
> > +                }
> 
> If you check out Internals::Internals() you'll see this:
> 
> #if ENABLE(WIRELESS_PLAYBACK_TARGET)
>     if (document && document->page())
>         document->page()->setMockMediaPlaybackTargetPickerEnabled(true);
> #endif
> 
> ... which means the mock picker is always enabled inside test runs.
> 
> But this is actually what we want in this case - setting it to false will
> mean we'll get the real platform picker.
> 
> So I think you can just test for internals, and then
> setMockMediaPlaybackTargetPickerState("Wireless playback target",
> "DeviceNotAvailable");
> 
> I think that should mean the button will exist, but doesn't have anything to
> actually pick from. (Since we never select the target, I guess we didn't
> really need the addition of the DeviceNotAvailable state, but that will be
> nice to have in the future anyway)

Offline: setting the Enabled setting to false makes it possible to switch to using the native wireless environment than the mock one. So setting picker state will make Internals use the mock environment with no devices available.
Comment 30 Dean Jackson 2015-11-16 16:01:20 PST
Comment on attachment 265631 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=265631&action=review

> Source/WebCore/ChangeLog:28
> +        * testing/Internals.cpp: Update to use a reference to Page.

Nice.

> LayoutTests/media/controls/inline-elements-dropoff-order.html:10
> +            var supportsWirelessTargets = !!internals.setMockMediaPlaybackTargetPickerEnabled;

I know we say this only works in DRT, but maybe this line can be removed and....

> LayoutTests/media/controls/inline-elements-dropoff-order.html:21
> +                if (supportsWirelessTargets)
> +                    internals.setMockMediaPlaybackTargetPickerState("Wireless playback target", "DeviceNotAvailable");

... this can just be if (window.internals). That way when running in Safari we won't get an exception right away.

I should have noted this originally, sorry.

> LayoutTests/media/controls/inline-elements-dropoff-order.html:71
> +                    if (supportsWirelessTargets)

And I guess this would be if (window.internals) too.
Comment 31 Jon Lee 2015-11-17 10:40:53 PST
Comment on attachment 265631 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=265631&action=review

>> LayoutTests/media/controls/inline-elements-dropoff-order.html:21
>> +                    internals.setMockMediaPlaybackTargetPickerState("Wireless playback target", "DeviceNotAvailable");
> 
> ... this can just be if (window.internals). That way when running in Safari we won't get an exception right away.
> 
> I should have noted this originally, sorry.

"if (window.internals)" will end up breaking Yosemite and older. I'd expect supportsWirelessTargets to evaluate to false on those OSes.
Comment 32 WebKit Commit Bot 2015-11-19 02:06:26 PST
Comment on attachment 265631 [details]
Patch

Clearing flags on attachment: 265631

Committed r192622: <http://trac.webkit.org/changeset/192622>
Comment 33 WebKit Commit Bot 2015-11-19 02:06:33 PST
All reviewed patches have been landed.  Closing bug.
Comment 34 Ryan Haddad 2015-11-19 08:39:40 PST
media/controls/inline-elements-dropoff-order.html is failing on mac-wk1

El Capitan:
<https://build.webkit.org/builders/Apple%20El%20Capitan%20Release%20WK1%20(Tests)/builds/1402>

Mavericks:
<https://build.webkit.org/builders/Apple%20Mavericks%20Debug%20WK1%20(Tests)/builds/17146>
Comment 35 Ryan Haddad 2015-11-19 08:40:57 PST
(In reply to comment #34)
> media/controls/inline-elements-dropoff-order.html is failing on mac-wk1
> 
> El Capitan:
> <https://build.webkit.org/builders/
> Apple%20El%20Capitan%20Release%20WK1%20(Tests)/builds/1402>
> 
> Mavericks:
> <https://build.webkit.org/builders/Apple%20Mavericks%20Debug%20WK1%20(Tests)/
> builds/17146>

Also failing on El Capitan Debug WK2:
<https://build.webkit.org/results/Apple%20El%20Capitan%20Debug%20WK2%20(Tests)/r192626%20(1213)/results.html>
Comment 36 Alexey Proskuryakov 2015-11-19 09:34:32 PST
Let's roll it out? Not much use skipping a new test.
Comment 37 WebKit Commit Bot 2015-11-19 09:38:32 PST
Re-opened since this is blocked by bug 151446
Comment 38 Eric Carlson 2016-07-06 08:04:36 PDT
Created attachment 282894 [details]
Proposed patch.
Comment 39 Antoine Quint 2016-07-06 08:36:47 PDT
Comment on attachment 282894 [details]
Proposed patch.

View in context: https://bugs.webkit.org/attachment.cgi?id=282894&action=review

> LayoutTests/media/controls/inline-elements-dropoff-order.html:65
> +                }, 100);

I think it would be clearer to use a "resize" event handler followed by a requestAnimationFrame() to make sure that a layout has happened as a result of a resize for the controls.
Comment 40 Build Bot 2016-07-06 08:51:42 PDT
Comment on attachment 282894 [details]
Proposed patch.

Attachment 282894 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/1636136

New failing tests:
media/controls/inline-elements-dropoff-order.html
Comment 41 Build Bot 2016-07-06 08:51:46 PDT
Created attachment 282898 [details]
Archive of layout-test-results from ews102 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 42 Build Bot 2016-07-06 08:56:08 PDT
Comment on attachment 282894 [details]
Proposed patch.

Attachment 282894 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/1636142

New failing tests:
media/controls/inline-elements-dropoff-order.html
Comment 43 Build Bot 2016-07-06 08:56:12 PDT
Created attachment 282899 [details]
Archive of layout-test-results from ews104 for mac-yosemite-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews104  Port: mac-yosemite-wk2  Platform: Mac OS X 10.10.5
Comment 44 Build Bot 2016-07-06 09:04:40 PDT
Comment on attachment 282894 [details]
Proposed patch.

Attachment 282894 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/1636144

New failing tests:
media/controls/inline-elements-dropoff-order.html
Comment 45 Build Bot 2016-07-06 09:04:44 PDT
Created attachment 282900 [details]
Archive of layout-test-results from ews112 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews112  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 46 Eric Carlson 2016-07-06 11:46:09 PDT
Created attachment 282917 [details]
Updated patch for the bots.
Comment 47 Build Bot 2016-07-06 12:21:19 PDT
Comment on attachment 282917 [details]
Updated patch for the bots.

Attachment 282917 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/1636934

New failing tests:
media/controls/inline-elements-dropoff-order.html
Comment 48 Build Bot 2016-07-06 12:21:23 PDT
Created attachment 282924 [details]
Archive of layout-test-results from ews101 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews101  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 49 Build Bot 2016-07-06 12:28:11 PDT
Comment on attachment 282917 [details]
Updated patch for the bots.

Attachment 282917 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/1636954

New failing tests:
media/controls/inline-elements-dropoff-order.html
Comment 50 Build Bot 2016-07-06 12:28:17 PDT
Created attachment 282927 [details]
Archive of layout-test-results from ews107 for mac-yosemite-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews107  Port: mac-yosemite-wk2  Platform: Mac OS X 10.10.5
Comment 51 Build Bot 2016-07-06 12:36:24 PDT
Comment on attachment 282917 [details]
Updated patch for the bots.

Attachment 282917 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/1636963

New failing tests:
media/controls/inline-elements-dropoff-order.html
Comment 52 Build Bot 2016-07-06 12:36:28 PDT
Created attachment 282930 [details]
Archive of layout-test-results from ews112 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews112  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 53 Eric Carlson 2016-07-06 12:59:37 PDT
Created attachment 282936 [details]
Another updated patch for the bots.
Comment 54 Build Bot 2016-07-06 13:48:15 PDT
Comment on attachment 282936 [details]
Another updated patch for the bots.

Attachment 282936 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/1637310

New failing tests:
media/controls/inline-elements-dropoff-order.html
Comment 55 Build Bot 2016-07-06 13:48:19 PDT
Created attachment 282939 [details]
Archive of layout-test-results from ews102 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 56 Build Bot 2016-07-06 13:52:04 PDT
Comment on attachment 282936 [details]
Another updated patch for the bots.

Attachment 282936 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/1637318

New failing tests:
media/controls/inline-elements-dropoff-order.html
Comment 57 Build Bot 2016-07-06 13:52:08 PDT
Created attachment 282940 [details]
Archive of layout-test-results from ews107 for mac-yosemite-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews107  Port: mac-yosemite-wk2  Platform: Mac OS X 10.10.5
Comment 58 Build Bot 2016-07-06 14:04:53 PDT
Comment on attachment 282936 [details]
Another updated patch for the bots.

Attachment 282936 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/1637314

New failing tests:
media/controls/inline-elements-dropoff-order.html
Comment 59 Build Bot 2016-07-06 14:04:56 PDT
Created attachment 282943 [details]
Archive of layout-test-results from ews117 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews117  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 60 Eric Carlson 2016-07-06 17:25:56 PDT
Created attachment 282967 [details]
Another updated patch for the bots.
Comment 61 Build Bot 2016-07-06 17:54:16 PDT
Comment on attachment 282967 [details]
Another updated patch for the bots.

Attachment 282967 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/1638245

New failing tests:
media/controls/inline-elements-dropoff-order.html
Comment 62 Build Bot 2016-07-06 17:54:21 PDT
Created attachment 282972 [details]
Archive of layout-test-results from ews102 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 63 Build Bot 2016-07-06 18:01:34 PDT
Comment on attachment 282967 [details]
Another updated patch for the bots.

Attachment 282967 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/1638283

New failing tests:
media/controls/inline-elements-dropoff-order.html
Comment 64 Build Bot 2016-07-06 18:01:39 PDT
Created attachment 282974 [details]
Archive of layout-test-results from ews106 for mac-yosemite-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-yosemite-wk2  Platform: Mac OS X 10.10.5
Comment 65 Eric Carlson 2016-07-06 18:07:30 PDT
Created attachment 282975 [details]
One more time for the bots!
Comment 66 WebKit Commit Bot 2016-07-07 07:22:06 PDT
Comment on attachment 282975 [details]
One more time for the bots!

Clearing flags on attachment: 282975

Committed r202905: <http://trac.webkit.org/changeset/202905>
Comment 67 Ryan Haddad 2016-07-07 09:02:24 PDT
(In reply to comment #66)
> Comment on attachment 282975 [details]
> One more time for the bots!
> 
> Clearing flags on attachment: 282975
> 
> Committed r202905: <http://trac.webkit.org/changeset/202905>

media/controls/inline-elements-dropoff-order.html is failing on El Capitan bots:

https://build.webkit.org/results/Apple%20El%20Capitan%20Release%20WK2%20(Tests)/r202906%20(7428)/results.html

https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=media%2Fcontrols%2Finline-elements-dropoff-order.html
Comment 68 Eric Carlson 2016-07-07 09:26:07 PDT
Plus https://trac.webkit.org/r202911 to add El Capitan-specific test results.
Comment 69 WebKit Commit Bot 2016-07-07 13:47:46 PDT
Re-opened since this is blocked by bug 159522
Comment 70 Ryan Haddad 2016-07-07 13:56:08 PDT
(In reply to comment #69)
> Re-opened since this is blocked by bug 159522

Link to test failure on El Capitan WK1
https://build.webkit.org/results/Apple%20El%20Capitan%20Release%20WK1%20(Tests)/r202927%20(7633)/results.html
Comment 71 Eric Carlson 2016-07-08 15:34:25 PDT
Created attachment 283205 [details]
Updated patch.
Comment 72 Eric Carlson 2016-07-08 20:19:41 PDT
Created attachment 283242 [details]
Updated.
Comment 73 WebKit Commit Bot 2016-07-11 07:38:16 PDT
Comment on attachment 283242 [details]
Updated.

Clearing flags on attachment: 283242

Committed r203057: <http://trac.webkit.org/changeset/203057>
Comment 74 WebKit Commit Bot 2016-07-11 07:38:29 PDT
All reviewed patches have been landed.  Closing bug.
Comment 75 Eric Carlson 2016-07-11 10:48:42 PDT
Plus r203068 to add El Capitan specific results.