|
||||||||
Issue descriptionUserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/55.0.2883.87 Safari/537.36 Steps to reproduce the problem: 1. Trigger a desktop notification (e.g. https://davidwalsh.name/demo/notifications-api.php) What is the expected behavior? The notification should use the desktop environment's notification system. What went wrong? The notification is displayed by Chromium's built in notification system. Did this work before? No Chrome version: 55.0.2883.87 Channel: stable OS Version: Flash Version: Shockwave Flash 24.0 r0 The freedesktop.org specification (https://developer.gnome.org/notification-spec/) is supported on many popular distributions on Linux. Notification servers implementing the specification are available for distributions and desktop environments which do not include support by default. There are libraries (such as libnotify) available such for various languages which implement the specification. A good overview of desktop notifications on Linux is https://wiki.archlinux.org/index.php/Desktop_notifications There is a libnotify based extension for Firefox: https://addons.mozilla.org/en-US/firefox/addon/gnotifier/ At one time there were extensions available for Chromium but they were dependant on NPAPI
Showing comments 9 - 108
of 108
Older ›
Mar 26 2017,> On some desktops, like Unity, notifications aren't even clickable. For some people, that's a plus. And if it's something someone couldn't live with, he wouldn't chose Unity in the first place. Mar 28 2017,> if it's something someone couldn't live with, he wouldn't chose Unity in the first place. One might like Unity and used to use Chrome's/Chromium's clickable notifications. Interactiveness adds to the UX; it doesn't make it worse. Furthermore, on a browser, interactive notifications are extremely important since they're sent from the Internet which connects you to the outside world and clicking on it takes you to the appropriate place, so you can engage in real time, while, for example, when rendering is finished on your machine, there's not much of a point in having an interactive notification. Mar 28 2017,> One might like Unity and used to use Chrome's/Chromium's clickable notifications. Interactiveness adds to the UX; it doesn't make it worse. Furthermore, on a browser, interactive notifications are extremely important since they're sent from the Internet which connects you to the outside world and clicking on it takes you to the appropriate place, so you can engage in real time, while, for example, when rendering is finished on your machine, there's not much of a point in having an interactive notification. In that case I would argue, that there's a point in clickable notifications and that they would benefit not only Chromium. Therefore they should be implemented upstream and not only in Chromium. Mar 29 2017,When it comes to clickable notifications there is an easy solution, check if the current DE supports them, if it does show native notifications, otherwise show chrome notifications. With libnotify checking that is simple, just call: https://developer.gnome.org/libnotify/unstable/libnotify-notify.html#notify-get-server-caps , and check if it supports "actions". Mar 30 2017,> In that case I would argue, that there's a point in clickable notifications and that they would benefit not only Chromium. Therefore they should be implemented upstream and not only in Chromium. It won't be implemented in Unity 7, all the development is now focused on Unity 8 which is still unstable and, I believe, it's already implemented there. On KDE Plasma 5, the notification system is still in development, but I think, it's already mature enough to replace Chrome's notifications. > When it comes to clickable notifications there is an easy solution, check if the current DE supports them, if it does show native notifications, otherwise show chrome notifications. That seems to be a good option. Mar 30 2017,It should be possible for users who want libnotify regardless of clickability to be able to force it with some flag in chrome://flags/. Mar 31 2017,Everyone keeps conflating libnotify with the notification spec. libnotify is NOT the spec, it's merely a convenience lib(a crusty old blocking I/O lib at that) It would be best if everyone forgot it even existed. libnotify is not what you want to use. The notification spec is stupid simple. Just talk directly to DBus(hopefully in a purely asynchronous manner). Apr 1 2017,> On some desktops, like Unity, notifications aren't even clickable. I haven't read the Chromium notifications HIG(if it exists?) But notifications should be designed as not to assume that they were even seen let alone clickable. Notifications are a secondary interaction method not a primary method. Apps should be written so any interaction that notifications provided can also be done from the main UI. Providing functionality that can only be accessed though notifications is a big no, no. Notifications should also be worded as to not assume intractability. For example "You have a new message" is preferred over "Click to check your new messages." Apr 1 2017,> Notifications are a secondary interaction method not a primary method. Apps should be written so any interaction that notifications provided can also be done from the main UI. Providing functionality that can only be accessed though notifications is a big no, no. Removing functionality from notifications *is* the big no-no. If I receive a new email, a well-designed software should let me open it with a click of a button. > It should be possible for users who want libnotify regardless of clickability to be able to force it with some flag in chrome://flags/. Yes, providing a flag is the most important thing right now; in that case, everybody will be happy. But it should go both ways — users must be able to override both kinds of notifications. The question is, however, what should be the default behaviour. This one sounds nice: > When it comes to clickable notifications there is an easy solution, check if the current DE supports them, if it does show native notifications, otherwise show chrome notifications. So, although using native notifications should be a priority, having a fallback mode is a must. Apr 1 2017,> Removing functionality from notifications *is* the big no-no. If I receive a new email, a well-designed software should let me open it with a click of a button. "well-designed software" would not rely on a popup notification for core functionality. A notification is just that a notification. Basically "Hey look at me something at least semi-important has happened." As I've said it should never be assumed to have even been seen let alone depended upon to drive interaction forward. Relying on transient/close-able notifications as a primary interaction method is a flawed design that can lead to dead ends in user interaction. Anyway, to my knowledge Unity7 is the only DE that doesn't support buttons in notifications. It's up for debate if Unity7 is in fact "well-designed software"... > When it comes to clickable notifications there is an easy solution, check if the current DE supports them, if it does show native notifications, otherwise show chrome notifications. Example outputs of org.freedesktop.Notifications.GetCapabilities and org.freedesktop.Notifications.GetServerInformation on Fedora 24: Server information: spec_version 1.2 vendor GNOME name gnome-shell version 3.20.3 Server Capabilities: actions body body-markup icon-static persistence sound A couple things of note: All images/icons are expected to be local files. So icon/images would have to be cached locally 1st. Remote image resources are a No, no. body-markup is a bit of a lie. By markup they mean a very limited set. actions = supports buttons. persistence = All notifications are persistent, meaning they don't go away unless dismissed by the user. That can be overridden by passing the transient hint is desired. I'd advise anyone interested to read the actual spec and not the libnotify docs. https://developer.gnome.org/notification-spec/ Apr 2 2017,> Anyway, to my knowledge Unity7 is the only DE that doesn't support buttons in notifications. It's up for debate if Unity7 is in fact "well-designed software"... Not Unity 7; my point was regarding removing the features from Chrome/Chromium which makes interactive notifications possible on desktops that doesn't support it. Apr 4 2017,> Not Unity 7; my point was regarding removing the features from Chrome/Chromium which makes interactive notifications possible on desktops that doesn't support it. Unity is the only DE that has it's own Notification server that doesn't support buttons to my knowledge. All the other more popular DE's (Gnome, Plasma(KDE), XFCE, Mate, Cinnamon, etc.) support buttons. But technically speaking you can install the notification system from one DE in another. And there is a possibility that a system may not even have a Notification server installed at all. The 1st step would be to make sure the notification interface exists and then to check to see if the server supports actions. If so use native notifications if not use Chrome/Chromium notifications. Apr 5 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/421f86bb22686a086062ffde46fb746039dd2490 commit 421f86bb22686a086062ffde46fb746039dd2490 Author: thomasanderson <thomasanderson@google.com> Date: Wed Apr 05 06:32:42 2017 Add initial support for native Linux desktop notifications This CL adds a stub implementation of NotificationPlatformBridgeLinux, which is responsible for communicating notification changes to the desktop environment via D-Bus. Once this class is fully implemented, it is intended to be used by default when the host supports notifications. BUG= 676220 R=thestig@chromium.org,yoshiki@chromium.org Review-Url: https://codereview.chromium.org/2794103002 Cr-Commit-Position: refs/heads/master@{#461990} [modify] https://crrev.com/421f86bb22686a086062ffde46fb746039dd2490/chrome/browser/BUILD.gn [modify] https://crrev.com/421f86bb22686a086062ffde46fb746039dd2490/chrome/browser/about_flags.cc [modify] https://crrev.com/421f86bb22686a086062ffde46fb746039dd2490/chrome/browser/browser_process_impl.cc [modify] https://crrev.com/421f86bb22686a086062ffde46fb746039dd2490/chrome/browser/notifications/notification_display_service_factory.cc [modify] https://crrev.com/421f86bb22686a086062ffde46fb746039dd2490/chrome/browser/notifications/notification_platform_bridge.h [add] https://crrev.com/421f86bb22686a086062ffde46fb746039dd2490/chrome/browser/notifications/notification_platform_bridge_linux.cc [add] https://crrev.com/421f86bb22686a086062ffde46fb746039dd2490/chrome/browser/notifications/notification_platform_bridge_linux.h [modify] https://crrev.com/421f86bb22686a086062ffde46fb746039dd2490/chrome/common/BUILD.gn [modify] https://crrev.com/421f86bb22686a086062ffde46fb746039dd2490/chrome/common/chrome_features.cc [modify] https://crrev.com/421f86bb22686a086062ffde46fb746039dd2490/chrome/common/chrome_features.h [modify] https://crrev.com/421f86bb22686a086062ffde46fb746039dd2490/chrome/common/features.gni Apr 5 2017,
Apr 7 2017,X-reffing video for https://codereview.chromium.org/2805543005/ Apr 7 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/02fc962eaa3cd415db363f2289f8e8963a51130c commit 02fc962eaa3cd415db363f2289f8e8963a51130c Author: thomasanderson <thomasanderson@google.com> Date: Fri Apr 07 04:52:20 2017 Linux native notifications: Support closing and updating notifications BUG= 676220 R=thestig@chromium.org,peter@chromium.org Review-Url: https://codereview.chromium.org/2803873003 Cr-Commit-Position: refs/heads/master@{#462751} [modify] https://crrev.com/02fc962eaa3cd415db363f2289f8e8963a51130c/chrome/browser/notifications/notification_platform_bridge_linux.cc [modify] https://crrev.com/02fc962eaa3cd415db363f2289f8e8963a51130c/chrome/browser/notifications/notification_platform_bridge_linux.h [modify] https://crrev.com/02fc962eaa3cd415db363f2289f8e8963a51130c/chrome/browser/ui/libgtkui/gtk_util.h [modify] https://crrev.com/02fc962eaa3cd415db363f2289f8e8963a51130c/ui/base/BUILD.gn [add] https://crrev.com/02fc962eaa3cd415db363f2289f8e8963a51130c/ui/base/glib/scoped_gobject.h Apr 7 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/09fb058d6bfb0e74343a851f56e4032c44b56c19 commit 09fb058d6bfb0e74343a851f56e4032c44b56c19 Author: tonikitoo <tonikitoo@igalia.com> Date: Fri Apr 07 17:00:22 2017 Guard native notification code under ENABLE_NATIVE_NOTIFICATIONS guards about_flags.cc today opt in/out configurations manually, rather than using the existing ENABLE_NATIVE_NOTIFICATIONS build flag. This fixes the OZONE/Linux build. BUG= 676220 R=thomasanderson@google.com Review-Url: https://codereview.chromium.org/2802243002 Cr-Commit-Position: refs/heads/master@{#462896} [modify] https://crrev.com/09fb058d6bfb0e74343a851f56e4032c44b56c19/chrome/browser/about_flags.cc Apr 7 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/1e5d734b182b3bd7de2c36a88a26f05f857f59ec commit 1e5d734b182b3bd7de2c36a88a26f05f857f59ec Author: thomasanderson <thomasanderson@google.com> Date: Fri Apr 07 23:24:08 2017 Linux native notifications: Handle clicks and closes This CL adds the initial wiring to GLib and NativeNotificationDisplayService to get actions and closes working. BUG= 676220 R=thestig@chromium.org,peter@chromium.org Review-Url: https://codereview.chromium.org/2805543005 Cr-Commit-Position: refs/heads/master@{#463045} [modify] https://crrev.com/1e5d734b182b3bd7de2c36a88a26f05f857f59ec/chrome/browser/notifications/notification_platform_bridge_linux.cc [modify] https://crrev.com/1e5d734b182b3bd7de2c36a88a26f05f857f59ec/chrome/browser/notifications/notification_platform_bridge_linux.h Apr 10 2017,I may be wrong but it looks like you're not sending an image or icon with the notifications? Generally speaking you want to send at least the app's icon.(chrome/chromium) you would send them as a string by name without the file extension. Apr 10 2017,c#29 That's because the impl is not done yet :) When it is, I'll close this bug out as Fixed Apr 12 2017,For anyone wanting to try this out, you can in Chrome Dev (Unstable) by toggling 'Enable native notifications' in about://flags Apr 12 2017,Works great for me. Elementary OS 64bit (Ubuntu 16.04 based) with Pantheon Desktop. However it does not pass the picture (neither the picture included in the notification nor an application icon of some sort). I tested it here: http://www.bennish.net/web-notifications.html Apr 12 2017,c#32 Images not yet implemented. Work being done on this cl https://codereview.chromium.org/2806203003/ Apr 14 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d82f2908e3193915cdc573d54f962920714b847d commit d82f2908e3193915cdc573d54f962920714b847d Author: thomasanderson <thomasanderson@google.com> Date: Fri Apr 14 01:41:46 2017 Linux native notifications: Support icons This CL: * Adds support for notification icons * Adds an urgency hint * Supports updating NotificationCommon::Type BUG= 676220 R=peter@chromium.org,thestig@chromium.org Review-Url: https://codereview.chromium.org/2806203003 Cr-Commit-Position: refs/heads/master@{#464650} [modify] https://crrev.com/d82f2908e3193915cdc573d54f962920714b847d/chrome/browser/notifications/notification_platform_bridge_linux.cc [modify] https://crrev.com/d82f2908e3193915cdc573d54f962920714b847d/chrome/browser/notifications/notification_platform_bridge_linux.h Apr 15 2017,If possible, it should send along the "desktop-entry" hint so notification centers can group and/or filter based on the app that sent the notification (e.g. google-chrome.desktop) Apr 17 2017,>If possible, it should send along the "desktop-entry" hint so notification centers can group and/or filter based on the app that sent the notification (e.g. google-chrome.desktop) The desktop-entry hint should NOT include .desktop. "This specifies the name of the desktop filename representing the calling program. This should be the same as the prefix used for the application's .desktop file. An example would be "rhythmbox" from "rhythmbox.desktop". This can be used by the daemon to retrieve the correct icon for the application, for logging purposes, etc." Other useful hints are category, action-icons, and transient. category allows you to tell the notification server what type of notification you're sending it. There is a (rather limited) list of standard hints along with various non-standard vendor specific categories. For example x-gnome.music tells the notification server that the notification is a music notification in many notification servers. Many notification servers support icons in the action buttons. You can check by looking for "action-icons" in the server capabilities. If you would like to show icons in the action buttons the icon name would be used as the action id. In a few servers (GNOME for example) notifications are always persistent. Meaning they don't go away unless they are dismissed. You would check for that by looking for persistence in the server capabilities. You can override that with the transient hint if you like. Apr 18 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/b6b0a4c62db4757a27a61f19921ed70df20a42be commit b6b0a4c62db4757a27a61f19921ed70df20a42be Author: thomasanderson <thomasanderson@google.com> Date: Tue Apr 18 01:49:37 2017 Linux native notifications: Add support for custom notification buttons BUG= 676220 R=peter@chromium.org Review-Url: https://codereview.chromium.org/2814973002 Cr-Commit-Position: refs/heads/master@{#465091} [modify] https://crrev.com/b6b0a4c62db4757a27a61f19921ed70df20a42be/chrome/browser/notifications/notification_platform_bridge_linux.cc Apr 18 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/196a305434d54e05c519f1f26e129d7cf2f863ba commit 196a305434d54e05c519f1f26e129d7cf2f863ba Author: thomasanderson <thomasanderson@google.com> Date: Tue Apr 18 04:33:55 2017 Linux native notifications: Add desktop-entry hint This CL adds a desktop-entry hint so that the notification server can use the app's name and icon if it chooses to. BUG= 676220 R=thestig@chromium.org Review-Url: https://codereview.chromium.org/2811183009 Cr-Commit-Position: refs/heads/master@{#465138} [modify] https://crrev.com/196a305434d54e05c519f1f26e129d7cf2f863ba/chrome/browser/notifications/notification_platform_bridge_linux.cc Apr 21 2017,From the PyGobject docs: cancellable.reset Resets self to its uncancelled state. If cancellable is currently in use by any cancellable operation then the behavior of this function is undefined. Note that it is generally not a good idea to reuse an existing cancellable for more operations after it has been cancelled once, as this function might tempt you to do. The recommended practice is to drop the reference to a cancellable after cancelling it, and let it die with the outstanding async operations. You should create a fresh cancellable for further async operations. Not sure if it applies to other languages or if it's Python specific but just a little food for thought. Apr 21 2017,There's no real point in sending a cancellable along for the ride if you don't have to. (again in Python you can just send None) It doesn't close/cancel the notification. Once DBus gets it it's to late. cancellables are used to cancel pending async calls. Once the call is actually made you can't take it back. In this case to cancel a notification you'd have to be pretty darn quick, basically micro secs. Apr 21 2017,c#39, c#40 The cancellable is going away soon as we're switching to a different dbus library. It's original purpose was so that NotifyCompleteReceiver() didn't get called with an invalid pointer. Apr 21 2017,>as we're switching to a different dbus library As in not Gio? NotifyCompleteReceiver is the async callback for Notify correct? You should get either Null on an error or the Replace Id. In the case of an error handle the error and set the Replace Id back to 0. In my experience though I've never had Notify throw an error(as long as you feed it the proper data). If that were going to happen you would have ran into issues way before you got to that point. Apr 21 2017,>It's original purpose was so that NotifyCompleteReceiver() didn't get called with an invalid pointer. As I said unless you're a space ninja you couldn't possibly cancel that call even if you tried. Apr 27 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/700d8c729c8d08f6aa42f15343f297ff5cfdf393 commit 700d8c729c8d08f6aa42f15343f297ff5cfdf393 Author: thomasanderson <thomasanderson@google.com> Date: Thu Apr 27 03:16:05 2017 Refactor NotificationPlatformBridgeLinux This CL: * Removes the dependency on gdbus from NPBL and uses //src/dbus instead * Handle notifications asynchronously on a dedicated task runner * Modify NativeNotificationDisplayService to allow async initialization of the NotificationPlatformBridge (only on Linux) BUG= 676220 R=thestig@chromium.org Review-Url: https://codereview.chromium.org/2821533003 Cr-Commit-Position: refs/heads/master@{#467563} [modify] https://crrev.com/700d8c729c8d08f6aa42f15343f297ff5cfdf393/chrome/browser/BUILD.gn [add] https://crrev.com/700d8c729c8d08f6aa42f15343f297ff5cfdf393/chrome/browser/notifications/DEPS [modify] https://crrev.com/700d8c729c8d08f6aa42f15343f297ff5cfdf393/chrome/browser/notifications/native_notification_display_service.cc [modify] https://crrev.com/700d8c729c8d08f6aa42f15343f297ff5cfdf393/chrome/browser/notifications/native_notification_display_service.h [modify] https://crrev.com/700d8c729c8d08f6aa42f15343f297ff5cfdf393/chrome/browser/notifications/notification_platform_bridge.h [modify] https://crrev.com/700d8c729c8d08f6aa42f15343f297ff5cfdf393/chrome/browser/notifications/notification_platform_bridge_android.cc [modify] https://crrev.com/700d8c729c8d08f6aa42f15343f297ff5cfdf393/chrome/browser/notifications/notification_platform_bridge_android.h [modify] https://crrev.com/700d8c729c8d08f6aa42f15343f297ff5cfdf393/chrome/browser/notifications/notification_platform_bridge_linux.cc [modify] https://crrev.com/700d8c729c8d08f6aa42f15343f297ff5cfdf393/chrome/browser/notifications/notification_platform_bridge_linux.h [modify] https://crrev.com/700d8c729c8d08f6aa42f15343f297ff5cfdf393/chrome/browser/notifications/notification_platform_bridge_mac.h [modify] https://crrev.com/700d8c729c8d08f6aa42f15343f297ff5cfdf393/chrome/browser/notifications/notification_platform_bridge_mac.mm [modify] https://crrev.com/700d8c729c8d08f6aa42f15343f297ff5cfdf393/chrome/browser/notifications/stub_notification_platform_bridge.cc [modify] https://crrev.com/700d8c729c8d08f6aa42f15343f297ff5cfdf393/chrome/browser/notifications/stub_notification_platform_bridge.h [modify] https://crrev.com/700d8c729c8d08f6aa42f15343f297ff5cfdf393/chrome/common/features.gni May 3 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/c5a3921e63a83c63cd256d9f69eaf837e445699a commit c5a3921e63a83c63cd256d9f69eaf837e445699a Author: thomasanderson <thomasanderson@google.com> Date: Wed May 03 02:45:13 2017 Linux native notifications: Add initial unit test BUG= 676220 R=peter@chromium.org,thestig@chromium.org Review-Url: https://codereview.chromium.org/2849003002 Cr-Commit-Position: refs/heads/master@{#468876} [modify] https://crrev.com/c5a3921e63a83c63cd256d9f69eaf837e445699a/chrome/browser/notifications/notification_platform_bridge_linux.cc [modify] https://crrev.com/c5a3921e63a83c63cd256d9f69eaf837e445699a/chrome/browser/notifications/notification_platform_bridge_linux.h [add] https://crrev.com/c5a3921e63a83c63cd256d9f69eaf837e445699a/chrome/browser/notifications/notification_platform_bridge_linux_unittest.cc [modify] https://crrev.com/c5a3921e63a83c63cd256d9f69eaf837e445699a/chrome/test/BUILD.gn May 3 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/cbe7b7a3a4f5839a11461031904555c4c9eac92a commit cbe7b7a3a4f5839a11461031904555c4c9eac92a Author: thomasanderson <thomasanderson@google.com> Date: Wed May 03 04:26:32 2017 Linux native notifications: Implement GetDisplayedNotifications BUG= 676220 R=peter@chromium.org CC=thestig@chromium.org Review-Url: https://codereview.chromium.org/2848403002 Cr-Commit-Position: refs/heads/master@{#468890} [modify] https://crrev.com/cbe7b7a3a4f5839a11461031904555c4c9eac92a/chrome/browser/notifications/notification_platform_bridge_linux.cc May 3 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8cf43de8a87be9447615ea8bf8965c9979ff18f1 commit 8cf43de8a87be9447615ea8bf8965c9979ff18f1 Author: thestig <thestig@chromium.org> Date: Wed May 03 21:57:43 2017 Linux native notifications: Delete notification after sending CloseNotification. Also check the "dbus_id" when handling signals. Fix some nits as well. BUG= 676220 Review-Url: https://codereview.chromium.org/2859483003 Cr-Commit-Position: refs/heads/master@{#469148} [modify] https://crrev.com/8cf43de8a87be9447615ea8bf8965c9979ff18f1/chrome/browser/notifications/notification_platform_bridge_linux.cc [modify] https://crrev.com/8cf43de8a87be9447615ea8bf8965c9979ff18f1/chrome/browser/notifications/notification_platform_bridge_linux.h [modify] https://crrev.com/8cf43de8a87be9447615ea8bf8965c9979ff18f1/chrome/browser/notifications/notification_platform_bridge_linux_unittest.cc May 4 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/cb0550026886b2cdcd7bb9c975910dcb85056b4c commit cb0550026886b2cdcd7bb9c975910dcb85056b4c Author: thomasanderson <thomasanderson@google.com> Date: Thu May 04 19:52:55 2017 Linux native notifications: Add server capabilities metrics This CL adds the following metrics: * If the system has a dbus object at /org/Freedesktop/Notifications with interface org.Freedesktop.Notifications * Server capabilities as returned by GetCapabilities * Capabilities are stored in |capabilities_|. It is not used now, but will be in the future. * Whether native notifications are actually used BUG= 676220 R=peter@chromium.org,mpearson@chromium.org CC=thestig@chromium.org Review-Url: https://codereview.chromium.org/2856753002 Cr-Commit-Position: refs/heads/master@{#469434} [modify] https://crrev.com/cb0550026886b2cdcd7bb9c975910dcb85056b4c/chrome/browser/notifications/native_notification_display_service.cc [modify] https://crrev.com/cb0550026886b2cdcd7bb9c975910dcb85056b4c/chrome/browser/notifications/notification_platform_bridge_linux.cc [modify] https://crrev.com/cb0550026886b2cdcd7bb9c975910dcb85056b4c/chrome/browser/notifications/notification_platform_bridge_linux_unittest.cc [modify] https://crrev.com/cb0550026886b2cdcd7bb9c975910dcb85056b4c/tools/metrics/histograms/enums.xml [modify] https://crrev.com/cb0550026886b2cdcd7bb9c975910dcb85056b4c/tools/metrics/histograms/histograms.xml May 4 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/14fb44e3336230da008526506b10246140b5bde0 commit 14fb44e3336230da008526506b10246140b5bde0 Author: thomasanderson <thomasanderson@google.com> Date: Thu May 04 20:54:36 2017 Linux native notifications: Add notify and close tests BUG= 676220 R=peter@chromium.org CC=thestig@chromium.org Review-Url: https://codereview.chromium.org/2857293002 Cr-Commit-Position: refs/heads/master@{#469462} [modify] https://crrev.com/14fb44e3336230da008526506b10246140b5bde0/chrome/browser/notifications/notification_platform_bridge_linux.cc [modify] https://crrev.com/14fb44e3336230da008526506b10246140b5bde0/chrome/browser/notifications/notification_platform_bridge_linux_unittest.cc May 9 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/75583a655243a5dc047509e3524822ab8ae06706 commit 75583a655243a5dc047509e3524822ab8ae06706 Author: thomasanderson <thomasanderson@google.com> Date: Tue May 09 04:10:14 2017 Linux native notifications: Escape body text if body-markup is supported This CL makes NotificationPlatformBridgeLinux follow Ubuntu's Notification Development Guidelines: https://wiki.ubuntu.com/NotificationDevelopmentGuidelines BUG= 676220 R=thestig@chromium.org Review-Url: https://codereview.chromium.org/2868613002 Cr-Commit-Position: refs/heads/master@{#470185} [modify] https://crrev.com/75583a655243a5dc047509e3524822ab8ae06706/chrome/browser/notifications/notification_platform_bridge_linux.cc May 9 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/c3b7ae8000d7222d74e2d0942d1d8823ecc0fd52 commit c3b7ae8000d7222d74e2d0942d1d8823ecc0fd52 Author: thomasanderson <thomasanderson@google.com> Date: Tue May 09 05:33:03 2017 Linux native notifications: use image_path instead of image-path for spec 1.1 You're supposed to use 'image_path' for notification servers that only support the FDO Notification spec version 1.1, and 'image-path' for versions >= 1.2. ¯\_(ツ)_/¯ Also, ban servers that have a spec version <= 1.0 because these won't support notification icons at all. BUG= 676220 R=thestig@chromium.org Review-Url: https://codereview.chromium.org/2868433004 Cr-Commit-Position: refs/heads/master@{#470237} [modify] https://crrev.com/c3b7ae8000d7222d74e2d0942d1d8823ecc0fd52/chrome/browser/notifications/notification_platform_bridge_linux.cc [modify] https://crrev.com/c3b7ae8000d7222d74e2d0942d1d8823ecc0fd52/chrome/browser/notifications/notification_platform_bridge_linux_unittest.cc [modify] https://crrev.com/c3b7ae8000d7222d74e2d0942d1d8823ecc0fd52/tools/metrics/histograms/enums.xml May 10 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/bb3a8148c6f47151d82b7b9e490d8157d5e1fa6d commit bb3a8148c6f47151d82b7b9e490d8157d5e1fa6d Author: thestig <thestig@chromium.org> Date: Wed May 10 12:25:14 2017 Linux native notifications: Localize settings button. BUG= 676220 Review-Url: https://codereview.chromium.org/2873233002 Cr-Commit-Position: refs/heads/master@{#470538} [modify] https://crrev.com/bb3a8148c6f47151d82b7b9e490d8157d5e1fa6d/chrome/browser/notifications/notification_platform_bridge_linux.cc May 10 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6112abaaab5e7495941cddfe3a771a84f4c91bd7 commit 6112abaaab5e7495941cddfe3a771a84f4c91bd7 Author: thomasanderson <thomasanderson@google.com> Date: Wed May 10 22:49:01 2017 Linux native notifications: Add percentage to title for progress notifications BUG= 676220 R=peter@chromium.org Review-Url: https://codereview.chromium.org/2866333002 Cr-Commit-Position: refs/heads/master@{#470738} [modify] https://crrev.com/6112abaaab5e7495941cddfe3a771a84f4c91bd7/chrome/browser/notifications/notification_platform_bridge_linux.cc [modify] https://crrev.com/6112abaaab5e7495941cddfe3a771a84f4c91bd7/chrome/browser/notifications/notification_platform_bridge_linux_unittest.cc May 11 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/4e5beb03dc2aed6e11e4ea8acf8ce8e77e10c08e commit 4e5beb03dc2aed6e11e4ea8acf8ce8e77e10c08e Author: thestig <thestig@chromium.org> Date: Thu May 11 01:22:35 2017 Linux native notifications: Add const variables for DBus strings. BUG= 676220 Review-Url: https://codereview.chromium.org/2869313002 Cr-Commit-Position: refs/heads/master@{#470770} [modify] https://crrev.com/4e5beb03dc2aed6e11e4ea8acf8ce8e77e10c08e/chrome/browser/notifications/notification_platform_bridge_linux.cc May 11 2017,Would it be possible to set the X Urgency window hint when a notification is sent? This way a windowmanager (i3 in my case) can highlight the desktop chrome is running on. May 11 2017,> Would it be possible to set the X Urgency window hint when a notification is sent? This way a windowmanager (i3 in my case) can highlight the desktop chrome is running on. Not sure that's possible. If you have multiple tabs to the same origin in different browser windows, we can't be sure which window sent the notification. I think we would be unlikely to make such a change anyway.. :( May 11 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ed102436bc09f3e463af6974efec366c4e8423d0 commit ed102436bc09f3e463af6974efec366c4e8423d0 Author: thomasanderson <thomasanderson@google.com> Date: Thu May 11 20:16:42 2017 Linux native notifications: Support list notifications BUG= 676220 R=peter@chromium.org Review-Url: https://codereview.chromium.org/2875443002 Cr-Commit-Position: refs/heads/master@{#471053} [modify] https://crrev.com/ed102436bc09f3e463af6974efec366c4e8423d0/chrome/browser/notifications/notification_platform_bridge_linux.cc [modify] https://crrev.com/ed102436bc09f3e463af6974efec366c4e8423d0/chrome/browser/notifications/notification_platform_bridge_linux_unittest.cc May 11 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/2b6becedacaa1f65387896b261ae2103e7b5d145 commit 2b6becedacaa1f65387896b261ae2103e7b5d145 Author: thomasanderson <thomasanderson@google.com> Date: Thu May 11 20:37:46 2017 Linux native notifications: Support infinite timeouts BUG= 676220 R=peter@chromium.org Review-Url: https://codereview.chromium.org/2866363002 Cr-Commit-Position: refs/heads/master@{#471060} [modify] https://crrev.com/2b6becedacaa1f65387896b261ae2103e7b5d145/chrome/browser/notifications/notification_platform_bridge_linux.cc [modify] https://crrev.com/2b6becedacaa1f65387896b261ae2103e7b5d145/chrome/browser/notifications/notification_platform_bridge_linux_unittest.cc May 12 2017,Did native notifications stop working for anyone? The CL in c#51 whitelists servers that support an FDO spec version of at least 1.1 (which should be all of them). Our metrics currently show that this has disabled native notifications for a significant portion of users. If anyone is affected by this, can you please post the output of: $ dbus-send --session --dest=org.freedesktop.Notifications --type=method_call --print-reply=literal /org/freedesktop/Notifications org.freedesktop.Notifications.GetServerInformation For example, my machine outputs: gnome-shell GNOME 3.10.4 1.2 May 13 2017,xfce4-notifyd claims it only supports spec 0.9. GetServerInformation output: Xfce Notify Daemon Xfce 0.3.6 0.9 May 15 2017,c#60 Thanks, that would explain it! May 15 2017,X-reffing images for https://codereview.chromium.org/2876603004 May 16 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/42ebc4fd9b073bbcb4a2845aa5862c8fcab2a8a0 commit 42ebc4fd9b073bbcb4a2845aa5862c8fcab2a8a0 Author: thomasanderson <thomasanderson@google.com> Date: Tue May 16 02:56:09 2017 Linux native notifications: Support image notifications BUG= 676220 R=peter@chromium.org Review-Url: https://codereview.chromium.org/2872053002 Cr-Commit-Position: refs/heads/master@{#471993} [modify] https://crrev.com/42ebc4fd9b073bbcb4a2845aa5862c8fcab2a8a0/chrome/browser/notifications/notification_platform_bridge_linux.cc [modify] https://crrev.com/42ebc4fd9b073bbcb4a2845aa5862c8fcab2a8a0/chrome/browser/notifications/notification_platform_bridge_linux_unittest.cc May 16 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f139811a9c8a079969a903da9b718daa04e2ced9 commit f139811a9c8a079969a903da9b718daa04e2ced9 Author: thomasanderson <thomasanderson@google.com> Date: Tue May 16 18:09:29 2017 Linux native notifications: Add attribution This CL adds the origin url to the notification body (if one exists), similar to how Chrome notifications do currently. This is one of the requirements specified in [1] (Google only). [1] https://docs.google.com/a/google.com/document/d/1cqXPpIyCf1RaEYx9XCPjZbO6U5oDeyslajznnG9wd_U/edit?usp=sharing BUG= 676220 R=peter@chromium.org,thestig@chromium.org TBR=mukai@chromium.org Review-Url: https://codereview.chromium.org/2876603004 Cr-Commit-Position: refs/heads/master@{#472160} [modify] https://crrev.com/f139811a9c8a079969a903da9b718daa04e2ced9/chrome/browser/notifications/notification_platform_bridge_linux.cc [modify] https://crrev.com/f139811a9c8a079969a903da9b718daa04e2ced9/chrome/browser/notifications/notification_platform_bridge_linux_unittest.cc [modify] https://crrev.com/f139811a9c8a079969a903da9b718daa04e2ced9/ui/message_center/notification.h May 18 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6f8167514ca5a2fc2235cd4694f60598cd49bbf6 commit 6f8167514ca5a2fc2235cd4694f60598cd49bbf6 Author: thomasanderson <thomasanderson@chromium.org> Date: Thu May 18 22:36:50 2017 Linux native notifications: Remove spec_version check The spec version check was added to blacklist servers that don't support icons. However, some servers like xfce4-notifyd list a spec version that doesn't have icons when the server actually does support icons. More context can be found on: https://codereview.chromium.org/2883983004/ BUG= 676220 R=peter@chromium.org,thestig@chromium.org Review-Url: https://codereview.chromium.org/2891823002 Cr-Commit-Position: refs/heads/master@{#472953} [modify] https://crrev.com/6f8167514ca5a2fc2235cd4694f60598cd49bbf6/chrome/browser/notifications/notification_platform_bridge_linux.cc [modify] https://crrev.com/6f8167514ca5a2fc2235cd4694f60598cd49bbf6/chrome/browser/notifications/notification_platform_bridge_linux_unittest.cc [modify] https://crrev.com/6f8167514ca5a2fc2235cd4694f60598cd49bbf6/tools/metrics/histograms/enums.xml May 18 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/49db099a77be5b377b63e41373840a2b8413e6ff commit 49db099a77be5b377b63e41373840a2b8413e6ff Author: thomasanderson <thomasanderson@chromium.org> Date: Thu May 18 22:49:46 2017 Linux native notifications: Require 'actions' and 'body' capabilities Actions and body text are necessary and expected to be available for web notifications, so blacklist servers that don't support them. The conditions for using native notifications are described at the end of [1] (Google-only). [1] https://docs.google.com/a/google.com/document/d/1cqXPpIyCf1RaEYx9XCPjZbO6U5oDeyslajznnG9wd_U/edit?usp=sharing BUG= 676220 R=peter@chromium.org,thestig@chromium.org Review-Url: https://codereview.chromium.org/2886903005 Cr-Commit-Position: refs/heads/master@{#472959} [modify] https://crrev.com/49db099a77be5b377b63e41373840a2b8413e6ff/chrome/browser/notifications/notification_platform_bridge_linux.cc [modify] https://crrev.com/49db099a77be5b377b63e41373840a2b8413e6ff/chrome/browser/notifications/notification_platform_bridge_linux_unittest.cc May 19 2017,Why blacklist notification daemons that don't allow actions? If a user explicitly chose not to have actions, chromium shouldn't ignore that user preference, and just show its own notifications. The whole point of this integration is to respect the users preference and integrate with their notification daemon of choice, not impose the dev's preferences. May 19 2017,Both Chrome (for configurability of permissions) and various web applications rely on the availability of actions. While user choice definitely is a good thing, we also need to make sure our behaviour continues to be consistent enough to provide a good interoperability story to developers. May 19 2017,So .. developers' wishes are more important than users'? May 19 2017,That's exactly my point: you're saying that what developers want is what matters the most, and TBH, I completely disagree. If a user chose notifications with no actions, you shouldn't force the opposite on him just to satisfy developers. If the user's choice is completely overriden, then there's no choice any more. May 19 2017,Not at all, but I do feel that there needs to be a balance between all parties. Based on excellent research done by Tom, metrics gathered through early users of this feature on Linux and expectations developers have based on the other notification centers supported by browsers we're intending to define a bottom line in regards to functionality. Effectively this comes down to {title, body, actions}, where almost all servers also support an icon. If there's support for additional features we'll progressively enhance to use that too. May 19 2017,I agree that there needs to be a balance between parties. Completely ignoring user preference is not a balance. Recommending that they use a daemon with actions is actually a balance (follow their preference, but recommend why something else might fit better). Again, you're quoting developer expectation. It's nice to consider developer expectations when working on a feature, but it's not nice to override end-user preference merely to suite those expectations. I don't really see any reason why you can't not-show actions to users who've explicitly opted to not-have actions on their notifications. (also, the doc that specifies why these decisions are made seem to be private). May 19 2017,On a more practical note; ignoring the notification daemon if the server doesn't allow actions will only result in one thing: someone hacking a server so that it CLAIMS to support actions, but does not (eg: alter notify-osd to show that capability). This is extremely trivial, and will only result in: 1. Users still having their preference respected, but, 2. Applications being unable to determine what that preference is. May 22 2017,
Jun 2 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ba0a42d0d3c05a9b9e5b125436e29b4a114b115f commit ba0a42d0d3c05a9b9e5b125436e29b4a114b115f Author: miguelg <miguelg@chromium.org> Date: Fri Jun 02 18:56:18 2017 Don't display a settings button for extension notifications BUG= 676220 Review-Url: https://codereview.chromium.org/2921853003 Cr-Commit-Position: refs/heads/master@{#476741} [modify] https://crrev.com/ba0a42d0d3c05a9b9e5b125436e29b4a114b115f/chrome/browser/notifications/notification_platform_bridge_linux.cc Jun 30 2017,What is the current status on this ? Is there a working state i can use yet ? Jun 30 2017,Yeah, but it's guarded behind a flag. To enable it, go to chrome:flags, search for "enable native notifications" and set it to enabled. Jul 4 2017,
Issue 392448 has been merged into this issue. Jul 6 2017,Just created an issue which is related to this work: https://bugs.chromium.org/p/chromium/issues/detail?id=739851 Thanks for adding support for native notifications! Jul 8 2017,The icon and image do not appear in my case. I'm not sure what the badge setting is supposed to do, though. Browser: Chrome 59.0.3071.104 Desktop: KDE Plasma 5.10 Jul 8 2017,It is rather odd behavior that the default action opens settings rather than focusing the tab. Jul 8 2017,I run chrome beta (60), and I get icons and Gmail opens when I click on an notification from Gmail etc. Den lör 8 juli 2017 19:03tng… via monorail < monorail+v2.2349933353@chromium.org> skrev: Jul 9 2017,c#80: Icons and images were not implemented in Chrome 59. They definitely work in 61 (dev) and probably 60 (beta) c#81: Known issue fixed in 60. This was bug 739851 Jul 29 2017,On GNOME (or any DE that's notifications support 'body-markup' for that matter) you need to escape the body of the notification. Currently you are not. Jul 30 2017,c#84: Already fixed in chrome beta or dev Aug 1 2017,Given the last couple round of comments, if there are problems with notifications in version 59 or 60, please considering trying version 61 or 62 first before reporting them. Chances are, newer versions already fixed the problems. Aug 2 2017,> Icons and images were not implemented in Chrome 59. They definitely work in 61 (dev) and probably 60 (beta) Confirmed that icons work in 60 (Stable); however, images and badges still don't seem to work. Sep 6 2017,
Sep 19 2017,I updated to Chrome 61 and the HTML tags are buggy (see image attached). Is this a problem in Chrome or libnotify? Sep 19 2017,c#89: What is the output of: $ dbus-send --session --dest=org.freedesktop.Notifications --type=method_call --print-reply=literal /org/freedesktop/Notifications org.freedesktop.Notifications.GetCapabilities For example, I get: array [ actions action-icons body body-markup icon-static persistence ] Sep 28 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/439ca94889b05ae0062cc47b94587ec23b35ae54 commit 439ca94889b05ae0062cc47b94587ec23b35ae54 Author: Tom Anderson <thomasanderson@chromium.org> Date: Thu Sep 28 20:21:17 2017 Linux native notifications: Only escape <, >, and & characters for HTML Canonical's notification development guidelines [1] specifies that only '<', '>', and '&' characters should be escaped when the notification server supports the 'body-markup' capability. Previously, we were escaping single and double quote characters as well. On Cinnamon, this would cause the ' character to render as '. This CL only escapes the recommended chracters. [1] https://wiki.ubuntu.com/NotificationDevelopmentGuidelines#If_a_notification_uses_a_.2BIBw.3C.2BIB0_or_.2BIBw.26.2BIB0_character.2C_and_it_has_missing_or_wrong_text BUG= 676220 TBR=peter@chromium.org CC=thestig@chromium.org Change-Id: I3e6f6ea60be52a9e4f406c728701f9f666362bc6 Reviewed-on: https://chromium-review.googlesource.com/688294 Reviewed-by: Thomas Anderson <thomasanderson@chromium.org> Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Cr-Commit-Position: refs/heads/master@{#505133} [modify] https://crrev.com/439ca94889b05ae0062cc47b94587ec23b35ae54/chrome/browser/notifications/notification_platform_bridge_linux.cc [modify] https://crrev.com/439ca94889b05ae0062cc47b94587ec23b35ae54/chrome/browser/notifications/notification_platform_bridge_linux_unittest.cc Sep 28 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/00a5cb268b61d7dfed3a253f96de22bbb21a8bec commit 00a5cb268b61d7dfed3a253f96de22bbb21a8bec Author: Tom Anderson <thomasanderson@chromium.org> Date: Thu Sep 28 20:38:37 2017 Linux native notifications: check for body-markup before adding a hyperlink A user on bug 676220 [1] had links showing up in the notification body like "<a href=...". We were already checking against the body-hyperlinks capability before adding the <a> tag, but I suspect that whatever notification server that is interprets 'body-hyperlinks' to mean that it will linkify the body, not that it will correctly handle <a>. This CL includes an added check for 'body-markup' when adding hyperlinks. [1] https://bugs.chromium.org/p/chromium/issues/detail?id=676220#c89 BUG= 676220 R=peter@chromium.org CC=thestig@chromium.org Change-Id: I66ec46c1a467317db3b31d9ba899b2258cb0fb97 Reviewed-on: https://chromium-review.googlesource.com/688111 Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Reviewed-by: Peter Beverloo <peter@chromium.org> Cr-Commit-Position: refs/heads/master@{#505140} [modify] https://crrev.com/00a5cb268b61d7dfed3a253f96de22bbb21a8bec/chrome/browser/notifications/notification_platform_bridge_linux.cc Sep 28 2017,X-reffing image for CL https://chromium-review.googlesource.com/#/c/chromium/src/+/690979 Sep 28 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/14b00f61f942f7667aaf15fdd184844c7a43d979 commit 14b00f61f942f7667aaf15fdd184844c7a43d979 Author: Tom Anderson <thomasanderson@chromium.org> Date: Thu Sep 28 21:10:35 2017 Linux native notifications: Make notifications timeout after 25s BUG= 676220 R=peter@chromium.org CC=thestig@chromium.org Change-Id: I0890514c912963f7e8e0e159ab614545beb047da Reviewed-on: https://chromium-review.googlesource.com/687965 Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Reviewed-by: Peter Beverloo <peter@chromium.org> Cr-Commit-Position: refs/heads/master@{#505156} [modify] https://crrev.com/14b00f61f942f7667aaf15fdd184844c7a43d979/chrome/browser/notifications/notification_platform_bridge_linux.cc [modify] https://crrev.com/14b00f61f942f7667aaf15fdd184844c7a43d979/chrome/browser/notifications/notification_platform_bridge_linux_unittest.cc Sep 28 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/29c369fbf772a073966c4f14e5b5acab3cab20d2 commit 29c369fbf772a073966c4f14e5b5acab3cab20d2 Author: Tom Anderson <thomasanderson@chromium.org> Date: Thu Sep 28 21:34:47 2017 Linux native notifications: Add suppress-sound hint when notification is silent BUG= 676220 R=peter@chromium.org CC=thestig@chromium.org Change-Id: I20a4a246b89f8be56bfdffcbfb7b7c38be203eaa Reviewed-on: https://chromium-review.googlesource.com/687887 Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Reviewed-by: Peter Beverloo <peter@chromium.org> Cr-Commit-Position: refs/heads/master@{#505165} [modify] https://crrev.com/29c369fbf772a073966c4f14e5b5acab3cab20d2/chrome/browser/notifications/notification_platform_bridge_linux.cc [modify] https://crrev.com/29c369fbf772a073966c4f14e5b5acab3cab20d2/chrome/browser/notifications/notification_platform_bridge_linux_unittest.cc Oct 3 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/141f543682c748e5d6e5153dc3040d0eb1e815e2 commit 141f543682c748e5d6e5153dc3040d0eb1e815e2 Author: Tom Anderson <thomasanderson@chromium.org> Date: Tue Oct 03 16:54:46 2017 Linux native notifications: Explicitly pass app_name and app_icon The app_name and app_icon used to be passed implicitly via the desktop file hint. Some notification servers, including Cinnamon, do not look at this hint. For this reason, always explicitly pass them. Also, Cinnamon now renders *both* a product logo and a notification icon, and notifications can look like this: https://bugs.chromium.org/p/chromium/issues/detail?id=676220#c93 BUG= 676220 R=peter@chromium.org,thestig@chromium.org Change-Id: I1fa63c793b51d642458a75bedd90bc1e512f6c8d Reviewed-on: https://chromium-review.googlesource.com/690979 Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Reviewed-by: Peter Beverloo <peter@chromium.org> Cr-Commit-Position: refs/heads/master@{#506070} [modify] https://crrev.com/141f543682c748e5d6e5153dc3040d0eb1e815e2/chrome/browser/notifications/notification_platform_bridge_linux.cc Oct 3 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/3d0f14dabec8dcceb76c1ba7d888e246b8b8d7a1 commit 3d0f14dabec8dcceb76c1ba7d888e246b8b8d7a1 Author: Tom Anderson <thomasanderson@chromium.org> Date: Tue Oct 03 17:06:41 2017 Linux native notifications: Limit origin url to eTLD+1 BUG= 676220 R=peter@chromium.org CC=thestig@chromium.org Change-Id: I0725d4ca3eed639f0284f3f03191d58abcca9295 Reviewed-on: https://chromium-review.googlesource.com/692682 Reviewed-by: Peter Beverloo <peter@chromium.org> Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Cr-Commit-Position: refs/heads/master@{#506079} [modify] https://crrev.com/3d0f14dabec8dcceb76c1ba7d888e246b8b8d7a1/chrome/browser/notifications/notification_platform_bridge_linux.cc [modify] https://crrev.com/3d0f14dabec8dcceb76c1ba7d888e246b8b8d7a1/chrome/browser/notifications/notification_platform_bridge_linux_unittest.cc Oct 12 2017,
Oct 12 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/88473e9c8382bb30d843a394bd6e66a7f005e8c2 commit 88473e9c8382bb30d843a394bd6e66a7f005e8c2 Author: Tom Anderson <thomasanderson@chromium.org> Date: Thu Oct 12 19:59:00 2017 Linux native notifications: Add a blank line to separate the origin from the body This CL adds another newline to separate the notification attribution from the message body. Also, it simplifies the logic for adding newlines so that we don't have to check if there's any text beforehand. BUG= 676220 R=peter@chromium.org CC=thestig@chromium.org Change-Id: I1ec6d7f4a570537bf7b44f03a0de79de52280c07 Reviewed-on: https://chromium-review.googlesource.com/709944 Reviewed-by: Peter Beverloo <peter@chromium.org> Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Cr-Commit-Position: refs/heads/master@{#508406} [modify] https://crrev.com/88473e9c8382bb30d843a394bd6e66a7f005e8c2/chrome/browser/notifications/notification_platform_bridge_linux.cc [modify] https://crrev.com/88473e9c8382bb30d843a394bd6e66a7f005e8c2/chrome/browser/notifications/notification_platform_bridge_linux_unittest.cc Oct 27 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/65ffa597151e7a0503fb98d81e0f309ae2e852aa commit 65ffa597151e7a0503fb98d81e0f309ae2e852aa Author: Tom Anderson <thomasanderson@chromium.org> Date: Fri Oct 27 16:47:01 2017 Enable native notifications on desktop Linux BUG= 676220 ,773790 R=peter@chromium.org CC=thestig@chromium.org,owencm@chromium.org Change-Id: Icc6334e093a72b936c14c963713173231567c63d Reviewed-on: https://chromium-review.googlesource.com/740123 Reviewed-by: Peter Beverloo <peter@chromium.org> Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Cr-Commit-Position: refs/heads/master@{#512207} [modify] https://crrev.com/65ffa597151e7a0503fb98d81e0f309ae2e852aa/chrome/common/chrome_features.cc Nov 1 2017, Project MemberThe following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/67b9d2c3efddfae1b7f9396aa9157b7c5dce4ecf commit 67b9d2c3efddfae1b7f9396aa9157b7c5dce4ecf Author: Tom Anderson <thomasanderson@chromium.org> Date: Wed Nov 01 16:58:04 2017 Linux native notifications: Fix clickable buttons On Peter's notification generator [1], creating a notification with "Action" set to alert() and everyting else set to default settings, notification->clickable() is false when I expected the notification to be clickable. This CL always makes notifications clickable regardless of the clickable() state. [1] https://tests.peter.sh/notification-generator/ BUG= 676220 R=peter@chromium.org CC=thestig@chromium.org Change-Id: I326b6fa5b64e8a18aa95ea47dfbcbe5caaaa9bdc Reviewed-on: https://chromium-review.googlesource.com/748213 Reviewed-by: Peter Beverloo <peter@chromium.org> Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Cr-Commit-Position: refs/heads/master@{#513159} [modify] https://crrev.com/67b9d2c3efddfae1b7f9396aa9157b7c5dce4ecf/chrome/browser/notifications/notification_platform_bridge_linux.cc Dec 8 2017,
Dec 13 2017,Hi, I'm running version 63.0.3239.84, and still have the problem with the hyperlinks. The notification daemon (xfce4-notifyd) capabilities are: array [ actions body body-hyperlinks body-markup icon-static x-canonical-private-icon-only ] Is this an issue of xfce's notification daemon? Dec 13 2017,c#103 is the issue fixed with google-chrome-unstable, which is at 64.0.3282.24 ? Dec 13 2017,Hi, thank you for your fast reply. Unfortunately it's the same result. Dec 13 2017,c#105 I believe this is a bug in xfce4-notifyd. It claims it supports both body-hyperlinks and body-markup, so the hyperlink should work. They work in other notification daemons. Dec 13 2017,Ok. Thank you very much for your help! Dec 22 2017,
Marking as fixed. Native notifications will be enabled by default in M64. If there are further issues, please file a separate bug.
Showing comments 9 - 108
of 108
Older ›
|
||||||||
►
Sign in to add a comment |