Tab previews are deleted while still in active list

[Pat McGowan]

v174 on mx4

I was watching the contents of the .cache/webbrowser-app/captures directory while changing tabs.

Select a tab from the list (happened to be blank)
The web page is loaded
The file containing preview for the previously active tab, top in the list, is deleted from the disk.
The tab is still shown in the active list but now its blank.

I cannot reproduce this every time.

Adding a new tab subsequently crashed the browser

[Olivier Tilloy (osomon)]

> The file containing preview for the previously active tab, top in the
> list, is deleted from the disk.

There should never be a preview for the current tab (unless it’s also a top site). In the tabs list view, the first tab is a special case, it shows the live webview, not a cached capture. When selecting a different tab, a capture is taken for the tab that was current (so the opposite of what you’re describing should be happening).

I found some obvious issues with the handling of previews, and fixed them in the linked branch.
I’m not sure how to reproduce the issue you’re describing though, so I can’t guarantee it’s fixed by my branch (which, regardless, improves the current situation).

[Launchpad Janitor (janitor)]

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

---------------
webbrowser-app (0.23+16.04.20151222.1-0ubuntu1) xenial; urgency=medium

  * Add keyboard shorcuts for 'next' (Ctrl+G) and 'previous'
    (Ctrl+Shift+G) while in find-in-page mode. (LP: #1522152)
  * Do not allow interacting with tabs and the chrome while the settings
    page is shown. (LP: #1524353)
  * Fix a crash when clearing the navigation history while the new tab
    view is open. (LP: #1525937)
  * Fix deleting multiple domains from the history view. Add unit tests
    for the HistoryView component. (LP: #1527144)
  * Fix some obvious issues in PreviewManager. (LP: #1519019)
  * Fix the tab preview sometimes covering the tab chrome, in the tabs
    view. Offset tab previews by 1px to cover the semi-transparent
    horizontal line at the bottom of the tab chrome asset. (LP:
    #1520624)
  * Recognize openoffice and MS-office file formats as documents, now
    that the document viewer app knows how to render them. (LP:
    #1523452)
  * Update translation template.
  * Use the screen size (diagonal in millimeters) to determine which
    default UA string and which UA overrides to use. An empirical
    threshold of 19cm is being used, it is subject to further
    adjustments. (LP: #1504084)

 -- Olivier Tilloy <email address hidden> Tue, 22 Dec 2015 18:15:54 +0000