Bug 199639

Summary: Add threading assertion to WeakPtrFactory::createWeakPtr()
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: Web Template FrameworkAssignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: benjamin, cmarcelo, commit-queue, dbates, ews-watchlist, Hironori.Fujii, pvollan, rniwa, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 199652, 199656, 199698, 199700, 199727, 199775, 199777    
Bug Blocks: 199517    
Attachments:
Description Flags
WIP Patch
none
WIP Patch
ews-watchlist: commit-queue-
Archive of layout-test-results from ews100 for mac-highsierra
none
Archive of layout-test-results from ews106 for mac-highsierra-wk2
none
Archive of layout-test-results from ews123 for ios-simulator-wk2
none
Archive of layout-test-results from ews112 for mac-highsierra
none
Archive of layout-test-results from ews214 for win-future
none
WIP Patch
ews-watchlist: commit-queue-
Archive of layout-test-results from ews213 for win-future
none
WIP Patch
ews-watchlist: commit-queue-
Archive of layout-test-results from ews213 for win-future
none
WIP Patch
ews-watchlist: commit-queue-
Archive of layout-test-results from ews214 for win-future
none
WIP Patch
ews-watchlist: commit-queue-
Archive of layout-test-results from ews212 for win-future
none
WIP Patch
none
Patch none

Description Chris Dumez 2019-07-09 14:02:12 PDT
Add threading assertion to WeakPtrFactory::createWeakPtr().
Comment 1 Chris Dumez 2019-07-09 14:02:41 PDT
Created attachment 373760 [details]
WIP Patch
Comment 2 Chris Dumez 2019-07-09 14:07:03 PDT
Created attachment 373762 [details]
WIP Patch
Comment 3 EWS Watchlist 2019-07-09 15:36:27 PDT
Comment on attachment 373762 [details]
WIP Patch

Attachment 373762 [details] did not pass mac-ews (mac):
Output: https://webkit-queues.webkit.org/results/12701571

New failing tests:
fast/block/float/list-marker-is-float-crash.html
media/media-source/media-source-seek-redundant-append.html
imported/w3c/web-platform-tests/2dcontext/imagebitmap/createImageBitmap-drawImage.html
media/audio-data-url.html
http/tests/media/clearkey/collect-webkit-media-session.html
http/tests/media/media-source/mediasource-play-then-seek-back-with-remote-control.html
imported/w3c/web-platform-tests/media-source/mediasource-play.html
media/video-src-blob-using-open-panel.html
media/sources-fallback-codecs.html
http/wpt/mediarecorder/MediaRecorder-AV-audio-video-dataavailable.html
imported/w3c/web-platform-tests/media-source/mediasource-redundant-seek.html
imported/w3c/web-platform-tests/media-source/mediasource-config-change-mp4-av-video-bitrate.html
media/media-source/media-source-stalled-holds-sleep-assertion.html
http/tests/media/clearkey/clear-key-hls-aes128.html
media/unsupported-rtsp.html
inspector/console/webcore-logging.html
platform/mac/media/media-source/videoplaybackquality-decompressionsession.html
imported/w3c/web-platform-tests/2dcontext/imagebitmap/createImageBitmap-invalid-args.html
imported/w3c/web-platform-tests/media-source/mediasource-detach.html
imported/w3c/web-platform-tests/2dcontext/imagebitmap/createImageBitmap-transfer.html
imported/w3c/web-platform-tests/media-source/mediasource-play-then-seek-back.html
imported/w3c/web-platform-tests/media-source/mediasource-config-change-mp4-av-audio-bitrate.html
media/video-src-blob.html
imported/w3c/web-platform-tests/media-source/mediasource-avtracks.html
fast/forms/form-associated-element-crash3.html
Comment 4 EWS Watchlist 2019-07-09 15:36:29 PDT
Created attachment 373784 [details]
Archive of layout-test-results from ews100 for mac-highsierra

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews100  Port: mac-highsierra  Platform: Mac OS X 10.13.6
Comment 5 EWS Watchlist 2019-07-09 15:44:38 PDT
Comment on attachment 373762 [details]
WIP Patch

Attachment 373762 [details] did not pass mac-wk2-ews (mac-wk2):
Output: https://webkit-queues.webkit.org/results/12701602

