Public Session whitelisting - clean up |
||
Issue description- find a directory where to put files (and maybe split CrOS only / all) - refactor usage of IsPublicSession() function (make it accessible on all platforms) - audio/video/desktop/tab/page capture API changes share a lot of logic, factor out common code - TabCapture - don't filter captureOffscreenTab calls (more info at crrev.com/2558843002), move some files in to a different directory (comments by miu)
,
Dec 14 2016
,
Jan 9 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0af058d7d4b9312ce34b674316ea68a992c20d34 commit 0af058d7d4b9312ce34b674316ea68a992c20d34 Author: isandrk <isandrk@chromium.org> Date: Mon Jan 09 16:49:56 2017 Added IsPublicSession utility function to profiles_state Refactoring effort to put IsPublicSession utility function in one place. It's currently defined & used in the following places (with more to come): https://cs.chromium.org/chromium/src/chrome/browser/media/public_session_media_access_handler.cc?q=IsPublicSession&l=22 https://cs.chromium.org/chromium/src/chrome/browser/media/webrtc/public_session_tab_capture_access_handler.cc?q=IsPublicSession&l=22 https://cs.chromium.org/chromium/src/extensions/browser/api/web_request/web_request_api.cc?q=IsPublicSession&l=344 BUG=672620 Review-Url: https://codereview.chromium.org/2616423002 Cr-Commit-Position: refs/heads/master@{#442268} [modify] https://crrev.com/0af058d7d4b9312ce34b674316ea68a992c20d34/chrome/browser/profiles/profiles_state.cc [modify] https://crrev.com/0af058d7d4b9312ce34b674316ea68a992c20d34/chrome/browser/profiles/profiles_state.h |
||
►
Sign in to add a comment |
||
Comment 1 Deleted