New issue
Advanced search Search tips

Issue 756710 link

Starred by 1 user
Status: Verified
Owner:
tetsui@chromium.org
Closed: Aug 2017
Cc:
mkarkada@chromium.org
peter@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug

Blocked on:
issue 754508


Sign in to add a comment

Notification is overwritten when new Chrome extension app is installed.

Project Member Reported by tetsui@chromium.org, Aug 18 2017 
Chrome OS version: 62.0.3176.0, dev build
Platform version: 9827.0.0, dev channel caroline device
M-62 build

Steps to reproduce the problem:
1. Goto Chrome extensions (Chrome web store), try to install any extension app like Postman App> Add to chrome> Add App.
2. Notification shows up.
3. Now try to install another extension and observe the notification.
4. This new notification overwrites the previous one.

What is the expected behavior?
It shouldn't overwrite the previous one.

originally reported by mkarkada@
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 22 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/8e7c840733d0dbb953c0a66a64c6541f0038fc45

commit 8e7c840733d0dbb953c0a66a64c6541f0038fc45
Author: Tetsui Ohkubo <tetsui@chromium.org>
Date: Tue Aug 22 03:25:30 2017

Do not overwrite extension installed notification.

Previously, when installing multiple different extensions, new install
successful notification overwrited the last one.
This CL gives a unique tag of extension ID to each notification and
keep these notifications separate.

TEST=manual
BUG= 756710 

Change-Id: Ic5b949acf894b50157a877750491511b46f1912f
Reviewed-on: https://chromium-review.googlesource.com/623015
Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org>
Reviewed-by: Finnur Thorarinsson <finnur@chromium.org>
Cr-Commit-Position: refs/heads/master@{#496196}
[modify] https://crrev.com/8e7c840733d0dbb953c0a66a64c6541f0038fc45/chrome/browser/ui/extensions/extension_installed_notification.cc

Comment 2 by tetsui@chromium.org, Aug 22 2017

Status: Fixed (was: Assigned)

Comment 3 by mkarkada@chromium.org, Nov 8 2017

Status: Verified (was: Fixed)
Verified on M64 dev build (10106.0.0, 64.0.3261.0).
►

Sign in to add a comment