Bug 10237

Summary:

[S60] UI CR JMAK-6LFSH8 Unfinished download should be visible in the download list, before connection is established

Product:

WebKit

Reporter:

Stanislav Paltis <Stanislav.Paltis>

Component:

WebKit Misc.

Assignee:

Nobody <webkit-unassigned>

Status:

CLOSED FIXED

Severity:

Enhancement

CC:

S60webkit

Priority:

Keywords:

PlatformOnly

Version:

420+

Hardware:

S60 Emulator

OS:

S60 3rd edition

Attachments:

Description	Flags
Proposed patch for HttpDownloadObserver change	yongjun.zhang: review+
Proposed patch v2. fixed tabs -> spaces, crashes	zalan: review+
S60Headers with download Manager includes.	bradley.morrison: review+
Patch for S60 Headers.	zalan: review+

Stanislav Paltis

Reported 2006-08-03 12:26:39 PDT

Unfinished download should be visible in the download list, before connection is established Current situation: When the browser shuts down (crash or poweroff), ongoing download sessions will be paused, and stored under Tools->Download. However, this menu does not show up when the browser is launched after the interruption. It only shows up when connection is established, by going to some website, e.g. This is obviously unfriendly because a user looking to resume the unfinished download will not find it immediately. Resuming download must be a legitmate use case, just like openning a web page. if the user selects to resume a download as the first action, the 'connect to server' popup appears, and the download resumes after the user confirm the query.

Attachments
Proposed patch for HttpDownloadObserver change (11.90 KB, patch) 2006-08-03 13:32 PDT, Stanislav Paltis	yongjun.zhang: review+	Details Formatted Diff Diff
Proposed patch v2. fixed tabs -> spaces, crashes (25.90 KB, patch) 2006-08-05 16:19 PDT, Stanislav Paltis	zalan: review+	Details Formatted Diff Diff
S60Headers with download Manager includes. (240.48 KB, patch) 2006-08-07 18:03 PDT, Stanislav Paltis	bradley.morrison: review+	Details Formatted Diff Diff
Patch for S60 Headers. (1.03 KB, patch) 2006-08-07 18:04 PDT, Stanislav Paltis	zalan: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Bradley Morrison

Comment 1 2006-08-03 13:12:22 PDT

Setting to new, adding bug metadata

Stanislav Paltis

Comment 2 2006-08-03 13:32:14 PDT

Created attachment 9855 [details] Proposed patch for HttpDownloadObserver change

Yongjun Zhang

Comment 3 2006-08-03 15:00:48 PDT

Comment on attachment 9855 [details] Proposed patch for HttpDownloadObserver change r=me

Bradley Morrison

Comment 4 2006-08-03 15:06:16 PDT

Applied, r15783. Please do not include tabs in source files.

Bradley Morrison

Comment 5 2006-08-03 15:29:25 PDT

This broke the SDK build.

Bradley Morrison

Comment 6 2006-08-03 15:42:39 PDT

Rolled out r15783 (r15786).

Stanislav Paltis

Comment 7 2006-08-05 16:19:46 PDT

Created attachment 9895 [details] Proposed patch v2. fixed tabs -> spaces, crashes

zalan

Comment 8 2006-08-07 15:26:14 PDT

Comment on attachment 9895 [details] Proposed patch v2. fixed tabs -> spaces, crashes r=me

zalan

Comment 9 2006-08-07 15:26:53 PDT

r15811

Stanislav Paltis

Comment 10 2006-08-07 18:03:58 PDT

Created attachment 9923 [details] S60Headers with download Manager includes.

Stanislav Paltis

Comment 11 2006-08-07 18:04:33 PDT

Created attachment 9924 [details] Patch for S60 Headers.

zalan

Comment 12 2006-08-08 07:47:00 PDT

Comment on attachment 9924 [details] Patch for S60 Headers. r=me

zalan

Comment 13 2006-08-08 07:48:42 PDT

r15816

Krishna

Comment 14 2006-08-10 12:14:46 PDT

Closed.

Joel Parks

Comment 15 2011-03-21 11:52:25 PDT

re-purposing InTSW keyword for use by QtWebkit team

Note You need to log in before you can comment on or make changes to this bug.