Bug 185014 – Use unified build for UIProcess

• Component: WebKit2
• Status: RESOLVED FIXED

[Michael Catanzaro]

Use unified build for UIProcess

[Tim Horton]

Created attachment 347957 [details]
Patch

[EWS Watchlist]

Attachment 347957 [details] did not pass style-queue:


ERROR: Source/WebKit/UIProcess/mac/WKImmediateActionController.mm:404:  'page' is incorrectly named. It should be named 'protector' or 'protected_page'.  [readability/naming/protected] [4]
ERROR: Source/WebKit/UIProcess/mac/WKImmediateActionController.mm:440:  'page' is incorrectly named. It should be named 'protector' or 'protected_page'.  [readability/naming/protected] [4]
Total errors found: 2 in 110 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Tim Horton]

Created attachment 347961 [details]
Patch

[EWS Watchlist]

Attachment 347961 [details] did not pass style-queue:


ERROR: Source/WebKit/UIProcess/mac/WKImmediateActionController.mm:404:  'page' is incorrectly named. It should be named 'protector' or 'protected_page'.  [readability/naming/protected] [4]
ERROR: Source/WebKit/UIProcess/mac/WKImmediateActionController.mm:440:  'page' is incorrectly named. It should be named 'protector' or 'protected_page'.  [readability/naming/protected] [4]
Total errors found: 2 in 110 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Tim Horton]

Created attachment 347962 [details]
Patch

[EWS Watchlist]

Attachment 347962 [details] did not pass style-queue:


ERROR: Source/WebKit/UIProcess/mac/WKImmediateActionController.mm:404:  'page' is incorrectly named. It should be named 'protector' or 'protected_page'.  [readability/naming/protected] [4]
ERROR: Source/WebKit/UIProcess/mac/WKImmediateActionController.mm:440:  'page' is incorrectly named. It should be named 'protector' or 'protected_page'.  [readability/naming/protected] [4]
Total errors found: 2 in 111 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[WebKit Commit Bot]

Comment on attachment 347962 [details]
Patch

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

Last 5000 characters of output:
 file Source/WebKit/UIProcess/VisitedLinkStore.cpp
