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

Issue 590743 link

Starred by 4 users
Status: WontFix
Owner:
steve...@chromium.org
Closed: Mar 2018
Cc:
█ dim...@chromium.org
█ dskaram@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 3
Type: Bug



Sign in to add a comment

appWindow.setIcon API works only in dev channel

Reported by santosh....@gmail.com, Feb 29 2016 
UserAgent: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/48.0.2564.116 Safari/537.36

Steps to reproduce the problem:
1. Create new window from chrome packaged app using chrome.window.create
2. Inside window, call chrome.app.window.current().setIcon(url_to_icon)
3. Check the icon on chrome shelf or OS taskbar

What is the expected behavior?
Icon of the packaged app should change to the new icon set.

What went wrong?
Icon doesn't change and the following error is shown in the console:

app.currentWindowInternal.setIcon: This function is currently only available in the Dev channel.

Did this work before? No 

Chrome version: 48.0.2564.116  Channel: stable
OS Version: 10.0
Flash Version: Shockwave Flash 20.0 r0

This works in the dev channel as the error suggests.

Comment 1 by dskaram@chromium.org, Feb 29 2016

Cc: steve...@chromium.org
That's a bug, we should fix that. We decided to enable panels on Chrome OS on all channels but clearly missed this. The code that needs to be fixed is here:

https://code.google.com/p/chromium/codesearch#chromium/src/extensions/browser/api/app_current_window_internal/app_current_window_internal_api.cc&q=app_current_window_internal_api&sq=package:chromium&l=300

That if() should be wrapped in an #if !defined(OS_CHROMEOS).

Comment 2 by dtapu...@chromium.org, Mar 30 2016

Components: Platform>Extensions>API

Comment 3 by danielhe...@gmail.com, Oct 27 2016 

Now that panels have been removed for extensions, this api or support for <link rel=icon> should be shipped.

Comment 4 by steve...@chromium.org, Oct 27 2016

Cc: -steve...@chromium.org dim...@chromium.org dskaram@chromium.org
Owner: steve...@chromium.org
Status: Assigned (was: Unconfirmed)
I can put this on my plate, but it won't be super high priority, so if someone else wants to grab it feel free.

Comment 5 by steve...@chromium.org, Mar 8 2018

Labels: Pri-3

Comment 6 by dskaram@google.com, Mar 8 2018

Status: WontFix (was: Assigned)
Moving to Android APIs for these use cases.
►

Sign in to add a comment