Bug 117188

Summary: [BLACKBERRY]RSS not working with browser
Product: WebKit Reporter: minggang wang <mingwang>
Component: WebKit BlackBerryAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, rwlbuis
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
path
mingwang: commit-queue-
new patch
mingwang: review-
correct patch rwlbuis: review+, commit-queue: commit-queue-

Description minggang wang 2013-06-04 03:43:02 PDT
Internal PR 341330 
Internal reviewed by Joe Mason
Browser won't recognize the RSS content, when the "Content-Type" in http header is wrong (not "text/xml").
Comment 1 minggang wang 2013-06-04 03:56:21 PDT
Created attachment 203675 [details]
path
Comment 2 minggang wang 2013-06-04 04:09:52 PDT
Created attachment 203678 [details]
new patch
Comment 3 minggang wang 2013-06-04 04:17:58 PDT
Created attachment 203679 [details]
correct patch
Comment 4 minggang wang 2013-06-05 01:22:12 PDT
Comment on attachment 203678 [details]
new patch

wrong patch
Comment 5 minggang wang 2013-06-05 01:35:28 PDT
Comment on attachment 203675 [details]
path

wrong patch
Comment 6 minggang wang 2013-06-05 01:36:35 PDT
Comment on attachment 203679 [details]
correct patch

pls review this patch
Comment 7 Rob Buis 2013-06-13 10:11:39 PDT
Comment on attachment 203679 [details]
correct patch

Looks good.
Comment 8 WebKit Commit Bot 2013-06-13 10:12:49 PDT
Comment on attachment 203679 [details]
correct patch

Rejecting attachment 203679 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-01', 'validate-changelog', '--check-oops', '--non-interactive', 203679, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

/Volumes/Data/EWS/WebKit/Source/WebCore/ChangeLog neither lists a valid reviewer nor contains the string "Unreviewed" or "Rubber stamp" (case insensitive).

Full output: http://webkit-queues.appspot.com/results/896009
Comment 9 Rob Buis 2013-07-30 07:40:23 PDT
(In reply to comment #6)
> (From update of attachment 203679 [details])
> pls review this patch

Sorry for forgetting about this one. Mingwang if you still want this in you have to fix the Changelog. If it is not needed you can close the bug.