Bug 14613

Summary:

[S60] RSS-links does not work anymore if some article was downloaded from web feeds.

Product:

WebKit

Reporter:

Hong Zhao <hong.a.zhao>

Component:

WebKit API

Assignee:

Nobody <webkit-unassigned>

Status:

CLOSED FIXED

Severity:

Normal

Keywords:

PlatformOnly

Priority:

Version:

523.x (Safari 3)

Hardware:

S60 Emulator

OS:

S60 3rd edition

Attachments:

Description	Flags
Pass BrowserControl's specialLoadObserver to correspondent defaultContentHandler in 3.2.	Sachin.Padma: review-
Updated with the review comments.	none
Updated with the review comments.	Sachin.Padma: review-
Pass the Browser control's special load observer to the correspondent default load handler in 3.1m.	Sachin.Padma: review-
Update with the review comments.	none
Update the patch with review comments.	none
Update with review comments.	Sachin.Padma: review+
Updated with review comments in 3.1m.	Sachin.Padma: review+

Hong Zhao

Reported 2007-07-13 13:24:05 PDT

TSW# ID: EMKU-72Z8EY - Steps to reproduce: 1. go to URL http://www.hs.fi/rss/ 2. select the link http://www.hs.fi/uutiset/rss/ under "25 tuoreinta uutisotsikkoa viimeisen vuorokauden ajalta:" 3. Open web feed and open some latest article from there. 4. Close article after it was downloaded. 5. Press RSK couple of times to go back to the web feeds and finally to the bookmarks view. 6. Press LSK Options and select 'Back to the page'. 7 Try to open another rss links (e.g. Kotimaa) from the page. 8. Select 'Exit' from Options menu. - Actual outcome: 7. "Not supported" is displayed if any of rss-links was tried to open. User can not open rss-links anymore if some article was downloaded from web feeds. 8. "Application closed: Web!" is displayed. - Expected outcome: RSS-links should be opened succesfully and web feeds should be added to web feeds folder.

Attachments
Pass BrowserControl's specialLoadObserver to correspondent defaultContentHandler in 3.2. (13.42 KB, patch) 2007-07-13 20:45 PDT, Hong Zhao	Sachin.Padma: review-	Details Formatted Diff Diff
Updated with the review comments. (7.90 KB, patch) 2007-07-16 12:27 PDT, Hong Zhao	no flags	Details Formatted Diff Diff
Updated with the review comments. (9.96 KB, patch) 2007-07-16 12:52 PDT, Hong Zhao	Sachin.Padma: review-	Details Formatted Diff Diff
Pass the Browser control's special load observer to the correspondent default load handler in 3.1m. (9.85 KB, patch) 2007-07-16 13:20 PDT, Hong Zhao	Sachin.Padma: review-	Details Formatted Diff Diff
Update with the review comments. (9.98 KB, patch) 2007-07-18 09:45 PDT, Hong Zhao	no flags	Details Formatted Diff Diff
Update the patch with review comments. (9.91 KB, patch) 2007-07-18 09:53 PDT, Hong Zhao	no flags	Details Formatted Diff Diff
Update with review comments. (9.98 KB, patch) 2007-07-19 08:23 PDT, Hong Zhao	Sachin.Padma: review+	Details Formatted Diff Diff
Updated with review comments in 3.1m. (10.00 KB, patch) 2007-07-19 08:32 PDT, Hong Zhao	Sachin.Padma: review+	Details Formatted Diff Diff
Show Obsolete (6) View All Add attachment proposed patch, testcase, etc.

Hong Zhao

Comment 1 2007-07-13 20:45:55 PDT

Created attachment 15509 [details] Pass BrowserControl's specialLoadObserver to correspondent defaultContentHandler in 3.2.

Sachin Padma

Comment 2 2007-07-16 07:53:47 PDT

Comment on attachment 15509 [details] Pass BrowserControl's specialLoadObserver to correspondent defaultContentHandler in 3.2. No need to change implementation of SelfDownloadCOntentTypes. There is no need to have SpeicalLoadObserver as member of contentDispatcher.

Hong Zhao

Comment 3 2007-07-16 12:27:30 PDT

Created attachment 15536 [details] Updated with the review comments.

Hong Zhao

Comment 4 2007-07-16 12:52:39 PDT

Created attachment 15537 [details] Updated with the review comments.

Hong Zhao

Comment 5 2007-07-16 13:20:16 PDT

Created attachment 15538 [details] Pass the Browser control's special load observer to the correspondent default load handler in 3.1m.

Sachin Padma

Comment 6 2007-07-18 08:41:42 PDT

Comment on attachment 15537 [details] Updated with the review comments. SetSelfDownLoadCOntentTypes has a leave function when it is non leaving function

Hong Zhao

Comment 7 2007-07-18 09:45:31 PDT

Created attachment 15560 [details] Update with the review comments.

Hong Zhao

Comment 8 2007-07-18 09:53:48 PDT

Created attachment 15561 [details] Update the patch with review comments.

Hong Zhao

Comment 9 2007-07-19 08:23:23 PDT

Created attachment 15584 [details] Update with review comments.

Sachin Padma

Comment 10 2007-07-19 08:29:21 PDT

Comment on attachment 15584 [details] Update with review comments. r=me

Sachin Padma

Comment 11 2007-07-19 08:30:14 PDT

landed in ccb in r24445

Hong Zhao

Comment 12 2007-07-19 08:32:44 PDT

Created attachment 15585 [details] Updated with review comments in 3.1m.

Sachin Padma

Comment 13 2007-07-19 08:42:11 PDT

landed in 3.1m in r24446

Bradley Morrison

Comment 14 2008-04-09 11:39:40 PDT

Bulk closing of all s60 platform bugs. Sorry for the noise!

Joel Parks

Comment 15 2011-03-21 11:53:34 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.