The attached patch is inteded to fix a layering violation, usage of FrameLoaderClientQt, in WebCore/platform/graphics/MediaPlayerPrivateQt.cpp.
Created attachment 79465 [details] proposed patch...
Comment on attachment 79465 [details] proposed patch... Patch itself looks good, but we need a ChangeLog entry.
Created attachment 79532 [details] proposed patch II... Added missing Changelog.
Comment on attachment 79532 [details] proposed patch II... View in context: https://bugs.webkit.org/attachment.cgi?id=79532&action=review r- as the ChangeLog needs to be fixed. > Source/WebCore/ChangeLog:5 > + [Qt] Fix Layering violation in MediaPlayerPrivateQt... One dot please. > Source/WebCore/ChangeLog:8 > + No new tests. (OOPS!) OOPS would need to go - from http://webkit.org/coding/contributing.html The "No new tests. (OOPS!)" line appears if prepare-ChangeLog did not detect the addition of test cases. If your patch does not require test cases (or test cases are not possible), you should include a line stating such. Otherwise all changes require test cases which should be mentioned in the ChangeLog.
Created attachment 79554 [details] proposed patch III... Fix changelog file some more...
Comment on attachment 79554 [details] proposed patch III... OK. Can we remove the include of FrameLoaderClientQt.h?
Created attachment 79601 [details] proposed patch IV... Removed the FrameLoaderClientQt.h include...
Comment on attachment 79601 [details] proposed patch IV... Clearing flags on attachment: 79601 Committed r76265: <http://trac.webkit.org/changeset/76265>
All reviewed patches have been landed. Closing bug.