New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 692649 link

Starred by 1 user
Status: WontFix
Owner: ----
Closed: Mar 2017
Cc:
█ lpz@chromium.org
est...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug
Team-Security-UX



Sign in to add a comment

Certificate reporting opt-in text changes based on Finch parameter

Project Member Reported by mea...@chromium.org, Feb 15 2017 
The checkbox label in SSL errors changes based on whether ReportCertificateErrors field trial is force enabled from the command line. Attached are two screenshots finch1 and finch2.

finch1: No command line flags, automatically assigned to Finch group ReportCertificateErrors/ShowAndPossiblySend

finch2: Has --force-fieldtrials=ReportCertificateErrors/ShowAndPossiblySend

The opt-in text is selected by ChooseOptInTextResource in safe_browsing_prefs.cc based on Scout preferences, though I'm not sure why force-fieldtrials affects the result.
finch1.png
38.2 KB View Download
finch2.png
34.9 KB View Download

Comment 1 by lpz@chromium.org, Feb 16 2017 

Just an initial guess, but I wonder if this might be a side-effect of how the--force-fieldtrials flag works? It seems like it might first clear all present features/fieldtrials, and then enable the ones from the command-line. This would implicitly remove the Scout feature and thus revert the opt-in text.

To get both ReportCertificateErrors/ShowAndPossiblySend and Scout, you might need to also set --enable-features=CanShowScoutOptIn.

Comment 2 by jialiul@chromium.org, Mar 2 2017

Status: WontFix (was: Untriaged)
Agree with #1, it looks mostly like a side-effect. I have tried:
(1) --force-fieldtrials=ReportCertificateErrors/ShowAndPossiblySend --enable-features=CanShowScoutOptIn
and 
(2) --force-fieldtrials=ReportCertificateErrors/ShowAndPossiblySend --disable-features=CanShowScoutOptIn

opt-in text shows as expected. 
I'll mark it as Won'tFix. Feel free to reopen.

Comment 3 by mea...@chromium.org, Mar 2 2017 

Sorry for the delay. Yes, that explanation makes sense, thanks!
►

Sign in to add a comment