Bug 57974

Summary:

[Qt] We should use USE(QT_MULTIMEDIA) rather than ENABLE(QT_MULTIMEDIA).

Product:

WebKit

Reporter:

Alexis Menard (darktears) <menard>

Component:

WebKit Qt

Assignee:

Alexis Menard (darktears) <menard>

Status:

RESOLVED FIXED

Severity:

Minor

CC:

benjamin, commit-queue, kling, vestbo

Priority:

Keywords:

Qt, QtTriaged

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Proposed patch.	none
V2 with changelog's typo fixed.	none

Description Alexis Menard (darktears) 2011-04-06 13:01:18 PDT

We should use USE(QT_MULTIMEDIA) rather than ENABLE(QT_MULTIMEDIA), it sounds better and is aligned with what the others do.

Comment 1 Alexis Menard (darktears) 2011-04-06 13:03:32 PDT

Created attachment 88495 [details]
Proposed patch.

Comment 2 Alexis Menard (darktears) 2011-04-06 13:07:11 PDT

Created attachment 88496 [details]
V2 with changelog's typo fixed.

Comment 3 Andreas Kling 2011-04-06 13:09:02 PDT

Comment on attachment 88496 [details]
V2 with changelog's typo fixed.

Good idea.

Comment 4 WebKit Commit Bot 2011-04-06 15:17:31 PDT

Comment on attachment 88496 [details]
V2 with changelog's typo fixed.

Clearing flags on attachment: 88496

Committed r83108: <http://trac.webkit.org/changeset/83108>

Comment 5 WebKit Commit Bot 2011-04-06 15:17:38 PDT

All reviewed patches have been landed.  Closing bug.