New failing tests:
fast/block/float/list-marker-is-float-crash.html
media/media-source/media-source-seek-redundant-append.html
imported/w3c/web-platform-tests/2dcontext/imagebitmap/createImageBitmap-drawImage.html
media/audio-data-url.html
http/tests/media/clearkey/collect-webkit-media-session.html
http/tests/media/media-source/mediasource-play-then-seek-back-with-remote-control.html
imported/w3c/web-platform-tests/media-source/mediasource-play.html
media/video-src-blob-using-open-panel.html
media/sources-fallback-codecs.html
http/wpt/mediarecorder/MediaRecorder-AV-audio-video-dataavailable.html
imported/w3c/web-platform-tests/media-source/mediasource-redundant-seek.html
imported/w3c/web-platform-tests/media-source/mediasource-config-change-mp4-av-video-bitrate.html
media/media-source/media-source-stalled-holds-sleep-assertion.html
http/tests/media/clearkey/clear-key-hls-aes128.html
media/unsupported-rtsp.html
inspector/console/webcore-logging.html
platform/mac/media/media-source/videoplaybackquality-decompressionsession.html
imported/w3c/web-platform-tests/2dcontext/imagebitmap/createImageBitmap-invalid-args.html
imported/w3c/web-platform-tests/media-source/mediasource-detach.html
imported/w3c/web-platform-tests/2dcontext/imagebitmap/createImageBitmap-transfer.html
imported/w3c/web-platform-tests/media-source/mediasource-play-then-seek-back.html
imported/w3c/web-platform-tests/media-source/mediasource-config-change-mp4-av-audio-bitrate.html
imported/w3c/web-platform-tests/media-source/mediasource-avtracks.html
fast/forms/form-associated-element-crash3.html
Comment 6 EWS Watchlist 2019-07-09 15:44:39 PDT
Created attachment 373785 [details]
Archive of layout-test-results from ews106 for mac-highsierra-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-highsierra-wk2  Platform: Mac OS X 10.13.6
Comment 7 EWS Watchlist 2019-07-09 16:21:16 PDT
Comment on attachment 373762 [details]
WIP Patch

Attachment 373762 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: https://webkit-queues.webkit.org/results/12701813

New failing tests:
fast/block/float/list-marker-is-float-crash.html
imported/w3c/web-platform-tests/2dcontext/imagebitmap/createImageBitmap-invalid-args.html
imported/w3c/web-platform-tests/2dcontext/imagebitmap/createImageBitmap-transfer.html
media/audio-data-url.html
media/unsupported-rtsp.html
fast/forms/form-associated-element-crash3.html
http/wpt/mediarecorder/MediaRecorder-AV-audio-video-dataavailable.html
Comment 8 EWS Watchlist 2019-07-09 16:21:18 PDT
Created attachment 373788 [details]
Archive of layout-test-results from ews123 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews123  Port: ios-simulator-wk2  Platform: Mac OS X 10.14.5
Comment 9 EWS Watchlist 2019-07-09 16:40:42 PDT
Comment on attachment 373762 [details]
WIP Patch

Attachment 373762 [details] did not pass mac-debug-ews (mac):
Output: https://webkit-queues.webkit.org/results/12702083

