RESOLVED FIXED 144841
Use Ref instead of PassRefPtr in WebCore/page
https://bugs.webkit.org/show_bug.cgi?id=144841
Summary Use Ref instead of PassRefPtr in WebCore/page
Gyuyoung Kim
Reported 2015-05-09 20:58:46 PDT
To kill PassRefPtr, this patch replaces PassRefPtr with Ref or RefPtr in WebCore/page.
Attachments
Patch (53.86 KB, patch)
2015-05-09 21:00 PDT, Gyuyoung Kim
no flags
Patch (58.74 KB, patch)
2015-05-10 01:04 PDT, Gyuyoung Kim
no flags
Patch (59.84 KB, patch)
2015-05-10 02:23 PDT, Gyuyoung Kim
no flags
Patch for landing (61.97 KB, patch)
2015-05-10 18:33 PDT, Gyuyoung Kim
no flags
Patch for landing (61.97 KB, patch)
2015-05-10 19:25 PDT, Gyuyoung Kim
no flags
Gyuyoung Kim
Comment 1 2015-05-09 21:00:24 PDT
Gyuyoung Kim
Comment 2 2015-05-10 01:04:08 PDT
Gyuyoung Kim
Comment 3 2015-05-10 02:23:00 PDT
Darin Adler
Comment 4 2015-05-10 13:43:51 PDT
Comment on attachment 252808 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=252808&action=review OK to check in as long as you get iOS building. > Source/WebCore/page/TextIndicator.h:86 > WEBCORE_EXPORT static PassRefPtr<TextIndicator> createWithSelectionInFrame(Frame&, TextIndicatorPresentationTransition); > WEBCORE_EXPORT static PassRefPtr<TextIndicator> createWithRange(const Range&, TextIndicatorPresentationTransition); Why did these two end up untouched? > Source/WebCore/page/WebKitPoint.h:39 > + return adoptRef(*new WebKitPoint()); I like to omit the optional () in cases like this. > Source/WebCore/page/scrolling/ios/ScrollingTreeIOS.cpp:44 > -RefPtr<ScrollingTreeIOS> ScrollingTreeIOS::create(AsyncScrollingCoordinator* scrollingCoordinator) > +Ref<ScrollingTreeIOS> ScrollingTreeIOS::create(AsyncScrollingCoordinator* scrollingCoordinator) page/scrolling/ios/ScrollingTreeIOS.cpp:44:41: error: return type of out-of-line definition of 'WebCore::ScrollingTreeIOS::create' differs from that in the declaration Need to modify the header too.
Gyuyoung Kim
Comment 5 2015-05-10 18:33:12 PDT
Created attachment 252834 [details] Patch for landing
Gyuyoung Kim
Comment 6 2015-05-10 19:23:55 PDT
(In reply to comment #4) > Comment on attachment 252808 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=252808&action=review > > OK to check in as long as you get iOS building. Yes, fixed. > > Source/WebCore/page/TextIndicator.h:86 > > WEBCORE_EXPORT static PassRefPtr<TextIndicator> createWithSelectionInFrame(Frame&, TextIndicatorPresentationTransition); > > WEBCORE_EXPORT static PassRefPtr<TextIndicator> createWithRange(const Range&, TextIndicatorPresentationTransition); > > Why did these two end up untouched? Oh, I missed. Fixed too. > > Source/WebCore/page/WebKitPoint.h:39 > > + return adoptRef(*new WebKitPoint()); > > I like to omit the optional () in cases like this. > > > Source/WebCore/page/scrolling/ios/ScrollingTreeIOS.cpp:44 > > -RefPtr<ScrollingTreeIOS> ScrollingTreeIOS::create(AsyncScrollingCoordinator* scrollingCoordinator) > > +Ref<ScrollingTreeIOS> ScrollingTreeIOS::create(AsyncScrollingCoordinator* scrollingCoordinator) > > page/scrolling/ios/ScrollingTreeIOS.cpp:44:41: error: return type of > out-of-line definition of 'WebCore::ScrollingTreeIOS::create' differs from > that in the declaration > > Need to modify the header too. ditto.
Gyuyoung Kim
Comment 7 2015-05-10 19:25:51 PDT
Created attachment 252835 [details] Patch for landing
WebKit Commit Bot
Comment 8 2015-05-10 22:54:46 PDT
Comment on attachment 252835 [details] Patch for landing Clearing flags on attachment: 252835 Committed r184066: <http://trac.webkit.org/changeset/184066>
WebKit Commit Bot
Comment 9 2015-05-10 22:54:51 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.