The browser doesn't need to store urls with 404 status in the history

[Adnane Belmadiaf]

How to reproduce :
- Open "http://www.google.com/test" in the browser (this a 404 url)
- Click on the addressbar, type "test" this show you the url "http://www.google.com/test" from the history

[Olivier Tilloy (osomon)]

In theory we should be able to filter out 404 URLs using the loadRequest.errorCode attribute in the onLoadingChanged handler (see http://qt-project.org/doc/qt-5.1/qtwebkit/qml-qtwebkit3-webview.html#onLoadingChanged-signal), however in practice the errorCode seems to always be 0.

[Adnane Belmadiaf (daker)]

I just tested that, yes it's always 0 so i think it's a bug

[Olivier Tilloy (osomon)]

Tested again with oxide, and for a 404 we’re getting an errorCode of 0. Not really sure what the error code is for, what we’d really need is a 'statusCode' property that contains the HTTP status code returned by the server.

[Olivier Tilloy (osomon)]

Adding an oxide task: Chris, do you think the OxideQLoadEvent class could gain a statusCode property?

[Chris Coulson (chrisccoulson)]

Yeah, that would be fine

[Chris Coulson (chrisccoulson)]

Note, "errorCode" is for errors that prevent us from getting a response and which cause the load to fail. A 404 is a valid response from the server, hence no error code :)

[Chris Coulson (chrisccoulson)]

And LoadEvent.errorCode is a bit of a bogus API, as this is basically the error code from Chromium's network stack, which is fairly meaningless outside of Chromium. LoadEvent.errorDomain is better (although it will still indicate ErrorDomainNone for a 404)

[Launchpad Janitor (janitor)]

This bug was fixed in the package webbrowser-app - 0.23+15.10.20150903.1-0ubuntu1

---------------
webbrowser-app (0.23+15.10.20150903.1-0ubuntu1) wily; urgency=medium

  [ CI Train Bot ]
  * New rebuild forced.
  * Resync trunk.

  [ Michael Terry ]
  * Use the Ubuntu/Canonical search tag when searching with DuckDuckGo.
    (LP: #1490283)

  [ Olivier Tilloy ]
  * Display a friendly message when the renderer process crashes or is
    killed. This adds a runtime dependency for webbrowser-app-autopilot
    on python3-psutil. (LP: #1375272)
  * Do not display the bottom edge hint on tablets in wide mode. (LP:
    #1488995)
  * Update translation template.

  [ Riccardo Padovani ]
  * Don't store urls with status different from 2xx in the history. (LP:
    #1244335)
  * Don't store urls with status different from 2xx in the history. (LP:
    #1244335)

  [ Ugo Riboni ]
  * Allow choosing the bookmark folder when bookmarking a link from the
    context menu. Disable the bookmark option when the link is already
    bookmarked. Ensure the bookmark star state in the chrome is always
    consistent with the bookmarked state of the current webview URL.
    (LP: #1477314)
  * Properly reset focus when the current tab changes (including as a
    result of closing tabs). (LP: #1488470)

 -- CI Train Bot <email address hidden> Thu, 03 Sep 2015 09:46:35 +0000