New failing tests:
fast/block/float/list-marker-is-float-crash.html
media/media-source/media-source-seek-redundant-append.html
imported/w3c/web-platform-tests/2dcontext/imagebitmap/createImageBitmap-drawImage.html
media/audio-data-url.html
http/tests/media/clearkey/collect-webkit-media-session.html
http/tests/media/media-source/mediasource-play-then-seek-back-with-remote-control.html
http/wpt/mediarecorder/MediaRecorder-AV-audio-video-dataavailable.html
media/video-src-blob-using-open-panel.html
media/sources-fallback-codecs.html
imported/w3c/web-platform-tests/media-source/mediasource-play.html
imported/w3c/web-platform-tests/media-source/mediasource-redundant-seek.html
imported/w3c/web-platform-tests/media-source/mediasource-config-change-mp4-av-video-bitrate.html
media/media-source/media-source-stalled-holds-sleep-assertion.html
http/tests/media/clearkey/clear-key-hls-aes128.html
media/unsupported-rtsp.html
inspector/console/webcore-logging.html
platform/mac/media/media-source/videoplaybackquality-decompressionsession.html
imported/w3c/web-platform-tests/2dcontext/imagebitmap/createImageBitmap-invalid-args.html
imported/w3c/web-platform-tests/media-source/mediasource-detach.html
imported/w3c/web-platform-tests/2dcontext/imagebitmap/createImageBitmap-transfer.html
imported/w3c/web-platform-tests/media-source/mediasource-play-then-seek-back.html
imported/w3c/web-platform-tests/media-source/mediasource-config-change-mp4-av-audio-bitrate.html
media/video-src-blob.html
fast/forms/form-associated-element-crash3.html
Comment 10 EWS Watchlist 2019-07-09 16:40:44 PDT
Created attachment 373790 [details]
Archive of layout-test-results from ews112 for mac-highsierra

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews112  Port: mac-highsierra  Platform: Mac OS X 10.13.6
Comment 11 EWS Watchlist 2019-07-09 17:13:21 PDT
Comment on attachment 373762 [details]
WIP Patch

Attachment 373762 [details] did not pass win-ews (win):
Output: https://webkit-queues.webkit.org/results/12702357

New failing tests:
fast/replaced/border-radius-clip.html
fast/dom/beforeload/remove-video-in-beforeload-listener.html
fast/css/relative-position-replaced-in-table-display-crash.html
imported/blink/fast/dom/Window/open-window-features-fuzz.html
fast/text/international/embed-bidi-style-in-isolate-crash.html
fast/dom/focus-shift-crash.html
fast/events/media-element-focus-tab.html
http/tests/security/canvas-remote-read-remote-video-hls.html
fast/events/tabindex-focus-blur-all.html
imported/blink/compositing/video/video-controls-layer-creation-squashing.html
Comment 12 EWS Watchlist 2019-07-09 17:13:24 PDT
Created attachment 373796 [details]
Archive of layout-test-results from ews214 for win-future

The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews214  Port: win-future  Platform: CYGWIN_NT-10.0-17763-3.0.5-338.x86_64-x86_64-64bit
Comment 13 Chris Dumez 2019-07-10 10:34:52 PDT
Created attachment 373844 [details]
WIP Patch
Comment 14 EWS Watchlist 2019-07-10 13:16:40 PDT
Comment on attachment 373844 [details]
WIP Patch

Attachment 373844 [details] did not pass win-ews (win):
Output: https://webkit-queues.webkit.org/results/12709173

New failing tests:
fast/replaced/border-radius-clip.html
fast/events/media-element-focus-tab.html
fast/css/relative-position-replaced-in-table-display-crash.html
imported/blink/fast/dom/Window/open-window-features-fuzz.html
fast/text/international/embed-bidi-style-in-isolate-crash.html
fast/dom/focus-shift-crash.html
fast/dom/beforeload/remove-video-in-beforeload-listener.html
http/tests/security/canvas-remote-read-remote-video-hls.html
fast/events/tabindex-focus-blur-all.html
imported/blink/compositing/video/video-controls-layer-creation-squashing.html
Comment 15 EWS Watchlist 2019-07-10 13:16:43 PDT
Created attachment 373860 [details]
Archive of layout-test-results from ews213 for win-future

The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews213  Port: win-future  Platform: CYGWIN_NT-10.0-17763-3.0.5-338.x86_64-x86_64-64bit
Comment 16 Chris Dumez 2019-07-10 16:18:51 PDT
Created attachment 373873 [details]
WIP Patch
Comment 17 EWS Watchlist 2019-07-10 18:55:06 PDT
Comment on attachment 373873 [details]
WIP Patch

Attachment 373873 [details] did not pass win-ews (win):
Output: https://webkit-queues.webkit.org/results/12711308

