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

Side by Side Diff: tests/PictureTest.cpp

Issue 2163283002: re-disable save/restore peephole optimization (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: I hate un-called-function-warning Created 4 years, 5 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
« no previous file with comments | « src/core/SkRecordOpts.cpp ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2012 Google Inc. 2 * Copyright 2012 Google Inc.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license that can be 4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file. 5 * found in the LICENSE file.
6 */ 6 */
7 7
8 #include "SkBigPicture.h" 8 #include "SkBigPicture.h"
9 #include "SkBBoxHierarchy.h" 9 #include "SkBBoxHierarchy.h"
10 #include "SkBlurImageFilter.h" 10 #include "SkBlurImageFilter.h"
(...skipping 1228 matching lines...) Expand 10 before | Expand all | Expand 10 after
1239 for (int i = 0; i < 50; ++i) { 1239 for (int i = 0; i < 50; ++i) {
1240 analyzer.analyzeClipPath(concaveClip, SkRegion::kIntersect_Op, true); 1240 analyzer.analyzeClipPath(concaveClip, SkRegion::kIntersect_Op, true);
1241 } 1241 }
1242 REPORTER_ASSERT(r, !analyzer.suitableForGpuRasterization()); 1242 REPORTER_ASSERT(r, !analyzer.suitableForGpuRasterization());
1243 } 1243 }
1244 1244
1245 #endif // SK_SUPPORT_GPU 1245 #endif // SK_SUPPORT_GPU
1246 1246
1247 //////////////////////////////////////////////////////////////////////////////// /////////////////// 1247 //////////////////////////////////////////////////////////////////////////////// ///////////////////
1248 1248
1249 // Disable until we properly fix https://bugs.chromium.org/p/skia/issues/detail? id=5548
1250 #if 0
1249 static void empty_ops(SkCanvas* canvas) { 1251 static void empty_ops(SkCanvas* canvas) {
1250 } 1252 }
1251 static void clip_ops(SkCanvas* canvas) { 1253 static void clip_ops(SkCanvas* canvas) {
1252 canvas->save(); 1254 canvas->save();
1253 canvas->clipRect(SkRect::MakeWH(20, 20)); 1255 canvas->clipRect(SkRect::MakeWH(20, 20));
1254 canvas->restore(); 1256 canvas->restore();
1255 } 1257 }
1256 static void matrix_ops(SkCanvas* canvas) { 1258 static void matrix_ops(SkCanvas* canvas) {
1257 canvas->save(); 1259 canvas->save();
1258 canvas->scale(2, 3); 1260 canvas->scale(2, 3);
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
1296 } 1298 }
1297 { 1299 {
1298 SkPictureRecorder rec; 1300 SkPictureRecorder rec;
1299 proc(rec.beginRecording(cull)); 1301 proc(rec.beginRecording(cull));
1300 sk_sp<SkDrawable> dr = rec.finishRecordingAsDrawable( 1302 sk_sp<SkDrawable> dr = rec.finishRecordingAsDrawable(
1301 SkPictureRecorder::kReturnNullF orEmpty_FinishFlag); 1303 SkPictureRecorder::kReturnNullF orEmpty_FinishFlag);
1302 REPORTER_ASSERT(r, !dr.get()); 1304 REPORTER_ASSERT(r, !dr.get());
1303 } 1305 }
1304 } 1306 }
1305 } 1307 }
1308 #endif
1309
OLDNEW
« no previous file with comments | « src/core/SkRecordOpts.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698