Bug 188025 - Allow ActiveDOMObject's canSuspend / suspend / resume overrides to destroy ActiveDOMObjects
Summary: Allow ActiveDOMObject's canSuspend / suspend / resume overrides to destroy Ac...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL:
Keywords: InRadar
Depends on:
Blocks: 187284
  Show dependency treegraph
 
Reported: 2018-07-25 15:30 PDT by Chris Dumez
Modified: 2018-07-27 08:59 PDT (History)
11 users (show)

See Also:


Attachments
Patch (10.49 KB, patch)
2018-07-25 15:38 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (10.47 KB, patch)
2018-07-25 16:21 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Dumez 2018-07-25 15:30:30 PDT
Allow ActiveDOMObject's canSuspend / suspend / resume overrides to destroy ActiveDOMObjects. There is a good reason to prevent construction of new ActiveDOMObjects from these overrides as this would cause break PageCache logic. However, it is fine for these overrides to destroy existing ActiveDOMObjects as long as we make our code iterating over ActiveDOMObjects support it.
Comment 1 Chris Dumez 2018-07-25 15:38:00 PDT
Created attachment 345794 [details]
Patch
Comment 2 Chris Dumez 2018-07-25 16:21:25 PDT
Created attachment 345798 [details]
Patch
Comment 3 WebKit Commit Bot 2018-07-25 17:34:24 PDT
Comment on attachment 345798 [details]
Patch

Rejecting attachment 345798 [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', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 345798, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Logging in as commit-queue@webkit.org...
Fetching: https://bugs.webkit.org/attachment.cgi?id=345798&action=edit
Fetching: https://bugs.webkit.org/show_bug.cgi?id=188025&ctype=xml&excludefield=attachmentdata
Processing 1 patch from 1 bug.
Updating working directory
Processing patch 345798 from bug 188025.
Fetching: https://bugs.webkit.org/attachment.cgi?id=345798
Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	Source/WebCore/ChangeLog

ERROR from SVN:
Item is out of date: File '/trunk/Source/WebCore/ChangeLog' is out of date
W: 98375d5bde7b8b5deb2161c8170939887189f626 and refs/remotes/origin/master differ, using rebase:
:040000 040000 c5a29e897de220921e005b5172e9e434ec122c72 09b5c312726605cb3511c4872bb8e2b3b16ce04d M	Source
Current branch master is up to date.
ERROR: Not all changes have been committed into SVN, however the committed
ones (if any) seem to be successfully integrated into the working tree.
Please see the above messages for details.


Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	Source/WebCore/ChangeLog

ERROR from SVN:
Item is out of date: File '/trunk/Source/WebCore/ChangeLog' is out of date
W: 98375d5bde7b8b5deb2161c8170939887189f626 and refs/remotes/origin/master differ, using rebase:
:040000 040000 c5a29e897de220921e005b5172e9e434ec122c72 09b5c312726605cb3511c4872bb8e2b3b16ce04d M	Source
Current branch master is up to date.
ERROR: Not all changes have been committed into SVN, however the committed
ones (if any) seem to be successfully integrated into the working tree.
Please see the above messages for details.


Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit
Updating OpenSource
From https://git.webkit.org/git/WebKit
   49e70b34d9f..d294af02a32  master     -> origin/master
Partial-rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc ...
Currently at 234219 = 49e70b34d9fd9d0b75ac25fc2aafd34a8f6fd3b6
r234220 = 9c74f145f3f5375c8d17382ddaa69790c6b7b259
r234222 = c3452943748c48453648405a5de829db87d9a580
r234223 = d294af02a323a020fb8a1f7d16c748d30047c272
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: https://webkit-queues.webkit.org/results/8656037
Comment 4 WebKit Commit Bot 2018-07-25 18:29:17 PDT
Comment on attachment 345798 [details]
Patch

Clearing flags on attachment: 345798

Committed r234226: <https://trac.webkit.org/changeset/234226>
Comment 5 WebKit Commit Bot 2018-07-25 18:29:19 PDT
All reviewed patches have been landed.  Closing bug.
Comment 6 Radar WebKit Bug Importer 2018-07-25 19:23:58 PDT
<rdar://problem/42608604>