RESOLVED FIXED 117926
Remove unneeded include header files from WebCore 
https://bugs.webkit.org/show_bug.cgi?id=117926
Summary Remove unneeded include header files from WebCore
Gyuyoung Kim
Reported 2013-06-24 02:53:44 PDT
We don't need to include unused headers. See also https://chromiumcodereview.appspot.com/17388003.
Attachments
Patch for ews (58.29 KB, patch)
2013-06-24 02:55 PDT, Gyuyoung Kim 		no flags
DetailsFormatted DiffDiff
Patch for ews (58.27 KB, patch)
2013-06-24 03:43 PDT, Gyuyoung Kim 		no flags
DetailsFormatted DiffDiff
Patch (58.26 KB, patch)
2013-06-24 07:10 PDT, Gyuyoung Kim 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Gyuyoung Kim
Comment 1 2013-06-24 02:55:19 PDT
Created attachment 205281 [details] Patch for ews
Build Bot
Comment 2 2013-06-24 03:03:13 PDT
Comment on attachment 205281 [details] Patch for ews Attachment 205281 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/871362
Build Bot
Comment 3 2013-06-24 03:33:40 PDT
Comment on attachment 205281 [details] Patch for ews Attachment 205281 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/916323
Gyuyoung Kim
Comment 4 2013-06-24 03:43:04 PDT
Created attachment 205283 [details] Patch for ews
Build Bot
Comment 5 2013-06-24 04:10:43 PDT
Comment on attachment 205283 [details] Patch for ews Attachment 205283 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/977142
Build Bot
Comment 6 2013-06-24 04:23:46 PDT
Comment on attachment 205283 [details] Patch for ews Attachment 205283 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/871376
Gyuyoung Kim
Comment 7 2013-06-24 07:10:21 PDT
Created attachment 205295 [details] Patch
Gyuyoung Kim
Comment 8 2013-06-24 18:37:05 PDT
Comment on attachment 205295 [details] Patch Clearing flags on attachment: 205295 Committed r151947: <http://trac.webkit.org/changeset/151947>
Gyuyoung Kim
Comment 9 2013-06-24 18:37:12 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.