Bug 141901

Summary: Remove unnecessary PlatformTimeRanges::create()
Product: WebKit Reporter: Gyuyoung Kim <gyuyoung.kim>
Component: WebCore Misc.Assignee: Gyuyoung Kim <gyuyoung.kim>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, commit-queue, darin
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Gyuyoung Kim
Reported 2015-02-23 02:07:06 PST
As mentioned in https://bugs.webkit.org/show_bug.cgi?id=139019#c12, PlatformTimeRanges::create() functions have just created an instance of PlatformTimeRanges. Even PlatformTimeRanges supports create() factory function as well as public constructor. This functionality can be replaced with std::make_unique<> use directly.
Attachments
Patch (15.88 KB, patch)
2015-02-23 02:08 PST, Gyuyoung Kim 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Gyuyoung Kim
Comment 1 2015-02-23 02:08:35 PST
Created attachment 247108 [details] Patch
WebKit Commit Bot
Comment 2 2015-02-23 16:25:05 PST
Comment on attachment 247108 [details] Patch Clearing flags on attachment: 247108 Committed r180532: <http://trac.webkit.org/changeset/180532>
WebKit Commit Bot
Comment 3 2015-02-23 16:25:09 PST
All reviewed patches have been landed. Closing bug.
Alex Christensen
Comment 4 2015-02-23 18:55:16 PST
Fixed typo in http://trac.webkit.org/changeset/180541
Note You need to log in before you can comment on or make changes to this bug.