patching file Source/WebKit/UIProcess/WKInspectorHighlightView.mm
patching file Source/WebKit/UIProcess/WebBackForwardList.cpp
patching file Source/WebKit/UIProcess/WebContextMenuListenerProxy.cpp
patching file Source/WebKit/UIProcess/WebCookieManagerProxy.cpp
patching file Source/WebKit/UIProcess/WebEditCommandProxy.cpp
patching file Source/WebKit/UIProcess/WebFrameProxy.cpp
patching file Source/WebKit/UIProcess/WebFullScreenManagerProxy.cpp
patching file Source/WebKit/UIProcess/WebInspectorProxy.cpp
patching file Source/WebKit/UIProcess/WebNavigationState.cpp
patching file Source/WebKit/UIProcess/WebOpenPanelResultListenerProxy.cpp
patching file Source/WebKit/UIProcess/WebPageInjectedBundleClient.cpp
patching file Source/WebKit/UIProcess/WebPageProxy.cpp
patching file Source/WebKit/UIProcess/WebProcessPool.cpp
patching file Source/WebKit/UIProcess/WebProcessProxy.cpp
patching file Source/WebKit/UIProcess/WebStorage/LocalStorageDatabaseTracker.cpp
patching file Source/WebKit/UIProcess/WebURLSchemeHandler.cpp
patching file Source/WebKit/UIProcess/WebURLSchemeTask.cpp
patching file Source/WebKit/UIProcess/gtk/AcceleratedBackingStoreWayland.cpp
patching file Source/WebKit/UIProcess/gtk/AcceleratedBackingStoreX11.cpp
patching file Source/WebKit/UIProcess/ios/DragDropInteractionState.mm
patching file Source/WebKit/UIProcess/ios/InputViewUpdateDeferrer.mm
patching file Source/WebKit/UIProcess/ios/PageClientImplIOS.mm
patching file Source/WebKit/UIProcess/ios/SmartMagnificationController.mm
patching file Source/WebKit/UIProcess/ios/TextCheckerIOS.mm
patching file Source/WebKit/UIProcess/ios/ViewGestureControllerIOS.mm
patching file Source/WebKit/UIProcess/ios/WKActionSheetAssistant.mm
patching file Source/WebKit/UIProcess/ios/WKApplicationStateTrackingView.mm
patching file Source/WebKit/UIProcess/ios/WKGeolocationProviderIOS.mm
patching file Source/WebKit/UIProcess/ios/WKKeyboardScrollingAnimator.mm
patching file Source/WebKit/UIProcess/ios/WKLegacyPDFView.mm
patching file Source/WebKit/UIProcess/ios/WKPDFView.mm
patching file Source/WebKit/UIProcess/ios/WKScrollView.mm
patching file Source/WebKit/UIProcess/ios/WKSystemPreviewView.mm
patching file Source/WebKit/UIProcess/ios/WebPageProxyIOS.mm
patching file Source/WebKit/UIProcess/mac/CorrectionPanel.mm
patching file Source/WebKit/UIProcess/mac/PageClientImplMac.mm
patching file Source/WebKit/UIProcess/mac/RemoteWebInspectorProxyMac.mm
patching file Source/WebKit/UIProcess/mac/TextCheckerMac.mm
patching file Source/WebKit/UIProcess/mac/TiledCoreAnimationDrawingAreaProxy.mm
patching file Source/WebKit/UIProcess/mac/ViewGestureControllerMac.mm
patching file Source/WebKit/UIProcess/mac/ViewSnapshotStore.mm
patching file Source/WebKit/UIProcess/mac/WKFullKeyboardAccessWatcher.mm
patching file Source/WebKit/UIProcess/mac/WKFullScreenWindowController.mm
patching file Source/WebKit/UIProcess/mac/WKImmediateActionController.mm
patching file Source/WebKit/UIProcess/mac/WKInspectorViewController.mm
patching file Source/WebKit/UIProcess/mac/WKPrintingView.mm
patching file Source/WebKit/UIProcess/mac/WKTextFinderClient.mm
patching file Source/WebKit/UIProcess/mac/WKTextInputWindowController.mm
patching file Source/WebKit/UIProcess/mac/WKViewLayoutStrategy.mm
patching file Source/WebKit/UIProcess/mac/WebColorPickerMac.mm
patching file Source/WebKit/UIProcess/mac/WebContextMenuProxyMac.mm
patching file Source/WebKit/UIProcess/mac/WebInspectorProxyMac.mm
patching file Source/WebKit/UIProcess/mac/WebPageProxyMac.mm
patching file Source/WebKit/UIProcess/mac/WebPopupMenuProxyMac.mm
patching file Source/WebKit/UIProcess/win/PageClientImpl.cpp
patching file Source/WebKit/UIProcess/win/TextCheckerWin.cpp
patching file Source/WebKit/UIProcess/win/WebContextMenuProxyWin.cpp
patching file Source/WebKit/UIProcess/win/WebPopupMenuProxyWin.cpp
patching file Source/WebKit/UIProcess/win/WebView.cpp
patching file Source/WebKit/UIProcess/wpe/TextCheckerWPE.cpp
patching file Source/WebKit/UIProcess/wpe/WebPasteboardProxyWPE.cpp
patching file Source/WebKit/WebKit.xcodeproj/project.pbxproj

Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit
Updating OpenSource
From https://git.webkit.org/git/WebKit
   973aa89455c..d7cb7b5eed1  master     -> origin/master
Partial-rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc ...
Currently at 235257 = 973aa89455c79e2e8f42fc9694c50f2b1fa29151
r235258 = 1e8ac4869b84e040b52f333d13b10143fc946ccc
r235259 = 5afc1d04cc556af31b74618b3d04255e6db777d3
r235260 = 69e00bb40762e37185478f0000bf7845147eb20c
r235261 = ebc7eff21c81f858a2254cec49419e7a0783dc43
r235262 = d7cb7b5eed19afb44d76d1b02f74f937a3ac7ec8
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/8965187

[Tim Horton]

Created attachment 347978 [details]
Patch

[EWS Watchlist]

Attachment 347978 [details] did not pass style-queue:


ERROR: Source/WebKit/UIProcess/mac/WKImmediateActionController.mm:404:  'page' is incorrectly named. It should be named 'protector' or 'protected_page'.  [readability/naming/protected] [4]
ERROR: Source/WebKit/UIProcess/mac/WKImmediateActionController.mm:440:  'page' is incorrectly named. It should be named 'protector' or 'protected_page'.  [readability/naming/protected] [4]
Total errors found: 2 in 111 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[WebKit Commit Bot]

Comment on attachment 347978 [details]
Patch

Clearing flags on attachment: 347978

Committed r235265: <https://trac.webkit.org/changeset/235265>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/43667139>