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

Issue 595487 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Sep 2016
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 571056



Sign in to add a comment

Mac Native notifications don't propagate click events for in page notifications

Project Member Reported by miguelg@chromium.org, Mar 16 2016

Issue description

Service Worker notifications propagate fine.
 
Blocking: -571056
Blocking: 571056
Owner: miguelg@chromium.org
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 8 2016

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

commit 87e986d94f7bf82930af385d88d7e3b79eac4515
Author: miguelg <miguelg@chromium.org>
Date: Fri Jul 08 18:04:44 2016

Introduce a new API to handle native notification clicks

BUG= 596161 , 595487 

Review-Url: https://codereview.chromium.org/2093953002
Cr-Commit-Position: refs/heads/master@{#404436}

[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/android/javatests/src/org/chromium/chrome/browser/notifications/NotificationPlatformBridgeIntentTest.java
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/notifications/message_center_display_service.cc
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/notifications/message_center_display_service.h
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/notifications/native_notification_display_service.cc
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/notifications/native_notification_display_service.h
[add] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/notifications/non_persistent_notification_handler.cc
[add] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/notifications/non_persistent_notification_handler.h
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/notifications/notification_common.h
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/notifications/notification_display_service.h
[add] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/notifications/notification_handler.h
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/notifications/notification_platform_bridge.h
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/notifications/notification_platform_bridge_android.cc
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/notifications/notification_platform_bridge_android.h
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/notifications/notification_platform_bridge_mac.h
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/notifications/notification_platform_bridge_mac.mm
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/notifications/notification_platform_bridge_mac_unittest.mm
[add] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/notifications/persistent_notification_handler.cc
[add] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/notifications/persistent_notification_handler.h
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/notifications/platform_notification_service_impl.cc
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/notifications/platform_notification_service_impl.h
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/notifications/stub_notification_platform_bridge.cc
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/notifications/stub_notification_platform_bridge.h
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/ui/cocoa/notifications/notification_builder_mac.h
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/ui/cocoa/notifications/notification_builder_mac.mm
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/ui/cocoa/notifications/notification_builder_mac_unittest.mm
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/ui/cocoa/notifications/notification_constants_mac.h
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/ui/cocoa/notifications/notification_constants_mac.mm
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/ui/cocoa/notifications/notification_response_builder_mac.mm
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/browser/ui/cocoa/notifications/notification_response_builder_mac_unittest.mm
[modify] https://crrev.com/87e986d94f7bf82930af385d88d7e3b79eac4515/chrome/chrome_browser.gypi

Status: Fixed (was: Available)

Sign in to add a comment