Remove "Create a new profile" button for "fresh" profiles (including default profile) in managed account confirm dialogue |
||||||||||||||
Issue descriptionCurrently when the profile is signed in even on a completely fresh new profile (especially when this is governed by the ForceBrowserSignin policy the user is shown two confirmation dialogs as shown on pictures 6 and 7 here https://docs.google.com/viewer?a=v&pid=forums&srcid=MDgwNzk3OTY0Nzg0ODAyODUzNTcBMDM0NjEyNTk5NTQ5MzI4MTExMDgBcTNWc3lKY0FBQUFKATAuMQFnb29nbGVwcm9kdWN0Zm9ydW1zLmNvbQF2Mg . Instead they should at best be presented with one (if needed for legal reasons) or none if possible. Assigning to Owen who knows this code best now.
,
Jun 15 2017
--Chrome Identity automated triaging-- This bug is Untriaged and has gone for two weeks without any activity, so it is being moved to Available. Please see https://goo.gl/78kbny for more details. Please remove the Services>SignIn or UI>Browser>Profiles components if this bug isn't related to Chrome Identity. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jun 15 2017
We always have to display the sync confirmation dialogue, for both privacy and legal purposes (picture 7). I agree that if it's a "fresh," default profile, we shouldn't have the "Create a new profile" in the managed account confirmation dialogue. Updating the title of the bug to reflect that. I don't think there's anything else to do here.
,
Jun 15 2018
--Chrome Identity automated triaging-- This bug is Available and has gone one year without any activity. If another month passes without any activity, this bug will be closed out. Please provide an update with the latest status for this bug. Please see https://goo.gl/78kbny for more details. Please remove the Services>SignIn or UI>Browser>Profiles components if this bug isn't related to Chrome Identity. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jun 15 2018
This is still a valid bug. I'm not sure if anyone on the Enterprise team has time to pick this up; the sign-in team is too busy to triage it currently.
,
Jun 18 2018
,
Jun 18 2018
Moving to assigned to make sure that zmin@ sees it.
,
Jul 18
--Chrome Identity automated triaging-- This bug is Assigned and has gone one month without any activity, so it is being moved to Available to indicate that it is not actively being worked on. If you are working on this bug, please mark yourself as the owner and move back to Assigned. Please see https://goo.gl/78kbny for more details. Please remove the Services>SignIn or UI>Browser>Profiles components if this bug isn't related to Chrome Identity. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Aug 2
,
Sep 3
--Chrome Identity automated triaging-- This bug is Assigned and has gone one month without any activity, so it is being moved to Available to indicate that it is not actively being worked on. If you are working on this bug, please mark yourself as the owner and move back to Assigned. Please see https://goo.gl/78kbny for more details. Please remove the Services>SignIn or UI>Browser>Profiles components if this bug isn't related to Chrome Identity. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Oct 4
,
Oct 4
,
Oct 7
,
Nov 6
,
Dec 10
,
Dec 13
Maybe I'll take a stab at this one. If I understand comment 3, the request here is to change the logic in ui::CheckShouldPromptForNewProfile so that it provides false for prompt_for_new_profile when the profile is brand new. How does one get a browser into the right state so that this dialog will show?
,
Dec 13
Hi Greg, I was also looking a little into this one. Just wanted to give you some notes and let you continue since you probably know better: This dialog is a Sync specific dialog so it seems to only be accessible through the "Turn on Sync" button on the top right, by signing through chrome:://chrome-signin or opening a browser that has a google account signed in but with sync disabled (this one will bring up the Chrome welcome screen with "Web browsing with Google Smarts" where you can click on "Turn on Sync...") You can get the dialog with the "Create new profile" button if you do the following steps on any fresh user-data-dir: 1. run chrome.exe --user-data-dir=<something that does not exist> 2. goto to chrome://chrome-signin 3. Perform signin with your managed corp account. 4. You will get the mentioned dialog. However if you do the following steps instead you will get the dialog without the "create new profile": 1. run chrome.exe --user-data-dir=<something that does not exist> 2. Click on the grey user icon on the top right and "Turn on Sync" 3. Sign in with your corp account. In this situation you will get a dialog with only "Link Data" or "Cancel". There is a caveat to the above repro. If you leave your browser on the NTP long enough two quick links will eventually appear on that page (http:/go and http://goto, most likely downloaded through policy). Once these two links appear the flow mentioned above will also lead to a dialog with the "Create new profile" button. For the buggy situations ui::CheckShouldPromptForNewProfile will fail due to: helper->CheckHasTypedURLs(); In case #1: because you have obviously typed in "chrome:://chrome-signin" as your url. In case #2 where the http://go and http://goto link appear the same thing happens but in the background (likely a buggy association of those links with the PAGE_TRANSITION_TYPED transition type) So there may be 2 bugs to fix. 1. Why is the auto loading of http://go and http://goto associated with a typed transition. 2. Do we want to allow a first typed transition of chrome://chrome-signin to be accepted as a fresh profile? Maybe this case can still be marked as not fresh since the user did in fact type in a url. Hope this helps you.
,
Dec 13
Drive-by. I'm not understanding all the details here, but please note that chrome://chrome-signin is deprecated. It's probably not worth making any changes in that flow. To my knowledge this is only used in very small edge cases (such as the force signin policy) and we want to remove it completely.
,
Dec 13
chrome://welcome is the new FRE/welcome page experience that we use instead of chrome://chrome-signin.
,
Dec 14
Thanks for the info, droger. The issue is with the "Link your Chrome data to this account?" dialog. If that dialog will continue to be used in the force signin case, then we'd like to fix this. Will it, or is that dialog also going away? tienmai: thanks for all of the details. I'm happy to continue with this, though I'm not sure I know any better than you. :-) Feel free to run with it if you wish.
,
Dec 14
Yes the dialog is also shown as part of the new flow, and the code for the dialog is shared. However the code that triggers the dialog is not shared. Basically the deprecated code is here: https://cs.chromium.org/chromium/src/chrome/browser/ui/sync/one_click_signin_sync_starter.cc?rcl=b94e1fd68e12827ed018ab5b41fbe32251df5a0a&l=237 The new code is here: https://cs.chromium.org/chromium/src/chrome/browser/ui/webui/signin/dice_turn_sync_on_helper.cc?rcl=b94e1fd68e12827ed018ab5b41fbe32251df5a0a&l=244 I don't think you need to fix the deprecated code, we're planning to remove it.
,
Dec 14
I've tried to repo the bug as-described using the BrowserSignin=force policy setting, and I don't see account linking dialog at all (launch with fresh user data dir, sign in, see a browser, accept the sync dialog). Should I? |
||||||||||||||
►
Sign in to add a comment |
||||||||||||||
Comment 1 by zmin@chromium.org
, May 31 2017