Summary: | [EME] Add Setting for accessing storage location for MediaKeys data | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jer Noble <jer.noble> | ||||||||
Component: | New Bugs | Assignee: | Jer Noble <jer.noble> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | buildbot, calvaris, commit-queue, eric.carlson, esprehn+autocc, glenn, gyuyoung.kim, jonlee, philipj, rniwa, sergio, webkit-bug-importer | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Bug Depends on: | 137120 | ||||||||||
Bug Blocks: | |||||||||||
Attachments: |
|
Description
Jer Noble
2014-10-28 13:23:47 PDT
Created attachment 240568 [details]
Patch
Created attachment 240659 [details]
Patch
Comment on attachment 240659 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=240659&action=review > Source/WebCore/page/ChromeClient.h:450 > + virtual String mediaKeysStorageDirectoryForOrigin(SecurityOrigin*) { return emptyString(); } I don't think the ChromeClient is the place for this. I think there's way more appropriate places to put directory accessors. Comment on attachment 240659 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=240659&action=review >> Source/WebCore/page/ChromeClient.h:450 >> + virtual String mediaKeysStorageDirectoryForOrigin(SecurityOrigin*) { return emptyString(); } > > I don't think the ChromeClient is the place for this. I think there's way more appropriate places to put directory accessors. Okay, I'll move these methods into Settings. Created attachment 240710 [details]
Patch
Comment on attachment 240710 [details] Patch Clearing flags on attachment: 240710 Committed r175400: <http://trac.webkit.org/changeset/175400> All reviewed patches have been landed. Closing bug. |