Summary: | Subtitle menu should only appear when useful. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jeremy Jones <jeremyj-wk> | ||||||
Component: | Media | Assignee: | Jeremy Jones <jeremyj-wk> | ||||||
Status: | CLOSED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue, eric.carlson, glenn, jer.noble, philipj, sergio | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | iPhone / iPad | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Jeremy Jones
2014-12-01 09:53:57 PST
Created attachment 242319 [details]
Patch
Comment on attachment 242319 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=242319&action=review > Source/WebCore/platform/ios/WebVideoFullscreenInterfaceAVKit.mm:399 > + return [[self legibleMediaSelectionOptions] count] > numDefaultLegibleOptions; Does this also take into account audio tracks? (In reply to comment #3) > Comment on attachment 242319 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=242319&action=review > > > Source/WebCore/platform/ios/WebVideoFullscreenInterfaceAVKit.mm:399 > > + return [[self legibleMediaSelectionOptions] count] > numDefaultLegibleOptions; > > Does this also take into account audio tracks? Audio tracks are not currently implemented. Bug: 131236 I can update this change to anticipate that implementation. - return [[self audioMediaSelectionOptions] count] > 0; + return [[self audioMediaSelectionOptions] count] > 1; Comment on attachment 242319 [details]
Patch
r=me with the change noted.
Created attachment 242331 [details]
Patch
Comment on attachment 242331 [details] Patch Clearing flags on attachment: 242331 Committed r176734: <http://trac.webkit.org/changeset/176734> All reviewed patches have been landed. Closing bug. |