Bug 183301

Summary: Build fix after r229174
Product: WebKit Reporter: Christopher Reid <chris.reid>
Component: PlatformAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, ap, benjamin, cdumez, cmarcelo, commit-queue, darin, dbates, don.olmstead, ews-watchlist, himanshuarya9162, Jadewyatt26, katiemolina44, lenam7865, maxpetis92, pvollan, webkit-bug-importer, ysuzuki
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch achristensen: review+, commit-queue: commit-queue-

Description Christopher Reid 2018-03-02 13:31:48 PST
The build fails when #if !PLATFORM(COCOA) && !OS(LINUX) && !OS(WINDOWS) is true
MemoryPressureHandler::holdOff was updated to use WTF::Seconds but the MemoryPressureHandler::holdOff definition in that condition is still using unsigned.
Comment 1 Christopher Reid 2018-03-02 13:34:47 PST
Created attachment 334921 [details]
Patch
Comment 2 WebKit Commit Bot 2018-03-02 14:00:29 PST
Comment on attachment 334921 [details]
Patch

Rejecting attachment 334921 [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', 334921, '--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/6740411
Comment 3 Christopher Reid 2018-03-02 14:21:06 PST
(In reply to WebKit Commit Bot from comment #2)
> Comment on attachment 334921 [details]
> Patch
> 
> Rejecting attachment 334921 [details] from commit-queue.

It looks like the commit-queue landed this patch then marked it as rejected due to a timeout 15 seconds later.

The patch landed under https://trac.webkit.org/changeset/229187/webkit
Comment 4 Christopher Reid 2018-03-02 14:31:15 PST
Manually closing this bug. All patches have landed.
Comment 5 Radar WebKit Bug Importer 2018-03-02 14:33:06 PST
<rdar://problem/38082498>