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

Issue 825095 link

Starred by 0 users

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2018
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Assert DesktopPWAWindowing is enabled in GetInstalledPwaForUrl

Project Member Reported by alancutter@chromium.org, Mar 23 2018

Issue description

There's no check for whether DesktopPWAWindowing is enabled in GetInstalledPwaForUrl.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Mar 27 2018

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

commit d28bdd019bb65c267d3c5b814324227b4cf48dd4
Author: Alan Cutter <alancutter@chromium.org>
Date: Tue Mar 27 04:03:07 2018

Assert DesktopPWAWindowing is enabled in GetInstalledPwaForUrl

Bug:  825095 
Change-Id: I45d7af704636189a4a94a8c125c66092c5192e96
Reviewed-on: https://chromium-review.googlesource.com/977347
Reviewed-by: Avi Drissman <avi@chromium.org>
Reviewed-by: Ben Wells <benwells@chromium.org>
Reviewed-by: Giovanni Ortuño Urquidi <ortuno@chromium.org>
Commit-Queue: Alan Cutter <alancutter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#546005}
[modify] https://crrev.com/d28bdd019bb65c267d3c5b814324227b4cf48dd4/chrome/browser/extensions/extension_util.cc
[modify] https://crrev.com/d28bdd019bb65c267d3c5b814324227b4cf48dd4/chrome/browser/renderer_context_menu/render_view_context_menu_browsertest.cc

Status: Fixed (was: Assigned)

Sign in to add a comment