Summary: | Cleanup: Remove unnecessary #includes | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Daniel Bates <dbates> | ||||||
Component: | WebCore Misc. | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | eric, ian, laszlo.gombos, mitz | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Daniel Bates
2009-11-29 17:59:54 PST
Created attachment 44001 [details]
Patch
No functionality was changed. So, no new tests.
Created attachment 44003 [details]
Patch
Found another one:
css/CSSImportRule.cpp: MediaList.h
Comment on attachment 44003 [details]
Patch
OK. I assume you used the script?
No. I did this by hand. What script are you talking about? And if we have a script, why don't we use it? (In reply to comment #3) > (From update of attachment 44003 [details]) > OK. I assume you used the script? Committed r51473: <http://trac.webkit.org/changeset/51473> This change seems to break the QtWebkit build. Here is the build error: WebCore/css/CSSFontFaceSrcValue.cpp: In member function 'virtual void WebCore::CSSFontFaceSrcValue::addSubresourceStyleURLs(WTF::ListHashSet<WebCore::KURL, WebCore::KURLHash>&, const WebCore::CSSStyleSheet*)': WebCore/css/CSSFontFaceSrcValue.cpp:74: error: 'addSubresourceURL' was not declared in this scope make[1]: *** [obj/release/CSSFontFaceSrcValue.o] Error 1 Daniel, can you check this ? This is now fixed by the following change: http://trac.webkit.org/changeset/51520 |