Prompt for extension overrides the default NTP is not shown after adding few extensions |
|||||||
Issue descriptionGoogle Chrome 61.0.3163.11 (Official Build) dev (64-bit) Revision 0 Platform 9765.5.0 (Official Build) dev-channel caroline What steps will reproduce the problem? 1. Add an extension (A) that overrides the default new tab page 2. Open new tab page. See the prompt that says "Is this the new tab page you were expecting" 3. Add another new tab extension (B) 4. Open new tab and see prompt as in step 2 5. Add another extension (C) and open new tab What is the expected result? Should show prompt for extension overrides the default NTP What happens instead? Prompt is not shown.
,
Aug 30 2017
,
Sep 15 2017
Currently, we'll only show the bubble once per profile per session. This was intentional, but can be reconsidered. Do we want to do something different?
,
Sep 15 2017
What was the rationale for only showing the bubble once per profile session? Is it because it's uncommon that the profile would add two NTP override extensions within the same session?
,
Sep 15 2017
Once per session was because we didn't want to prompt the user about the same extension if they happened to open two tabs in the same session. We could also limit based on extension, but it was just suitably unlikely that any user would install two extensions, and not clearly better to show the prompt and risk being noisy.
,
Sep 16 2017
Makes sense. I could see limiting based on extension (e.g. it would only show once per profile per extension per session) but to your point, given the unlikeliness that the user would install two extensions, I think we can treat as a low priority.
,
Sep 18 2017
If we're certain that it's desired behavior to limit based on <profile, extension> pair, then it's pretty trivial to do that. Are we sure we want to show the user the bubble multiple times in that case?
,
Sep 22 2017
In my view, it's appropriate for the warning to show every time a new extension overrides the user's new tab page for the first time. Can you think of any scenario where this would be problematic?
,
Sep 28 2017
Nope, this should probably be fine. catmullings@, you wanna take this one on?
,
Oct 17 2017
,
Nov 22 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/478a043aaa94cce0bebbded00aabeef45fd11d85 commit 478a043aaa94cce0bebbded00aabeef45fd11d85 Author: Catherine Mullings <catmullings@chromium.org> Date: Wed Nov 22 07:06:06 2017 Extensions: Show NTP override prompt per ntp extension per profile Currently, the NTP override prompt only shows once per profile for any NTP overriding extension installed. The NTP override prompt should be displayed each time an NTP overriding extension is installed for a profile. This CL implements such a change. Bug: 748803 Change-Id: I896d9360a2a6e84f8586da481eb49be2428e9df0 Reviewed-on: https://chromium-review.googlesource.com/722454 Commit-Queue: catmullings <catmullings@chromium.org> Reviewed-by: Devlin <rdevlin.cronin@chromium.org> Reviewed-by: catmullings <catmullings@chromium.org> Cr-Commit-Position: refs/heads/master@{#518557} [modify] https://crrev.com/478a043aaa94cce0bebbded00aabeef45fd11d85/chrome/browser/extensions/dev_mode_bubble_delegate.cc [modify] https://crrev.com/478a043aaa94cce0bebbded00aabeef45fd11d85/chrome/browser/extensions/dev_mode_bubble_delegate.h [modify] https://crrev.com/478a043aaa94cce0bebbded00aabeef45fd11d85/chrome/browser/extensions/extension_message_bubble_controller.cc [modify] https://crrev.com/478a043aaa94cce0bebbded00aabeef45fd11d85/chrome/browser/extensions/extension_message_bubble_controller.h [modify] https://crrev.com/478a043aaa94cce0bebbded00aabeef45fd11d85/chrome/browser/extensions/extension_message_bubble_controller_unittest.cc [modify] https://crrev.com/478a043aaa94cce0bebbded00aabeef45fd11d85/chrome/browser/extensions/ntp_overridden_bubble_delegate.cc [modify] https://crrev.com/478a043aaa94cce0bebbded00aabeef45fd11d85/chrome/browser/extensions/ntp_overridden_bubble_delegate.h [modify] https://crrev.com/478a043aaa94cce0bebbded00aabeef45fd11d85/chrome/browser/extensions/proxy_overridden_bubble_delegate.cc [modify] https://crrev.com/478a043aaa94cce0bebbded00aabeef45fd11d85/chrome/browser/extensions/proxy_overridden_bubble_delegate.h [modify] https://crrev.com/478a043aaa94cce0bebbded00aabeef45fd11d85/chrome/browser/extensions/settings_api_bubble_delegate.cc [modify] https://crrev.com/478a043aaa94cce0bebbded00aabeef45fd11d85/chrome/browser/extensions/settings_api_bubble_delegate.h [modify] https://crrev.com/478a043aaa94cce0bebbded00aabeef45fd11d85/chrome/browser/extensions/suspicious_extension_bubble_delegate.cc [modify] https://crrev.com/478a043aaa94cce0bebbded00aabeef45fd11d85/chrome/browser/extensions/suspicious_extension_bubble_delegate.h
,
Nov 22 2017
,
May 17 2018
10684.0.0, 68.0.3432.0 |
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by jawag@chromium.org
, Jul 26 2017Labels: OS-Windows