Bug 96268

Summary: [chromium] MediaStream API: Remove the Descriptor postfix
Product: WebKit Reporter: Tommy Widenflycht <tommyw>
Component: WebKit Misc.Assignee: Tommy Widenflycht <tommyw>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, dglazkov, eric.carlson, feature-media-reviews, fishd, jamesr, tkent+wkapi, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 80589    
Attachments:
Description Flags
Patch none

Tommy Widenflycht
Reported 2012-09-10 06:15:48 PDT
After consideration I realized that there is no need whatsoever in using the Descriptor prefix in the WekKit embedder API. Remove from WebRTCSessionDescription and WebRTCICECandidate to start with.
Attachments
Patch (51.05 KB, patch)
2012-09-10 06:21 PDT, Tommy Widenflycht 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Tommy Widenflycht
Comment 1 2012-09-10 06:21:37 PDT
Created attachment 163111 [details] Patch
Tommy Widenflycht
Comment 2 2012-09-10 06:22:19 PDT
No code changes except for the two renames.
WebKit Review Bot
Comment 3 2012-09-10 06:24:51 PDT
Please wait for approval from abarth@webkit.org, dglazkov@chromium.org, fishd@chromium.org, jamesr@chromium.org or tkent@chromium.org before submitting, as this patch contains changes to the Chromium public API. See also https://trac.webkit.org/wiki/ChromiumWebKitAPI.
Adam Barth
Comment 4 2012-09-10 08:41:19 PDT
Comment on attachment 163111 [details] Patch Thanks!
WebKit Review Bot
Comment 5 2012-09-10 08:50:27 PDT
Comment on attachment 163111 [details] Patch Clearing flags on attachment: 163111 Committed r128064: <http://trac.webkit.org/changeset/128064>
WebKit Review Bot
Comment 6 2012-09-10 08:50:30 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.