RESOLVED FIXED 148912
[Mac] Add a mock AppleTV device for testing
https://bugs.webkit.org/show_bug.cgi?id=148912
Summary [Mac] Add a mock AppleTV device for testing
Eric Carlson
Reported 2015-09-06 12:27:20 PDT
Add a mock AppleTV device to make testing possible.
Attachments
Proposed patch. (88.54 KB, patch)
2015-09-06 13:10 PDT, Eric Carlson
buildbot: commit-queue-
Archive of layout-test-results from ews106 for mac-mavericks-wk2 (667.40 KB, application/zip)
2015-09-06 13:56 PDT, Build Bot
no flags
Archive of layout-test-results from ews101 for mac-mavericks (591.87 KB, application/zip)
2015-09-06 14:11 PDT, Build Bot
no flags
Updated patch. (89.38 KB, patch)
2015-09-06 15:19 PDT, Eric Carlson
no flags
Updated patch. (89.30 KB, patch)
2015-09-06 16:17 PDT, Eric Carlson
no flags
YAUP. (89.41 KB, patch)
2015-09-06 16:54 PDT, Eric Carlson
no flags
Patch for landing. (89.55 KB, patch)
2015-11-09 12:43 PST, Eric Carlson
commit-queue: commit-queue-
Updated patch for landing. (89.38 KB, patch)
2015-11-09 16:37 PST, Eric Carlson
commit-queue: commit-queue-
One more time? (92.95 KB, patch)
2015-11-09 18:00 PST, Eric Carlson
commit-queue: commit-queue-
Another patch for landing. (92.93 KB, patch)
2015-11-09 18:07 PST, Eric Carlson
commit-queue: commit-queue-
Another patch for landing. (92.93 KB, patch)
2015-11-09 18:17 PST, Eric Carlson
commit-queue: commit-queue-
One more time? (89.49 KB, patch)
2015-11-09 21:26 PST, Eric Carlson
no flags
Radar WebKit Bug Importer
Comment 1 2015-09-06 12:31:46 PDT
Eric Carlson
Comment 2 2015-09-06 13:10:50 PDT
Created attachment 260713 [details] Proposed patch.
WebKit Commit Bot
Comment 3 2015-09-06 13:13:02 PDT
Attachment 260713 [details] did not pass style-queue: ERROR: Source/WebCore/platform/graphics/MediaPlaybackTargetPicker.cpp:31: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.h:32: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:2866: Missing space before { [whitespace/braces] [5] Total errors found: 3 in 49 files If any of these errors are false positives, please file a bug against check-webkit-style.
Build Bot
Comment 4 2015-09-06 13:56:25 PDT
Comment on attachment 260713 [details] Proposed patch. Attachment 260713 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.webkit.org/results/146305 New failing tests: media/controls/airplay-picker.html
Build Bot
Comment 5 2015-09-06 13:56:27 PDT
Created attachment 260716 [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
Build Bot
Comment 6 2015-09-06 14:11:42 PDT
Comment on attachment 260713 [details] Proposed patch. Attachment 260713 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/146327 New failing tests: media/controls/airplay-picker.html
Build Bot
Comment 7 2015-09-06 14:11:45 PDT
Created attachment 260717 [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
Eric Carlson
Comment 8 2015-09-06 15:19:25 PDT
Created attachment 260719 [details] Updated patch.
Eric Carlson
Comment 9 2015-09-06 16:17:55 PDT
Created attachment 260721 [details] Updated patch.
Eric Carlson
Comment 10 2015-09-06 16:54:14 PDT
Jer Noble
Comment 11 2015-09-08 14:51:09 PDT
Comment on attachment 260726 [details] YAUP. Unofficial r+, but needs a WK2 owner to do a final sign-off.
Tim Horton
Comment 12 2015-09-10 11:09:29 PDT
Comment on attachment 260726 [details] YAUP. View in context: https://bugs.webkit.org/attachment.cgi?id=260726&action=review r+ the WK2 parts, with Jer's r+ for the other bits. > Source/WebKit2/Shared/WebCoreArgumentCoders.cpp:2211 > + return decodePlatformData(decoder, target); There's this funny thing where if there's ever a path through a decoder that doesn't write to a field, you can end up with stale data if you re-decode into the same target (which happens on occasion). Please consider clearing the platform data if we're not writing to it, and the others if we're not writing to them.
Alexey Proskuryakov
Comment 13 2015-09-17 14:15:30 PDT
Please unskip media/controls/airplay-picker.html when this is done.
Eric Carlson
Comment 14 2015-11-09 12:43:39 PST
Created attachment 265082 [details] Patch for landing.
WebKit Commit Bot
Comment 15 2015-11-09 13:32:30 PST
Comment on attachment 265082 [details] Patch for landing. Rejecting attachment 265082 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 265082, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: fs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc ... Currently at 192172 = 7110aaa00b572a4843a3b6327bba8ed9f46da4af r192173 = 3943995b1795d43bb008f0164ea176c68c977fbc r192174 = c4b8e58fb6cf85eaeeeab58749f7a3b5a68849c2 r192175 = 1d8f06915dd20f4bda15034462350931d75ed264 Done rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc First, rewinding head to replay your work on top of it... Fast-forwarded master to refs/remotes/origin/master. Full output: http://webkit-queues.webkit.org/results/407024
Eric Carlson
Comment 16 2015-11-09 16:37:44 PST
Created attachment 265117 [details] Updated patch for landing.
WebKit Commit Bot
Comment 17 2015-11-09 16:39:46 PST
Comment on attachment 265117 [details] Updated patch for landing. Rejecting attachment 265117 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'apply-attachment', '--no-update', '--non-interactive', 265117, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: 3. patching file LayoutTests/media/controls/airplay-picker-expected.txt patching file LayoutTests/media/controls/airplay-picker.html patching file LayoutTests/media/controls/controls-test-helpers.js patching file LayoutTests/platform/mac/TestExpectations Hunk #1 FAILED at 1363. 1 out of 1 hunk FAILED -- saving rejects to file LayoutTests/platform/mac/TestExpectations.rej Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Full output: http://webkit-queues.webkit.org/results/407599
Eric Carlson
Comment 18 2015-11-09 18:00:03 PST
Created attachment 265133 [details] One more time?
WebKit Commit Bot
Comment 19 2015-11-09 18:02:46 PST
Comment on attachment 265133 [details] One more time? Rejecting attachment 265133 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-02', 'apply-attachment', '--no-update', '--non-interactive', 265133, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: g file Source/WebKit/mac/WebView/WebViewInternal.h patching file LayoutTests/ChangeLog Hunk #1 succeeded at 1 with fuzz 3. patching file LayoutTests/media/controls/airplay-picker-expected.txt patching file LayoutTests/media/controls/airplay-picker.html patching file LayoutTests/media/controls/controls-test-helpers.js patching file LayoutTests/platform/mac/TestExpectations Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Full output: http://webkit-queues.webkit.org/results/407939
Eric Carlson
Comment 20 2015-11-09 18:07:27 PST
Created attachment 265134 [details] Another patch for landing.
WebKit Commit Bot
Comment 21 2015-11-09 18:09:47 PST
Comment on attachment 265134 [details] Another patch for landing. Rejecting attachment 265134 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'apply-attachment', '--no-update', '--non-interactive', 265134, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: g file Source/WebKit/mac/WebView/WebViewInternal.h patching file LayoutTests/ChangeLog Hunk #1 succeeded at 1 with fuzz 3. patching file LayoutTests/media/controls/airplay-picker-expected.txt patching file LayoutTests/media/controls/airplay-picker.html patching file LayoutTests/media/controls/controls-test-helpers.js patching file LayoutTests/platform/mac/TestExpectations Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Full output: http://webkit-queues.webkit.org/results/407988
Eric Carlson
Comment 22 2015-11-09 18:17:04 PST
Created attachment 265136 [details] Another patch for landing.
WebKit Commit Bot
Comment 23 2015-11-09 18:19:41 PST
Comment on attachment 265136 [details] Another patch for landing. Rejecting attachment 265136 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-01', 'apply-attachment', '--no-update', '--non-interactive', 265136, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: g file Source/WebKit/mac/WebView/WebViewInternal.h patching file LayoutTests/ChangeLog Hunk #1 succeeded at 1 with fuzz 3. patching file LayoutTests/media/controls/airplay-picker-expected.txt patching file LayoutTests/media/controls/airplay-picker.html patching file LayoutTests/media/controls/controls-test-helpers.js patching file LayoutTests/platform/mac/TestExpectations Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Full output: http://webkit-queues.webkit.org/results/408048
Eric Carlson
Comment 24 2015-11-09 21:26:31 PST
Created attachment 265142 [details] One more time?
WebKit Commit Bot
Comment 25 2015-11-09 22:25:56 PST
Comment on attachment 265142 [details] One more time? Clearing flags on attachment: 265142 Committed r192200: <http://trac.webkit.org/changeset/192200>
Csaba Osztrogonác
Comment 26 2015-11-10 03:57:11 PST
(In reply to comment #25) > Comment on attachment 265142 [details] > One more time? > > Clearing flags on attachment: 265142 > > Committed r192200: <http://trac.webkit.org/changeset/192200> It broke the build on Apple El Capitan bots: https://build.webkit.org/waterfall
Csaba Osztrogonác
Comment 27 2015-11-10 04:06:19 PST
Eric Carlson
Comment 28 2015-11-10 05:57:01 PST
(In reply to comment #27) > fixed in http://trac.webkit.org/changeset/192230 Thank you Ossy!
Note You need to log in before you can comment on or make changes to this bug.