Bug 122345 - [iOS] Upstream support for -webkit-video-playable-inline/webkit-playsinline
Summary: [iOS] Upstream support for -webkit-video-playable-inline/webkit-playsinline
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Daniel Bates
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-10-04 13:20 PDT by Daniel Bates
Modified: 2016-05-12 12:31 PDT (History)
11 users (show)

See Also:


Attachments
Patch (11.59 KB, patch)
2013-10-04 13:23 PDT, Daniel Bates
sam: review-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Bates 2013-10-04 13:20:16 PDT
Upstream support for CSS media query -webkit-video-playable-inline and the HTML attribute webkit-playsinline.
Comment 1 Daniel Bates 2013-10-04 13:23:41 PDT
Created attachment 213396 [details]
Patch
Comment 2 WebKit Commit Bot 2013-10-04 13:25:36 PDT
Attachment 213396 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/platform/iphone-simulator/iphone/fast/media/video-inline-expected.txt', u'LayoutTests/platform/iphone-simulator/iphone/fast/media/video-inline.html', u'Source/WebCore/ChangeLog', u'Source/WebCore/css/MediaFeatureNames.h', u'Source/WebCore/css/MediaQueryEvaluator.cpp', u'Source/WebCore/html/HTMLAttributeNames.in', u'Source/WebCore/html/HTMLMediaElement.cpp', u'Source/WebCore/html/HTMLMediaElement.h', u'Source/WebKit/mac/ChangeLog', u'Source/WebKit/mac/WebView/WebPreferences.mm']" exit_code: 1
Source/WebCore/css/MediaQueryEvaluator.cpp:638:  video_playable_inlineMediaFeatureEval is incorrectly named. Don't use underscores in your identifier names.  [readability/naming/underscores] [4]
Total errors found: 1 in 11 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Sam Weinig 2013-10-07 11:30:00 PDT
Comment on attachment 213396 [details]
Patch

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

I don't see any reason for this to iOS specific.

> Source/WebCore/css/MediaFeatureNames.h:34
> +#if PLATFORM(IOS)

This should be ENABLE(...)

> Source/WebCore/css/MediaFeatureNames.h:35
> +#define IOS_VIDEO_PLAYABLE_INLINE(macro) macro(video_playable_inline, "-webkit-video-playable-inline")

This should not have an iOS prefix.