Ticket #3620 (closed defect: fixed)

Opened 12 months ago

Last modified 11 months ago

Browse is displaying "Loading..." in the tab that triggers a download or a PDF load

Reported by: manuq Owned by: erikos
Priority: Unspecified by Maintainer Milestone: 0.96
Component: Browse Version: Unspecified
Severity: Unspecified Keywords: 12.1.0, patch, olpc-test-passed
Cc: humitos Distribution/OS: Unspecified
Bug Status: Unconfirmed

Description

TestCase:

Thay are changing now to "Loading...".

Attachments

fix_loading.patch Download (2.8 KB) - added by manuq 12 months ago.
0001-Unset-loading.-message-when-the-loading-fails-SL-362.patch Download (3.7 KB) - added by manuq 12 months ago.

Change History

Changed 12 months ago by manuq

  • cc humitos added
  • keywords 12.1.0 added
  • component changed from untriaged to Browse
  • owner set to erikos
  • milestone changed from Unspecified by Release Team to 0.96

Changed 12 months ago by manuq

Changed 12 months ago by manuq

When browse handles a pdf or a download, the loading status of the webview in the current tab changes to WEBKIT_LOAD_PROVISIONAL and then to WEBKIT_LOAD_FAILED. I will take a look at Epiphany code to see if this is expected, as WebKit documentation is not telling me.

fix_loading.patch is a fix for this bug, if the "failed" status is expected.

Changed 12 months ago by manuq

Yes, what Ephy does is similar, it just unsets the "Loading..." title when failed. Sending the patch to the mailing list.

Changed 12 months ago by manuq

  • keywords 12.1.0, patch added; 12.1.0 removed

Changed 12 months ago by humitos

I think it works properly. I'm going to explain what happens here:

  1. Open Browse
  2. Go to this url ticket
  3. Click on the link file (in the description)
  4. The tab title changes to "Loading..."
  5. When the download is done, the tab title changes again to title of the webpage.

If this is the expected behavior, this patch works! :)

I tested this on XO 1.75 os11 and git version of Browse

Changed 12 months ago by erikos

This does work as expected. Good work to look at the epiphany code for reference. Please push that one.

Changed 12 months ago by manuq

  • keywords patch, olpc-test-pending added; patch removed
  • status changed from new to closed
  • resolution set to fixed

Pushed as d6c56f8184de2b3980aca34fb644d2a8f0f4b64f

Changed 11 months ago by greenfeld

  • keywords olpc-test-passed added; olpc-test-pending removed

This is fixed in 12.1.0 os16/Browse-140.

Note: See TracTickets for help on using tickets.