Bug 14067 - [S60]: Browser Redirection issue with youtube.com director links
Summary: [S60]: Browser Redirection issue with youtube.com director links
Status: CLOSED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Plug-ins (show other bugs)
Version: 523.x (Safari 3)
Hardware: S60 Hardware S60 3rd edition
: P1 Critical
Assignee: Nobody
URL: http://www.youtube.com
Keywords:
Depends on:
Blocks:
 
Reported: 2007-06-11 05:58 PDT by Mahesh Kulkarni
Modified: 2011-03-21 11:53 PDT (History)
2 users (show)

See Also:


Attachments
Redirection problem. (1.18 KB, patch)
2007-06-11 06:10 PDT, Mahesh Kulkarni
Sachin.Padma: review-
Details | Formatted Diff | Diff
3.1 patch With change log. (1.81 KB, patch)
2007-06-18 06:23 PDT, Mahesh Kulkarni
Sachin.Padma: review-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mahesh Kulkarni 2007-06-11 05:58:57 PDT
Steps to reproduce:

1) open www.youtube.com
2) Click any of Director Videos on top left banner.
3) Browser loads same page. 

Expected behavior:
Browser loads that video and start playing. 

Actual behavior: 
Browser loads same page again. 

Other Information: 
TSW ID: PKII-73JE2R
Comment 1 Mahesh Kulkarni 2007-06-11 06:10:52 PDT
Created attachment 14932 [details]
Redirection problem.

_REASON:
Director link redirects to some other URI which is encoded and encoded URL is sent in the request. Youtube server couldn't understand the URL and same page is reloaded.  

_SOLUTION: 
Decoded the URL before making the request in resourceLoader when 303 response is received. 

_MODULE_EFFECTED:
resourceLoader of webKit. 

_TSW ERROR ID: PKII-73JE2R
Comment 2 Sachin Padma 2007-06-11 07:26:09 PDT
Comment on attachment 14932 [details]
Redirection problem.

Symbian's redirect filter should do the encoding and decoding for redirected urls. If there is an error in this, please submit a bug against Symbian's redirect filter
Comment 3 Mahesh Kulkarni 2007-06-12 06:37:56 PDT
We had discussed this bug as temporary fix as this bug is a showstopper for webvideo program. We knew that this was a symbian bug. But I couldn't confirm the same after some time of debugging. Shall we raise a bug to symbian with same? 
And as it is a showstopper can we add the change until we get symbian fix?

(In reply to comment #2)
> (From update of attachment 14932 [details] [edit])
> Symbian's redirect filter should do the encoding and decoding for redirected
> urls. If there is an error in this, please submit a bug against Symbian's
> redirect filter
> 

Comment 4 Mahesh Kulkarni 2007-06-18 06:23:49 PDT
Created attachment 15100 [details]
3.1 patch With change log. 

Can this be checked in as temporary change until symbian fix? As it is a show stopper TSW error.
Comment 5 Mahesh Kulkarni 2007-06-26 23:12:36 PDT
Comment on attachment 15100 [details]
3.1 patch With change log. 

This is 3.1 patch. Shall we raise a symbian bug for the same?
Comment 6 Sachin Padma 2007-06-27 06:59:31 PDT
Comment on attachment 15100 [details]
3.1 patch With change log. 

see my comments from before
Comment 7 Bradley Morrison 2008-04-09 11:31:48 PDT
Bulk update of s60 bugs - tagging and closing.
Comment 8 Bradley Morrison 2008-04-09 11:39:26 PDT
Bulk closing of all s60 platform bugs. 

Sorry for the noise!
Comment 9 Joel Parks 2011-03-21 11:53:31 PDT
re-purposing InTSW keyword for use by QtWebkit team