Bug 153590

Summary: [GStreamer] Clean up includes and headers related with GStreamerGL
Product: WebKit Reporter: Gwang Yoon Hwang <yoon>
Component: New BugsAssignee: Gwang Yoon Hwang <yoon>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, pnormand, zan
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
pnormand: review+
Patch none

Description Gwang Yoon Hwang 2016-01-28 05:11:39 PST
[GStreamer] Clean up includes and headers related with GStreamerGL
Comment 1 Gwang Yoon Hwang 2016-01-28 05:13:09 PST
Created attachment 270108 [details]
Patch
Comment 2 Philippe Normand 2016-01-28 05:44:46 PST
Comment on attachment 270108 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=270108&action=review

Neat webkitpy changes!

> Source/WebCore/ChangeLog:8
> +        Removes gstglmemory from the including list and reorder includes to

Remove

> Source/WebCore/ChangeLog:11
> +        without voilating it.

typo: voilating
Comment 3 Philippe Normand 2016-01-28 05:46:16 PST
Comment on attachment 270108 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=270108&action=review

> Tools/ChangeLog:10
> +        Removes VideoSinkGStreamer1.cpp which is not exist anymore.

which doesn't exist anymore
Comment 4 Gwang Yoon Hwang 2016-01-28 06:04:18 PST
Created attachment 270111 [details]
Patch
Comment 5 WebKit Commit Bot 2016-01-28 07:16:10 PST
Comment on attachment 270111 [details]
Patch

Clearing flags on attachment: 270111

Committed r195736: <http://trac.webkit.org/changeset/195736>
Comment 6 Philippe Normand 2016-05-14 08:06:21 PDT
This one can be closed I suppose!