Summary: | Move Synthetic Editing Commands to behind an experimental feature flag | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Megan Gardner <megan_gardner> | ||||||||||
Component: | New Bugs | Assignee: | Megan Gardner <megan_gardner> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | commit-queue, rniwa, simon.fraser, thorton, wenson_hsieh | ||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||
Version: | WebKit Nightly Build | ||||||||||||
Hardware: | Unspecified | ||||||||||||
OS: | Unspecified | ||||||||||||
Attachments: |
|
Description
Megan Gardner
2019-06-13 13:42:58 PDT
Created attachment 372077 [details]
Patch
Comment on attachment 372077 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=372077&action=review > Source/WebCore/page/RuntimeEnabledFeatures.h:188 > + void setSyntheticEditingCommandsEnabled(bool isEnabled) { m_syntheticEditingCommandsEnabled = isEnabled; } > + bool syntheticEditingCommandsEnabled() const { return m_syntheticEditingCommandsEnabled; } Should this be Setting instead? Comment on attachment 372077 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=372077&action=review >> Source/WebCore/page/RuntimeEnabledFeatures.h:188 >> + bool syntheticEditingCommandsEnabled() const { return m_syntheticEditingCommandsEnabled; } > > Should this be Setting instead? Probably. There is no reason this needs to be enabled across the process. Created attachment 372081 [details]
Patch
Comment on attachment 372081 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=372081&action=review Now this is a setting, you can write tests for it (if we have the plumbing). > Source/WebKitLegacy/mac/WebView/WebView.mm:3165 > + RuntimeEnabledFeatures::sharedFeatures().setSyntheticEditingCommandsEnabled([preferences syntheticEditingCommandsEnabled]); Nope! Created attachment 372087 [details]
Patch
Created attachment 372137 [details]
Patch for landing
Comment on attachment 372137 [details] Patch for landing Clearing flags on attachment: 372137 Committed r246444: <https://trac.webkit.org/changeset/246444> All reviewed patches have been landed. Closing bug. |