After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 777835 - Set initial notification permissions
Set initial notification permissions
Status: RESOLVED FIXED
Product: epiphany
Classification: Core
Component: General
3.23.x
Other Linux
: Normal normal
: ---
Assigned To: Epiphany Maintainers
Epiphany Maintainers
Depends on:
Blocks:
 
 
Reported: 2017-01-27 15:02 UTC by Michael Catanzaro
Modified: 2017-01-31 01:46 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
permissions-manager: Cache permissions when enumerating them (9.72 KB, patch)
2017-01-31 01:46 UTC, Michael Catanzaro 		committed Details | Review
Set initial notification permissions (9.70 KB, patch)
2017-01-31 01:46 UTC, Michael Catanzaro 		committed Details | Review

Description Michael Catanzaro 2017-01-27 15:02:13 UTC 
Set initial notification permissions using new WebKit API: https://bugs.webkit.org/show_bug.cgi?id=163366
Comment 1 Michael Catanzaro 2017-01-31 01:46:32 UTC 
The following fixes have been pushed:
ddcc22e permissions-manager: Cache permissions when enumerating them
3c87365 Set initial notification permissions
Comment 2 Michael Catanzaro 2017-01-31 01:46:37 UTC 
Created attachment 344620 [details] [review]
permissions-manager: Cache permissions when enumerating them

GKeyFile performs synchronous file I/O to read permissions from the
permissions store, so we'd better do it as little as possible. This
commit ensures we read from the notification permissions file just twice
ever, instead of twice each time a new tab is created.
Comment 3 Michael Catanzaro 2017-01-31 01:46:42 UTC 
Created attachment 344621 [details] [review]
Set initial notification permissions