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

Issue 774144 link

Starred by 1 user
Status: WontFix
Owner:
█ awdf@chromium.org
Last visit > 30 days ago
Closed: Oct 2017
Cc:
█ bever...@google.com
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug



Sign in to add a comment

Win10 Notifications type TEXT is not propagated correctly from webpage

Project Member Reported by finnur@chromium.org, Oct 12 2017 
Steps to repro:

1. On Windows 10, navigate to https://tests.peter.sh/notification-generator
2. Under 'Actions' select 'One action (type text with placeholder)'
3. Press 'Display the notification'

When the Notification object reaches NotificationPlatformBridgeWin, the button added (accessed through notification->buttons()[i]) is of type 'BUTTON' instead of 'TEXT'.

Comment 1 by awdf@chromium.org, Oct 19 2017

Status: Started (was: Assigned)
Are you testing with experimental web platform features enabled?

That's weird, because this seems to be working on Android, and this should go through the same path up until just before the NPB::Display call.

It could be that I have some Android-specific logic somewhere preventing this being propagated. I'll take a look and report back here tomorrow.

Comment 2 by finnur@chromium.org, Oct 20 2017 

I wasn't, but now I am and the result is the same. :/

Comment 3 by finnur@chromium.org, Oct 23 2017

Status: WontFix (was: Started)
Hmmm, seems to be working as intended now. Interesting.

Only thing that changed, I think, is that I sync'ed to latest a couple of days ago, but nothing relevant should have changed as a result of that. Weird.
►

Sign in to add a comment