WebKit Style specifies that a header file should include all header files it relies on except for config.h QWebPageClient.h does not adhere to this standard in regards to QCursor Patch coming soon.
Created attachment 43261 [details] Proposed Patch
Comment on attachment 43261 [details] Proposed Patch Rejecting patch 43261 from commit-queue. Failed to run "['WebKitTools/Scripts/run-webkit-tests', '--no-launch-safari', '--quiet', '--exit-after-n-failures=1']" exit_code: 1 Running build-dumprendertree Running tests from /Users/eseidel/Projects/CommitQueue/LayoutTests Testing 11628 test cases. inspector/console-dir.html -> crashed Exiting early after 1 failures. 9322 tests run. 367.50s total testing time 9321 test cases (99%) succeeded 1 test case (<1%) crashed 5 test cases (<1%) had stderr output
Comment on attachment 43261 [details] Proposed Patch Sigh. Bug 31461.
Comment on attachment 43261 [details] Proposed Patch Clearing flags on attachment: 43261 Committed r51137: <http://trac.webkit.org/changeset/51137>
All reviewed patches have been landed. Closing bug.