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

Issue 748803 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows , Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Prompt for extension overrides the default NTP is not shown after adding few extensions

Project Member Reported by sdantul...@chromium.org, Jul 26 2017

Issue description

Google 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.
 

Comment 1 by jawag@chromium.org, Jul 26 2017

Cc: rdevlin....@chromium.org
Labels: OS-Windows
+Devlin

I'm able to reproduce this too, on both Chrome OS & Windows. I also don't see the prompt if I install/uninstall an extension, and then re-install the extension again.

Not sure if this is by design. @Devlin, do we have any limits on the number of times the bubble is shown?

Comment 2 by jawag@chromium.org, Aug 30 2017

Cc: -rdevlin....@chromium.org
Owner: rdevlin....@chromium.org
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?

Comment 4 by jawag@chromium.org, 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?
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.

Comment 6 by jawag@chromium.org, Sep 16 2017

Labels: -Pri-2 Pri-3
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.
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?

Comment 8 by jawag@chromium.org, 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?
Cc: rdevlin....@chromium.org
Owner: catmulli...@chromium.org
Nope, this should probably be fine.

catmullings@, you wanna take this one on?
Status: Started (was: Assigned)
Project Member

Comment 11 by bugdroid1@chromium.org, 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

Status: Fixed (was: Started)
Status: Verified (was: Fixed)
10684.0.0, 68.0.3432.0

Sign in to add a comment