New failing tests:
fast/replaced/border-radius-clip.html
fast/dom/beforeload/remove-video-in-beforeload-listener.html
fast/css/relative-position-replaced-in-table-display-crash.html
imported/blink/fast/dom/Window/open-window-features-fuzz.html
fast/text/international/embed-bidi-style-in-isolate-crash.html
fast/dom/focus-shift-crash.html
fast/events/media-element-focus-tab.html
http/tests/security/canvas-remote-read-remote-video-hls.html
fast/events/tabindex-focus-blur-all.html
imported/blink/compositing/video/video-controls-layer-creation-squashing.html
Comment 18 EWS Watchlist 2019-07-10 18:55:08 PDT
Created attachment 373891 [details]
Archive of layout-test-results from ews213 for win-future

The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews213  Port: win-future  Platform: CYGWIN_NT-10.0-17763-3.0.5-338.x86_64-x86_64-64bit
Comment 19 Chris Dumez 2019-07-10 19:39:58 PDT
I have a fix for the remaining api-is failures at Bug 199698. This leaves the win failures which I have unable to fix at the moment because win-EWS does not upload crash traces :(
Comment 20 Chris Dumez 2019-07-11 10:15:52 PDT
Created attachment 373925 [details]
WIP Patch
Comment 21 EWS Watchlist 2019-07-11 16:01:48 PDT
Comment on attachment 373925 [details]
WIP Patch

Attachment 373925 [details] did not pass win-ews (win):
Output: https://webkit-queues.webkit.org/results/12718500

New failing tests:
fast/replaced/border-radius-clip.html
fast/dom/beforeload/remove-video-in-beforeload-listener.html
fast/css/relative-position-replaced-in-table-display-crash.html
imported/blink/fast/dom/Window/open-window-features-fuzz.html
fast/text/international/embed-bidi-style-in-isolate-crash.html
fast/dom/focus-shift-crash.html
fast/events/media-element-focus-tab.html
http/tests/security/canvas-remote-read-remote-video-hls.html
fast/events/tabindex-focus-blur-all.html
imported/blink/compositing/video/video-controls-layer-creation-squashing.html
Comment 22 EWS Watchlist 2019-07-11 16:01:50 PDT
Created attachment 373969 [details]
Archive of layout-test-results from ews214 for win-future

The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews214  Port: win-future  Platform: CYGWIN_NT-10.0-17763-3.0.5-338.x86_64-x86_64-64bit
Comment 23 Chris Dumez 2019-07-12 08:01:32 PDT
Created attachment 374011 [details]
WIP Patch
Comment 24 EWS Watchlist 2019-07-12 10:54:59 PDT
Comment on attachment 374011 [details]
WIP Patch

Attachment 374011 [details] did not pass win-ews (win):
Output: https://webkit-queues.webkit.org/results/12725921

New failing tests:
fast/replaced/border-radius-clip.html
fast/events/media-element-focus-tab.html
fast/css/relative-position-replaced-in-table-display-crash.html
imported/blink/fast/dom/Window/open-window-features-fuzz.html
fast/text/international/embed-bidi-style-in-isolate-crash.html
fast/dom/focus-shift-crash.html
fast/dom/beforeload/remove-video-in-beforeload-listener.html
http/tests/security/canvas-remote-read-remote-video-hls.html
fast/events/tabindex-focus-blur-all.html
imported/blink/compositing/video/video-controls-layer-creation-squashing.html
Comment 25 EWS Watchlist 2019-07-12 10:55:02 PDT
Created attachment 374021 [details]
Archive of layout-test-results from ews212 for win-future

The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews212  Port: win-future  Platform: CYGWIN_NT-10.0-17763-3.0.5-338.x86_64-x86_64-64bit
Comment 26 Chris Dumez 2019-07-13 09:16:22 PDT
Created attachment 374078 [details]
WIP Patch

Hoping for green bubbles all around.
Comment 27 Chris Dumez 2019-07-13 12:45:44 PDT
Created attachment 374080 [details]
Patch
Comment 28 WebKit Commit Bot 2019-07-14 15:18:09 PDT
Comment on attachment 374080 [details]
Patch

Clearing flags on attachment: 374080

Committed r247425: <https://trac.webkit.org/changeset/247425>
Comment 29 WebKit Commit Bot 2019-07-14 15:18:11 PDT
All reviewed patches have been landed.  Closing bug.
Comment 30 Radar WebKit Bug Importer 2019-07-14 15:19:35 PDT
<rdar://problem/53081768>