Bug 193698 - Clean up USE(WEB_THREAD)
Summary: Clean up USE(WEB_THREAD)
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Local Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Conrad Shultz
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2019-01-22 16:53 PST by Conrad Shultz
Modified: 2019-01-23 13:04 PST (History)
4 users (show)

See Also:


Attachments
Patch (51.14 KB, patch)
2019-01-22 22:52 PST, Conrad Shultz
thorton: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Conrad Shultz 2019-01-22 16:53:12 PST
We do not apply USE(WEB_THREAD) consistently. It would be good to audit all uses of the web thread and patch up relevant sites.
Comment 1 Conrad Shultz 2019-01-22 16:53:47 PST
<rdar://problem/47366779>
Comment 2 Conrad Shultz 2019-01-22 22:52:11 PST
Created attachment 359858 [details]
Patch
Comment 3 EWS Watchlist 2019-01-22 22:55:22 PST
Attachment 359858 [details] did not pass style-queue:


ERROR: Source/WebKitLegacy/ios/WebCoreSupport/WebFixedPositionContent.mm:173:  The parameter name "frame" adds no information, so it should be removed.  [readability/parameter_name] [5]
Total errors found: 1 in 21 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 4 Tim Horton 2019-01-23 12:28:06 PST
Comment on attachment 359858 [details]
Patch

ears=me
Comment 5 Conrad Shultz 2019-01-23 12:30:38 PST
The style issue is tracked by "check-webkit-style false positive for [readability/parameter_name] with declaration inside if expression." (https://bugs.webkit.org/show_bug.cgi?id=174485)
Comment 6 Conrad Shultz 2019-01-23 12:37:18 PST
Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	Source/WebCore/page/CaptionUserPreferencesMediaAF.cpp
	M	Source/WebCore/platform/cf/MainThreadSharedTimerCF.cpp
	M	Source/WebCore/platform/cocoa/ContentFilterUnblockHandlerCocoa.mm
	M	Source/WebCore/platform/graphics/cocoa/TextTrackRepresentationCocoa.mm
	M	Source/WebCore/platform/ios/wak/WAKWindow.mm
	M	Source/WebCore/platform/ios/LegacyTileCache.mm
	M	Source/WebCore/platform/ios/WebCoreMotionManager.mm
	M	Source/WebCore/platform/ios/WebVideoFullscreenControllerAVKit.mm
	M	Source/WebCore/platform/network/ios/NetworkStateNotifierIOS.mm
	M	Source/WebCore/rendering/RenderThemeIOS.mm
	M	Source/WebCore/ChangeLog
	M	Source/WebKitLegacy/ios/Misc/WebGeolocationProviderIOS.mm
	M	Source/WebKitLegacy/ios/WebCoreSupport/WebFixedPositionContent.mm
	M	Source/WebKitLegacy/ios/ChangeLog
	M	Source/WebKitLegacy/mac/Misc/WebCache.mm
	M	Source/WebKitLegacy/mac/WebCoreSupport/WebFrameLoaderClient.mm
	M	Source/WebKitLegacy/mac/WebCoreSupport/WebGeolocationClient.mm
	M	Source/WebKitLegacy/mac/WebView/WebFrame.mm
	M	Source/WebKitLegacy/mac/WebView/WebHTMLView.mm
	M	Source/WebKitLegacy/mac/WebView/WebView.mm
	M	Source/WebKitLegacy/mac/ChangeLog
r240353 = 24ba8bb9e1dad8679b492d9a2d47da619be85789 (refs/remotes/origin/master)
Comment 7 Simon Fraser (smfr) 2019-01-23 12:39:33 PST
I don't like the #ifdeffing of control flow here. Can't we make WebThreadRun just execute the block synchronously?
Comment 8 Conrad Shultz 2019-01-23 13:04:06 PST
(In reply to Simon Fraser (smfr) from comment #7)
> I don't like the #ifdeffing of control flow here. Can't we make WebThreadRun
> just execute the block synchronously?

Understood. Per discussion, rolled out the change in r240355.