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 655619 - Error pages shouldn't be stored in history
Error pages shouldn't be stored in history
Status: RESOLVED FIXED
Product: epiphany
Classification: Core
Component: History
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Epiphany Maintainers
Epiphany Maintainers
: 356778 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2011-07-30 09:07 UTC by Diego Escalante Urrelo (not reading bugmail)
Modified: 2012-08-14 09:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Don't restore zoom level and mark visits for error pages (2.24 KB, patch)
2012-04-04 11:16 UTC, Zan Dobersek
rejected Details | Review
Do not store error pages in history (5.05 KB, patch)
2012-08-14 09:39 UTC, Xan Lopez
committed Details | Review

Description Diego Escalante Urrelo (not reading bugmail) 2011-07-30 09:07:01 UTC
When a page can't be loaded, it shouldn't be stored in history. Otherwise you end with lots of typos in the browsing history.

This seems easy on a first check
Comment 1 Diego Escalante Urrelo (not reading bugmail) 2011-07-30 09:09:51 UTC
*** Bug 356778 has been marked as a duplicate of this bug. ***
Comment 2 Diego Escalante Urrelo (not reading bugmail) 2012-03-31 02:20:53 UTC
This is easy with EPHY_PAGE_VISIT_* in ephy-web-view.

I am wondering though if we should handle this in history-service.
Perhaps a new type of visit? or ignore this kind of visit in the service.

I wonder if it is better to have the decision behind the history API or in ephy-web-view.
Comment 3 Zan Dobersek 2012-04-04 11:16:48 UTC
Created attachment 211275 [details] [review]
Don't restore zoom level and mark visits for error pages

Introduce a new page visit value for error pages and set it when
necessary. This makes it possible to avoid restoring zoom level
and marking visit for these pages, meaning sites that have failed
to load are not leaked into the history database.
Comment 4 Xan Lopez 2012-07-31 14:10:24 UTC
(In reply to comment #3)
> Created an attachment (id=211275) [details] [review]
> Don't restore zoom level and mark visits for error pages
> 
> Introduce a new page visit value for error pages and set it when
> necessary. This makes it possible to avoid restoring zoom level
> and marking visit for these pages, meaning sites that have failed
> to load are not leaked into the history database.

Hum, shouldn't we just not store anything at all for error pages in the DB?
Comment 5 Claudio Saavedra 2012-07-31 14:34:55 UTC
Comment on attachment 211275 [details] [review]
Don't restore zoom level and mark visits for error pages

Yes, I think we shouldn't store erroneous pages in the history.
Comment 6 Xan Lopez 2012-08-14 09:39:12 UTC
Created attachment 221112 [details] [review]
Do not store error pages in history
Comment 7 Xan Lopez 2012-08-14 09:47:20 UTC
Comment on attachment 221112 [details] [review]
Do not store error pages in history

Pushed to master ad4b199dc292
Comment 8 Xan Lopez 2012-08-14 09:47:31 UTC
Closing.