Bug 191049 - Move 'ProcessSwapOnCrossSiteNavigationEnabled' to 'experimental' category.
Summary: Move 'ProcessSwapOnCrossSiteNavigationEnabled' to 'experimental' category.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-10-29 16:44 PDT by dewei_zhu
Modified: 2018-10-30 17:38 PDT (History)
6 users (show)

See Also:


Attachments
Patch (1.35 KB, patch)
2018-10-29 16:45 PDT, dewei_zhu
no flags Details | Formatted Diff | Diff
Patch for landing (1.35 KB, patch)
2018-10-29 16:50 PDT, dewei_zhu
no flags Details | Formatted Diff | Diff
Patch (1.99 KB, patch)
2018-10-30 17:17 PDT, dewei_zhu
dino: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description dewei_zhu 2018-10-29 16:44:45 PDT
Move 'ProcessSwapOnCrossSiteNavigationEnabled' to 'experimental' category.
Comment 1 dewei_zhu 2018-10-29 16:45:51 PDT
Created attachment 353333 [details]
Patch
Comment 2 Chris Dumez 2018-10-29 16:48:07 PDT
Comment on attachment 353333 [details]
Patch

r=me
Comment 3 Chris Dumez 2018-10-29 16:49:29 PDT
Comment on attachment 353333 [details]
Patch

Clearing flags on attachment: 353333

Committed r237573: <https://trac.webkit.org/changeset/237573>
Comment 4 Chris Dumez 2018-10-29 16:49:30 PDT
All reviewed patches have been landed.  Closing bug.
Comment 5 dewei_zhu 2018-10-29 16:50:08 PDT
Reopening to attach new patch.
Comment 6 dewei_zhu 2018-10-29 16:50:09 PDT
Created attachment 353334 [details]
Patch for landing
Comment 7 Radar WebKit Bug Importer 2018-10-29 16:50:32 PDT
<rdar://problem/45654625>
Comment 8 WebKit Commit Bot 2018-10-29 16:51:39 PDT
Comment on attachment 353334 [details]
Patch for landing

Rejecting attachment 353334 [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', 353334, '--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=353334&action=edit
Fetching: https://bugs.webkit.org/show_bug.cgi?id=191049&ctype=xml&excludefield=attachmentdata
Processing 1 patch from 1 bug.
Processing patch 353334 from bug 191049.
Fetching: https://bugs.webkit.org/attachment.cgi?id=353334
Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Parsed 2 diffs from patch file(s).
patching file Source/WebKit/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file Source/WebKit/Shared/WebPreferences.yaml
Hunk #1 FAILED at 1350.
1 out of 1 hunk FAILED -- saving rejects to file Source/WebKit/Shared/WebPreferences.yaml.rej

Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Full output: https://webkit-queues.webkit.org/results/9773376
Comment 9 Chris Dumez 2018-10-29 16:53:26 PDT
I already landed it in https://trac.webkit.org/changeset/237573
Comment 10 Dean Jackson 2018-10-30 10:22:28 PDT
Comment on attachment 353334 [details]
Patch for landing

Could you also move the entire feature into the section with other experimental features? (A few lines above this)
Comment 11 dewei_zhu 2018-10-30 11:28:13 PDT
Will do in a followup fix.
Comment 12 dewei_zhu 2018-10-30 17:17:49 PDT
Reopening to attach new patch.
Comment 13 dewei_zhu 2018-10-30 17:17:50 PDT
Created attachment 353445 [details]
Patch
Comment 14 Dean Jackson 2018-10-30 17:31:23 PDT
Comment on attachment 353445 [details]
Patch

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

> Source/WebKit/Shared/WebPreferences.yaml:1340
> +type: bool
> +defaultValue: false
> +humanReadableName: "Swap Processes on Cross-Site Navigation"
> +humanReadableDescription: "Swap WebContent processes on cross-site navigations"
> +category: experimental
> +webcoreBinding: none

Nit: indentation.
Comment 15 dewei_zhu 2018-10-30 17:34:19 PDT
Comment on attachment 353445 [details]
Patch

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

>> Source/WebKit/Shared/WebPreferences.yaml:1340
>> +webcoreBinding: none
> 
> Nit: indentation.

Nice catch. Tricked by Xcode.
Comment 16 dewei_zhu 2018-10-30 17:38:19 PDT
Landed in r237611.