<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>243255</bug_id>
          
          <creation_ts>2022-07-27 10:20:57 -0700</creation_ts>
          <short_desc>[GTK] Desktop notifications are not shown in GNOME</short_desc>
          <delta_ts>2022-07-28 07:41:48 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKitGTK</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>WORKSFORME</resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=240347</see_also>
          <bug_file_loc>https://ttsvetko.github.io/HTML5-Desktop-Notifications/</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Major</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Sergio Villar Senin">svillar</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>bugs-noreply</cc>
    
    <cc>cgarcia</cc>
    
    <cc>mcatanzaro</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1887094</commentid>
    <comment_count>0</comment_count>
    <who name="Sergio Villar Senin">svillar</who>
    <bug_when>2022-07-27 10:20:57 -0700</bug_when>
    <thetext>Steps to reproduce:

1. visit https://ttsvetko.github.io/HTML5-Desktop-Notifications/
2. enter some text for title and body of the notification and click &quot;Display Notification&quot; button

Expected result:
A desktop notification is shown

Actual result:
No desktop notification is shown</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1887097</commentid>
    <comment_count>1</comment_count>
    <who name="Sergio Villar Senin">svillar</who>
    <bug_when>2022-07-27 10:28:30 -0700</bug_when>
    <thetext>I&apos;ve just realized that the notification is properly sent. The things is that the desktop decides not to show it immediately but instead silently queue it in the notification area (which is usually hidden).

Perhaps it&apos;s just a matter of setting a different priority or type on the notification so that desktop shows it. With the current status, it looks like desktop notifications are not working.

For reference, the same page in Firefox or Chrome shows the desktop notification in real time.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1887118</commentid>
    <comment_count>2</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2022-07-27 11:39:24 -0700</bug_when>
    <thetext>Users have been complaining about this for a while. Basically all of the code here is in webkitWebViewShowNotification in WebKitWebView.cpp. Not sure what&apos;s wrong.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1887305</commentid>
    <comment_count>3</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2022-07-28 03:38:18 -0700</bug_when>
    <thetext>I guess you have tried with epiphany, not MiniBrowser. It works fine in MiniBrowser and the problem is epiphany setting all notifications as low urgency and gnome-shell never showing popups for low urgency notifications:

https://gitlab.gnome.org/GNOME/epiphany/-/blob/master/src/ephy-shell.c#L1010
https://gitlab.gnome.org/GNOME/gnome-shell/-/blob/main/js/ui/messageTray.js#L765</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1887315</commentid>
    <comment_count>4</comment_count>
    <who name="Sergio Villar Senin">svillar</who>
    <bug_when>2022-07-28 06:49:21 -0700</bug_when>
    <thetext>(In reply to Carlos Garcia Campos from comment #3)
&gt; I guess you have tried with epiphany, not MiniBrowser. It works fine in
&gt; MiniBrowser and the problem is epiphany setting all notifications as low
&gt; urgency and gnome-shell never showing popups for low urgency notifications:

Right, I&apos;ve just tried MB as well and saw the same
 
&gt; https://gitlab.gnome.org/GNOME/epiphany/-/blob/master/src/ephy-shell.c#L1010
&gt; https://gitlab.gnome.org/GNOME/gnome-shell/-/blob/main/js/ui/messageTray.
&gt; js#L765</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1887321</commentid>
    <comment_count>5</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2022-07-28 07:40:11 -0700</bug_when>
    <thetext>(In reply to Carlos Garcia Campos from comment #3)
&gt; I guess you have tried with epiphany, not MiniBrowser. It works fine in
&gt; MiniBrowser and the problem is epiphany setting all notifications as low
&gt; urgency

Where do you see that? Epiphany does not touch the GNotificationPriority, so they should use G_NOTIFICATION_PRIORITY_NORMAL.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1887323</commentid>
    <comment_count>6</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2022-07-28 07:41:48 -0700</bug_when>
    <thetext>(In reply to Michael Catanzaro from comment #5) 
&gt; Where do you see that? Epiphany does not touch the GNotificationPriority, so
&gt; they should use G_NOTIFICATION_PRIORITY_NORMAL.

Oh, I see you already fixed it in https://gitlab.gnome.org/GNOME/epiphany/-/commit/45efd6da62991fad1388ebda837e50ed7b5b3be2.

Next time, if you use a permanent link like https://gitlab.gnome.org/GNOME/epiphany/-/blob/8d5130cb13fb9a0609ce6c9f7b826fcbc260df28/src/ephy-shell.c#L1010 you can avoid confusing me. :)</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>