Bug 130645

Summary: Clean up #includes by geometry classes
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: New BugsAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: bunhere, cmarcelo, commit-queue, esprehn+autocc, glenn, gyuyoung.kim, kling, kondapallykalyan, luiz, macpherson, menard, noam, rakuco, sam, sergio, simon.fraser, zeno
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
For EWS
none
For EWS
none
For EWS
none
For EWS again
none
Will it never end?
none
Apparently not
none
Sigh
none
Trying again
none
Yep
none
We love you EFL
none
We love you EFL even more none

Description Simon Fraser (smfr) 2014-03-22 13:09:54 PDT
Clean up #includes by geometry classes
Comment 1 Simon Fraser (smfr) 2014-03-22 13:11:59 PDT
Created attachment 227566 [details]
Patch
Comment 2 Andreas Kling 2014-03-22 13:15:33 PDT
Comment on attachment 227566 [details]
Patch

Great! r=me if it builds :)
Comment 3 WebKit Commit Bot 2014-03-22 13:48:22 PDT
Comment on attachment 227566 [details]
Patch

Rejecting attachment 227566 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-02', 'build', '--no-clean', '--no-update', '--build-style=release', '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 500 characters of output:
ang.1_0.compiler
	CompileC /Volumes/Data/EWS/WebKit/WebKitBuild/WebCore.build/Release/WebCore.build/Objects-normal/x86_64/AccessibilityMockObject.o accessibility/AccessibilityMockObject.cpp normal x86_64 c++ com.apple.compilers.llvm.clang.1_0.compiler
	CompileC /Volumes/Data/EWS/WebKit/WebKitBuild/WebCore.build/Release/WebCore.build/Objects-normal/x86_64/AccessibilitySpinButton.o accessibility/AccessibilitySpinButton.cpp normal x86_64 c++ com.apple.compilers.llvm.clang.1_0.compiler
(7 failures)

Full output: http://webkit-queues.appspot.com/results/6569124461281280
Comment 4 Simon Fraser (smfr) 2014-03-22 14:14:53 PDT
Created attachment 227571 [details]
For EWS
Comment 5 Simon Fraser (smfr) 2014-03-22 16:55:26 PDT
Created attachment 227579 [details]
For EWS
Comment 6 Simon Fraser (smfr) 2014-03-22 17:10:57 PDT
Created attachment 227582 [details]
For EWS
Comment 7 Simon Fraser (smfr) 2014-03-22 17:58:27 PDT
Created attachment 227588 [details]
For EWS again
Comment 8 Simon Fraser (smfr) 2014-03-22 18:14:42 PDT
Created attachment 227589 [details]
Will it never end?
Comment 9 Simon Fraser (smfr) 2014-03-22 18:29:54 PDT
Created attachment 227591 [details]
Apparently not
Comment 10 Simon Fraser (smfr) 2014-03-22 18:39:55 PDT
Created attachment 227593 [details]
Sigh
Comment 11 Simon Fraser (smfr) 2014-03-22 19:07:57 PDT
Created attachment 227594 [details]
Trying again
Comment 12 Simon Fraser (smfr) 2014-03-22 20:58:49 PDT
Created attachment 227597 [details]
Yep
Comment 13 Simon Fraser (smfr) 2014-03-22 22:31:41 PDT
Created attachment 227603 [details]
We love you EFL
Comment 14 Simon Fraser (smfr) 2014-03-22 23:51:36 PDT
Created attachment 227606 [details]
We love you EFL even more
Comment 15 Andreas Kling 2014-03-23 01:05:18 PDT
Comment on attachment 227606 [details]
We love you EFL even more

Sending her off to landing.
Comment 16 WebKit Commit Bot 2014-03-23 01:36:40 PDT
Comment on attachment 227606 [details]
We love you EFL even more

Clearing flags on attachment: 227606

Committed r166138: <http://trac.webkit.org/changeset/166138>
Comment 17 WebKit Commit Bot 2014-03-23 01:36:48 PDT
All reviewed patches have been landed.  Closing bug.
Comment 18 Simon Fraser (smfr) 2014-03-23 13:09:45 PDT
http://trac.webkit.org/changeset/166139 is what I get by doing this before coffee.