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 755585 - Unused signals in EphyWebView
Unused signals in EphyWebView
Status: RESOLVED FIXED
Product: epiphany
Classification: Core
Component: General
3.18.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: Epiphany Maintainers
Epiphany Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-09-24 20:18 UTC by Michael Catanzaro
Modified: 2016-02-10 14:06 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
web-view: Remove signals that are never emitted or connected to (3.48 KB, patch)
2016-02-09 19:40 UTC, Michael Catanzaro
committed Details | Review
web-view: Fix a typo (794 bytes, patch)
2016-02-09 19:40 UTC, Michael Catanzaro
committed Details | Review
web-view: Remove ge_popups_blocked signal (2.42 KB, patch)
2016-02-09 19:40 UTC, Michael Catanzaro
committed Details | Review
web-view: Remove ge_modal_alert signal (3.09 KB, patch)
2016-02-09 19:40 UTC, Michael Catanzaro
committed Details | Review
web-view: Remove new-document-now signal (1.64 KB, patch)
2016-02-09 19:40 UTC, Michael Catanzaro
committed Details | Review
web-view: Remove loading-homepage signal (1.51 KB, patch)
2016-02-09 19:40 UTC, Michael Catanzaro
committed Details | Review

Description Michael Catanzaro 2015-09-24 20:18:45 UTC
There are tons of unused signals (signals that are never emitted, signals that are never connected to, and signals that are connected to only by EphyWebView itself) in EphyWebView. We should remove at least the ones that are never emitted... and maybe the others too?
Comment 1 Michael Catanzaro 2016-02-09 19:40:29 UTC
Created attachment 320743 [details] [review]
web-view: Remove signals that are never emitted or connected to
Comment 2 Michael Catanzaro 2016-02-09 19:40:32 UTC
Created attachment 320744 [details] [review]
web-view: Fix a typo
Comment 3 Michael Catanzaro 2016-02-09 19:40:36 UTC
Created attachment 320745 [details] [review]
web-view: Remove ge_popups_blocked signal

It's connected to, but never emitted
Comment 4 Michael Catanzaro 2016-02-09 19:40:39 UTC
Created attachment 320746 [details] [review]
web-view: Remove ge_modal_alert signal

It's connected to, but never emitted
Comment 5 Michael Catanzaro 2016-02-09 19:40:43 UTC
Created attachment 320747 [details] [review]
web-view: Remove new-document-now signal

Nothing connects to it
Comment 6 Michael Catanzaro 2016-02-09 19:40:46 UTC
Created attachment 320748 [details] [review]
web-view: Remove loading-homepage signal

Nothing connects to it. Also, we don't have homepages anymore.
Comment 7 Michael Catanzaro 2016-02-10 14:06:02 UTC
Attachment 320743 [details] pushed as c78ab6f - web-view: Remove signals that are never emitted or connected to
Attachment 320744 [details] pushed as 5b662da - web-view: Fix a typo
Attachment 320745 [details] pushed as 4ee6439 - web-view: Remove ge_popups_blocked signal
Attachment 320746 [details] pushed as ea93a62 - web-view: Remove ge_modal_alert signal
Attachment 320747 [details] pushed as 0c5dec3 - web-view: Remove new-document-now signal
Attachment 320748 [details] pushed as 9738979 - web-view: Remove loading-homepage signal