Esc closes the pdf tab

[humitos]

Steps to reproduce it:

1. Open Browse
2. Type "pdf example" on the URL entry
3. Click on the first result
4. A new tab is opened
5. Press Esc

Issue:

The PDF tab is closed

Expected behaviour:

Nothing should happen

Comment:

I get this error on the log:

1353330976.124008 DEBUG web-activity: keyboard: Stop loading
1353330976.124239 DEBUG root: Download PDF canceled
1353330976.124394 DEBUG root: Download error! code 0, detail 0: El usuario cancel�~C³ la descarga

Marked as regression. This don't happen in 12.1.0

[humitos]

This was introduce in: 41968d61. stop_loading for PDFTabPage does close the tab.

[manuq]

I see. Good catch! Patch looks good too. Pushed 843faffb .

[greenfeld]

I still see the PDF tab closed with this patch present in Browse-148 if I press ESC after clicking a PDF link but before the time the Web Browser has had time to fully download the file.

What exactly was fixed by this patch?

(Of note is that I'm slightly nervous about seeing relative (<=, <) comparisons made with enums or a flag field, but I presume you have double-checked this is acceptable.)

[humitos]

Replying to greenfeld:

I still see the PDF tab closed with this patch present in Browse-148 if I press ESC after clicking a PDF link but before the time the Web Browser has had time to fully download the file.

This behaviour is correct.

What exactly was fixed by this patch?

This patch makes ESC to not close the pdf tab once it has fully downloaded. I mean, you click on a pdf link, wait until it finishes the download progress and there press ESC: without this patch Browse would close the Tab.

[greenfeld]

ESC does not close the window after the PDF is fully downloaded in Browse-148.

[dnarvaez]

Milestone 0.98 deleted