Skip to content

Commit

Permalink
Update shader warm-up for recent Skia changes (#37955)
Browse files Browse the repository at this point in the history
The update is copied from an update we made to a Google-internal
client: cl/260202900

The update will save 1 shader compilation.

This should help solve our regression:
#31203

More regressions on iOS might be introduced later by
flutter/engine#9813 (comment)

Unfortunately, we didn't rebase our benchmarks so such regressions
were not detected.

Hence to fully solve #31203,
we might need to revert some change in
flutter/engine#9813 to make iOS shader warm-up
happen on the GPU thread again.
  • Loading branch information
liyuqian committed Aug 15, 2019
1 parent a24bfed commit 77aa495
Show file tree
Hide file tree
Showing 2 changed files with 53 additions and 0 deletions.
17 changes: 17 additions & 0 deletions packages/flutter/lib/src/painting/shader_warm_up.dart
Expand Up @@ -192,5 +192,22 @@ class DefaultShaderWarmUp extends ShaderWarmUp {
final ui.Paragraph paragraph = paragraphBuilder.build()
..layout(const ui.ParagraphConstraints(width: 60.0));
canvas.drawParagraph(paragraph, const ui.Offset(20.0, 20.0));

// Draw a rect inside a rrect with a non-trivial intersection. If the
// intersection is trivial (e.g., equals the rrect clip), Skia will optimize
// the clip out.
//
// Add an integral or fractional translation to trigger Skia's non-AA or AA
// optimizations (as did before in normal FillRectOp in rrect clip cases).
for (double fraction in <double>[0.0, 0.5]) {
canvas
..save()
..translate(fraction, fraction)
..clipRRect(ui.RRect.fromLTRBR(8, 8, 328, 248, const ui.Radius.circular(16)))
..drawRect(const ui.Rect.fromLTRB(10, 10, 320, 240), ui.Paint())
..restore();
canvas.translate(drawCallSpacing, 0.0);
}
canvas.translate(0.0, drawCallSpacing);
}
}
36 changes: 36 additions & 0 deletions packages/flutter/test/painting/shader_warm_up_test.dart
@@ -0,0 +1,36 @@
// Copyright 2019 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

import 'package:flutter/painting.dart';
import 'package:flutter_test/flutter_test.dart';

class TestCanvas implements Canvas {
TestCanvas([this.invocations]);

final List<Invocation> invocations;

@override
void noSuchMethod(Invocation invocation) {
invocations?.add(invocation);
}
}

void main() {
test('DefaultShaderWarmUp has expected canvas invocations', () {
final List<Invocation> invocations = <Invocation>[];
final TestCanvas canvas = TestCanvas(invocations);
const DefaultShaderWarmUp s = DefaultShaderWarmUp();
s.warmUpOnCanvas(canvas);

bool hasDrawRectAfterClipRRect = false;
for (int i = 0; i < invocations.length - 1; i += 1) {
if (invocations[i].memberName == #clipRRect && invocations[i + 1].memberName == #drawRect) {
hasDrawRectAfterClipRRect = true;
break;
}
}

expect(hasDrawRectAfterClipRRect, true);
});
}

0 comments on commit 77aa495

Please sign in to comment.