Bug 147842 - HTMLMediaElement needs way to find MediaDeviceInfo keys
Summary: HTMLMediaElement needs way to find MediaDeviceInfo keys
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Matthew Daiter
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-08-10 14:29 PDT by Matthew Daiter
Modified: 2015-08-10 16:28 PDT (History)
9 users (show)

See Also:


Attachments
Patch (4.71 KB, patch)
2015-08-10 14:32 PDT, Matthew Daiter
no flags Details | Formatted Diff | Diff
Patch (4.67 KB, patch)
2015-08-10 14:41 PDT, Matthew Daiter
no flags Details | Formatted Diff | Diff
Patch (4.60 KB, patch)
2015-08-10 15:16 PDT, Matthew Daiter
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Matthew Daiter 2015-08-10 14:29:57 PDT
Needed way for the HTMLMediaElement to find the MediaDeviceInfo keys. Must have this known so that the player can grab the keys and use them for interpreting the unique ids on the system.
Comment 1 Matthew Daiter 2015-08-10 14:32:07 PDT
Created attachment 258651 [details]
Patch
Comment 2 Matthew Daiter 2015-08-10 14:41:58 PDT
Created attachment 258653 [details]
Patch
Comment 3 Jer Noble 2015-08-10 15:01:10 PDT
Comment on attachment 258653 [details]
Patch

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

> Source/WebCore/page/Settings.h:275
> +    void setMediaDeviceInformationKeysStorageDirectory(const String& directory) { m_mediaDeviceInformationKeysStorageDirectory = directory; }
> +    const String& mediaDeviceInformationKeysStorageDirectory() const { return m_mediaDeviceInformationKeysStorageDirectory; }

Are you storing "media device information keys"? Or are you storing "media device identifiers"?  I'd argue it's the latter, which would make this "mediaDeviceIdentifierStorageDirectory()"
Comment 4 Matthew Daiter 2015-08-10 15:16:18 PDT
Created attachment 258657 [details]
Patch
Comment 5 Matthew Daiter 2015-08-10 15:16:48 PDT
Comment on attachment 258653 [details]
Patch

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

>> Source/WebCore/page/Settings.h:275
>> +    const String& mediaDeviceInformationKeysStorageDirectory() const { return m_mediaDeviceInformationKeysStorageDirectory; }
> 
> Are you storing "media device information keys"? Or are you storing "media device identifiers"?  I'd argue it's the latter, which would make this "mediaDeviceIdentifierStorageDirectory()"

Fixed!
Comment 6 WebKit Commit Bot 2015-08-10 16:28:14 PDT
Comment on attachment 258657 [details]
Patch

Clearing flags on attachment: 258657

Committed r188234: <http://trac.webkit.org/changeset/188234>
Comment 7 WebKit Commit Bot 2015-08-10 16:28:17 PDT
All reviewed patches have been landed.  Closing bug.