Bug 162376

Summary:

Remove a bunch of unnecessary includes

Product:

WebKit

Reporter:

Joseph Pecoraro <joepeck>

Component:

JavaScriptCore

Assignee:

Joseph Pecoraro <joepeck>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

ap, buildbot, commit-queue, joepeck, keith_miller, mark.lam, mirandaeliza777, msaboff, rniwa, saam

Priority:

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
[PATCH] Proposed Fix	mark.lam: review+, buildbot: commit-queue-
[PATCH] For Bots	buildbot: commit-queue-
Archive of layout-test-results from ews102 for mac-yosemite	none
Archive of layout-test-results from ews104 for mac-yosemite-wk2	none

Joseph Pecoraro

Reported 2016-09-21 22:49:07 PDT

Remove a bunch of unnecessary includes and forward declarations. Found while other builds were going on in and around files I was looking at.

Attachments
[PATCH] Proposed Fix (28.02 KB, patch) 2016-09-21 22:52 PDT, Joseph Pecoraro	mark.lam: review+ buildbot: commit-queue-	Details Formatted Diff Diff
[PATCH] For Bots (27.81 KB, patch) 2016-09-21 22:59 PDT, Joseph Pecoraro	buildbot: commit-queue-	Details Formatted Diff Diff
Archive of layout-test-results from ews102 for mac-yosemite (1.00 MB, application/zip) 2016-09-22 00:01 PDT, Build Bot	no flags	Details
Archive of layout-test-results from ews104 for mac-yosemite-wk2 (985.43 KB, application/zip) 2016-09-22 00:10 PDT, Build Bot	no flags	Details
View All Add attachment proposed patch, testcase, etc.

Joseph Pecoraro

Comment 1 2016-09-21 22:52:08 PDT

Created attachment 289522 [details] [PATCH] Proposed Fix

Mark Lam

Comment 2 2016-09-21 22:56:43 PDT

Comment on attachment 289522 [details] [PATCH] Proposed Fix r=me if EWS bots are happy.

Joseph Pecoraro

Comment 3 2016-09-21 22:59:38 PDT

Created attachment 289523 [details] [PATCH] For Bots

Build Bot

Comment 4 2016-09-22 00:01:40 PDT

Comment on attachment 289522 [details] [PATCH] Proposed Fix Attachment 289522 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/2123481 New failing tests: fast/images/move-image-to-new-document.html

Build Bot

Comment 5 2016-09-22 00:01:45 PDT

Created attachment 289525 [details] Archive of layout-test-results from ews102 for mac-yosemite The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews102 Port: mac-yosemite Platform: Mac OS X 10.10.5

Build Bot

Comment 6 2016-09-22 00:10:48 PDT

Comment on attachment 289523 [details] [PATCH] For Bots Attachment 289523 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.webkit.org/results/2123502 New failing tests: fast/images/move-image-to-new-document.html

Build Bot

Comment 7 2016-09-22 00:10:51 PDT

Created attachment 289527 [details] Archive of layout-test-results from ews104 for mac-yosemite-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews104 Port: mac-yosemite-wk2 Platform: Mac OS X 10.10.5

Joseph Pecoraro

Comment 8 2016-09-26 11:47:50 PDT

https://trac.webkit.org/changeset/206267

Note You need to log in before you can comment on or make changes to this bug.