[Webkit-unassigned] [Bug 123740] [webcore/html] remove extra header includes from cpp files.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Nov 4 15:28:54 PST 2013


https://bugs.webkit.org/show_bug.cgi?id=123740


Darin Adler <darin at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #215924|review?                     |review+
               Flag|                            |




--- Comment #8 from Darin Adler <darin at apple.com>  2013-11-04 15:27:41 PST ---
(From update of attachment 215924)
View in context: https://bugs.webkit.org/attachment.cgi?id=215924&action=review

> Source/WebCore/html/canvas/CanvasRenderingContext2D.cpp:62
> -#include <runtime/Uint8ClampedArray.h>
>  

We’re leaving an extra blank line here that we should delete.

> Source/WebCore/html/shadow/MediaControlsApple.cpp:37
> -#if ENABLE(VIDEO_TRACK)
> -#include "TextTrackCue.h"
> -#endif
>  

We’re leaving an extra blank line here that we should delete.

> Source/WebCore/html/shadow/MediaControlsBlackBerry.cpp:41
> -#if ENABLE(VIDEO_TRACK)
> -#include "TextTrackCue.h"
> -#endif
>  

We’re leaving an extra blank line here that we should delete.

> Source/WebCore/html/shadow/MediaControlsGtk.cpp:34
> -#include "Chrome.h"
> -#include "MouseEvent.h"
>  

We’re leaving an extra blank line here that we should delete.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the webkit-unassigned mailing list