Summary: | Expose still more WKPreferences SPI to match C SPI | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jeff Miller <jeffm> | ||||||
Component: | WebKit API | Assignee: | Jeff Miller <jeffm> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | achristensen, bugs-noreply, commit-queue, jeffm, jer.noble, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Jeff Miller
2018-02-22 10:38:49 PST
Correction, change _requiresUserActionForAudioPlayback to _requiresUserGestureForAudioPlayback and _requiresUserActionForVideoPlayback to _requiresUserGestureForVideoPlayback. Created attachment 334486 [details]
Patch
Comment on attachment 334486 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=334486&action=review > Source/WebKit/UIProcess/API/Cocoa/WKPreferences.mm:1091 > + WebCore::DeprecatedGlobalSettings::setLowPowerVideoAudioBufferSizeEnabled(enabled); This is probably setting a bool in the wrong process. > Source/WebKit/UIProcess/API/Cocoa/WKPreferencesPrivate.h:166 > +@property (nonatomic, setter=_setAVFoundationNSURLSessionEnabled:) BOOL _avFoundationNSURLSessionEnabled WK_API_AVAILABLE(macosx(WK_MAC_TBA)); This should always be enabled. (In reply to Alex Christensen from comment #4) > Comment on attachment 334486 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=334486&action=review > > > Source/WebKit/UIProcess/API/Cocoa/WKPreferences.mm:1091 > > + WebCore::DeprecatedGlobalSettings::setLowPowerVideoAudioBufferSizeEnabled(enabled); > > This is probably setting a bool in the wrong process. Good point. I don't think we can easily change this from the UI process. I'll remove this and talk to Jer about what he wants to do, e.g. maybe this deprecated setting should default to true anyway. > > > Source/WebKit/UIProcess/API/Cocoa/WKPreferencesPrivate.h:166 > > +@property (nonatomic, setter=_setAVFoundationNSURLSessionEnabled:) BOOL _avFoundationNSURLSessionEnabled WK_API_AVAILABLE(macosx(WK_MAC_TBA)); > > This should always be enabled. OK, I'll remove this as well. Created attachment 334537 [details]
Patch
Comment on attachment 334537 [details] Patch Rejecting attachment 334537 [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', 334537, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: rdparty/autoinstalled/mechanize/_urllib2_fork.py", line 332, in _call_chain result = func(*args) File "/Volumes/Data/EWS/WebKit/Tools/Scripts/webkitpy/thirdparty/autoinstalled/mechanize/_urllib2_fork.py", line 1170, in https_open return self.do_open(conn_factory, req) File "/Volumes/Data/EWS/WebKit/Tools/Scripts/webkitpy/thirdparty/autoinstalled/mechanize/_urllib2_fork.py", line 1118, in do_open raise URLError(err) urllib2.URLError: <urlopen error [Errno 60] Operation timed out> Full output: http://webkit-queues.webkit.org/results/6816090 Committed <https://trac.webkit.org/changeset/229294/webkit>. |