Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(54)

Side by Side Diff: third_party/WebKit/Source/platform/graphics/PicturePattern.cpp

Issue 2290903002: Change (Pass)RefPtr<SkXxx> into sk_sp<SkXxx>. (Closed)
Patch Set: Rebasing... Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "platform/graphics/PicturePattern.h" 5 #include "platform/graphics/PicturePattern.h"
6 6
7 #include "platform/graphics/skia/SkiaUtils.h" 7 #include "platform/graphics/skia/SkiaUtils.h"
8 #include "third_party/skia/include/core/SkPicture.h" 8 #include "third_party/skia/include/core/SkPicture.h"
9 #include "third_party/skia/include/core/SkShader.h" 9 #include "third_party/skia/include/core/SkShader.h"
10 10
11 namespace blink { 11 namespace blink {
12 12
13 PassRefPtr<PicturePattern> PicturePattern::create(PassRefPtr<SkPicture> picture, 13 PassRefPtr<PicturePattern> PicturePattern::create(sk_sp<SkPicture> picture,
14 RepeatMode repeatMode) 14 RepeatMode repeatMode)
15 { 15 {
16 return adoptRef(new PicturePattern(picture, repeatMode)); 16 return adoptRef(new PicturePattern(std::move(picture), repeatMode));
17 } 17 }
18 18
19 PicturePattern::PicturePattern(PassRefPtr<SkPicture> picture, RepeatMode mode) 19 PicturePattern::PicturePattern(sk_sp<SkPicture> picture, RepeatMode mode)
20 : Pattern(mode) 20 : Pattern(mode)
21 , m_tilePicture(toSkSp(picture)) 21 , m_tilePicture(std::move(picture))
22 { 22 {
23 // All current clients use RepeatModeXY, so we only support this mode for no w. 23 // All current clients use RepeatModeXY, so we only support this mode for no w.
24 ASSERT(isRepeatXY()); 24 ASSERT(isRepeatXY());
25 25
26 // FIXME: we don't have a good way to account for DL memory utilization. 26 // FIXME: we don't have a good way to account for DL memory utilization.
27 } 27 }
28 28
29 PicturePattern::~PicturePattern() 29 PicturePattern::~PicturePattern()
30 { 30 {
31 } 31 }
32 32
33 sk_sp<SkShader> PicturePattern::createShader(const SkMatrix& localMatrix) 33 sk_sp<SkShader> PicturePattern::createShader(const SkMatrix& localMatrix)
34 { 34 {
35 SkRect tileBounds = m_tilePicture->cullRect(); 35 SkRect tileBounds = m_tilePicture->cullRect();
36 36
37 return SkShader::MakePictureShader(m_tilePicture, 37 return SkShader::MakePictureShader(m_tilePicture,
38 SkShader::kRepeat_TileMode, SkShader::kRepeat_TileMode, &localMatrix, &t ileBounds); 38 SkShader::kRepeat_TileMode, SkShader::kRepeat_TileMode, &localMatrix, &t ileBounds);
39 } 39 }
40 40
41 } // namespace blink 41 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698