Issue metadata
Sign in to add a comment
|
Display Flash default as "Detect" instead of "Ask" again |
||||||||||||||||||||||||
Issue descriptionSome layers seem to be lying: https://cs.chromium.org/chromium/src/chrome/browser/plugins/flash_permission_context.cc?q=CONTENT_SETTING_DETECT_IMPORTANT_CONTENT&l=49&dr=C This results in Page Info showing Flash setting of CONTENT_SETTING_DEFAULT as if it were CONTENT_SETTING_ASK, even though ContentSettings::GetDefaultContentSetting() returns CONTENT_SETTING_DETECT_IMPORTANT_CONTENT as the default for Flash. raymes@, could you triage?
,
Feb 28 2017
I think this is intended by tommycli@ can confirm.
,
Feb 28 2017
,
Feb 28 2017
I remember a decision for Material Page Info that we wanted to show "Detect (Default)". If we show "Ask (Default)" instead of "Detect (Default)", then our most promising compromise in Issue (hide permissions set to the default value if that default value of "Ask") will have confusingly misrepresentative UI for the 2nd-most-commonly changed permission.
,
Mar 1 2017
> will have confusingly misrepresentative UI for the 2nd-most-commonly changed permission. Specifically, if we say "Page Info now hides all permissions that have a default value of "Ask", since a site cannot use such a permission until it shows you a prompt for it" then people will wonder why we're not hiding the Flash permission by default. I don't think a good solution to that is "oh, and Flash is an exception". Because it's actually *not*, and I think we should surface that again. (Which reminds me, that makes this a regression, assuming we change it back.)
,
Mar 1 2017
Whatever we change the string to, I do think it's important to have flash included by default in Page Info. The reason being that a prompt won't reliably be shown. If a site wants to use geolocation, a prompt will always be shown, but with HTML5 by default, that won't be the case. I think we might have changed this to "Ask" when we enabled HTML5 by default, so I don't think this is a regression. I understand your reasoning that the language might be inconsistent if the only thing that is Ask that we show is flash. But it's also hard to come up with a better string here. I'm not sure that Detect is accurate anymore.
,
Mar 1 2017
,
Mar 1 2017
Hey, I just saw the patch go by that enables HBD by default for 100% of Stable users... So it should definitely be "Ask".
,
Mar 1 2017
Thanks Tommy!
,
Mar 1 2017
I love opening bugs that are already resolved :)
,
Mar 1 2017
I think there's still a question as to what to how we display flash in PageInfo. It does say "Ask" by default but it behaves differently to other ASK permissions in the sense that a permission bubble won't reliably be shown.
,
Mar 1 2017
I agree with #6 that we should show Flash by default in PageInfo and that it would be inconsistent and confusing if it remained "ask". Why is "detect" inaccurate or bad? (Maybe I'm missing something)
,
Mar 1 2017
The most accurate string, in probably both places, is "Prefer HTML". There's not much detection going on in the post-HBD world -- and we only ask if the user clicks on a Download Flash link or a placeholder.
,
Mar 1 2017
One thing we could do is rename it back to "Detect" for as long as the site engagement threshhold is around. That would be pretty accurate for that case. Then when the site engagement threshhold is removed, we change it back to "Ask" but also remove it from Page Info. I think by that stage flash usage will be down and it will be less important to include it there. tommycli/lgarron: what do you think about that plan?
,
Mar 1 2017
^ I think Raymes' reasoning is solid.
,
Mar 1 2017
SGTM2. +Dom FYI.
,
Mar 1 2017
Detect still makes the most sense to me too. SGTM3. :)
,
Mar 3 2017
Alright, I'm going to send a CL to change back to "Detect".
,
Mar 3 2017
,
Mar 4 2017
I doubt a screenshot is necessary, but here we go. https://codereview.chromium.org/2731083002
,
Mar 4 2017
,
Mar 15 2017
I'm seeing inconsistent behavior with this as well. On Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/57.0.2987.98 Safari/537.36. If I visit http://webhtb.sourceforge.net/video_demo.html I'm not able to context click and "Run this plugin" (option is greyed out). However, upon click, I get a new kind of permission dialog (like the "Allow access to mic" dialog), and it then runs the SWF. However, nothing shows up in chrome://settings/contentExceptions#plugins ! Now the plugin always runs. Changin the url info menu settings as in #20 does allow me to block it again, but never get back the "run only this time" (e.g. ask) option.
,
Mar 24 2017
This is no longer as important now that hiding default permissions is on hold. How soon will "Ask" be the correct value?
,
Mar 27 2017
Probably Oct 2017: https://sites.google.com/a/chromium.org/dev/flash-roadmap#TOC-HTML5-By-Default-Target:-Chrome-55---Dec-2016- I'd prefer to keep the status quo so long as we're showing Detect in the UI surfaces. If we're changing all the UI surfaces to ASK, we should migrate all the actual content settings in one fell swoop.
,
Nov 10 2017
,
Dec 1 2017
raymes do you know what the current status of this is?
,
Dec 5 2017
The site engagement threshhold is now 100% and we're using "Ask" language. I think things are consistent. Marking WontFix. |
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by lgar...@chromium.org
, Feb 28 2017