Bug 106812 - [GStreamer] Fix indentation for GStreamer supported MIME types list
Summary: [GStreamer] Fix indentation for GStreamer supported MIME types list
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 98759
  Show dependency tree / graph
 
Reported: 2013-01-14 11:42 PST by Adrian Perez
Modified: 2013-01-14 13:16 PST (History)
7 users (show)

See Also:

Attachments
Patch (7.48 KB, patch)
2013-01-14 11:49 PST, Adrian Perez 		no flags Details | Formatted Diff | Diff
Patch (7.49 KB, patch)
2013-01-14 12:30 PST, Adrian Perez 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Adrian Perez 2013-01-14 11:42:57 PST 
[GTK] Fix indentation for GStreamer supported MIME types list
Comment 1 Adrian Perez 2013-01-14 11:48:08 PST 
Adding dependency.
Comment 2 Adrian Perez 2013-01-14 11:49:12 PST 
Created attachment 182605 [details]
Patch
Comment 3 Philippe Normand 2013-01-14 12:01:04 PST 
Comment on attachment 182605 [details]
Patch

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

> Source/WebCore/ChangeLog:8
> +        No new tests (OOPS!).

You can remove this line.
Comment 4 Eric Carlson 2013-01-14 12:11:18 PST 
Comment on attachment 182605 [details]
Patch

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

>> Source/WebCore/ChangeLog:8
>> +        No new tests (OOPS!).
> 
> You can remove this line.

Yes, I think the commit-hook will reject the patch if you don't remove it.
Comment 5 Adrian Perez 2013-01-14 12:30:54 PST 
Created attachment 182611 [details]
Patch
Comment 6 Adrian Perez 2013-01-14 12:35:17 PST 
(In reply to comment #3)
> (From update of attachment 182605 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=182605&action=review
> 
> > Source/WebCore/ChangeLog:8
> > +        No new tests (OOPS!).
> 
> You can remove this line.

Better, I have rewritten it as "No tests. No change in behavior."
(like in other changelog entries).
Comment 7 WebKit Review Bot 2013-01-14 13:15:57 PST 
Comment on attachment 182611 [details]
Patch

Clearing flags on attachment: 182611

Committed r139650: <http://trac.webkit.org/changeset/139650>
Comment 8 WebKit Review Bot 2013-01-14 13:16:01 PST 
All reviewed patches have been landed.  Closing bug.