Skip to content

Commit

Permalink
[flutter_releases] Cherry pick flutter-2.13-candidate.7 changes (#10… (
Browse files Browse the repository at this point in the history
…#103374)

* [flutter_releases] Cherry pick flutter-2.13-candidate.7 changes  (#103206)

* Revert "Fix jumping when doing long press for selecting text (#102270)" (#103142)

This reverts commit 6ea4aef.

* Roll engine with Cps

* Fix engine hash

* [flutter_releases] partial revert of repaint boundary change (#102962) (#103214)

Co-authored-by: Jonah Williams <jonahwilliams@google.com>

* [flutter_releases] remove assert on markNeedsCompositingBitsUpdate #103227 (#103225)

Co-authored-by: Casey Hillers <chillers@google.com>
Co-authored-by: Jonah Williams <jonahwilliams@google.com>
  • Loading branch information
3 people committed May 9, 2022
1 parent fba9214 commit 93a41bc
Show file tree
Hide file tree
Showing 6 changed files with 17 additions and 341 deletions.
2 changes: 1 addition & 1 deletion bin/internal/engine.version
Original file line number Diff line number Diff line change
@@ -1 +1 @@
07584c64bb7ba8d9e79214c3667cdb16cfa67923
74ac1de5d4f70d6db6cc7f99f6f259a7d073c3ab
6 changes: 0 additions & 6 deletions packages/flutter/lib/src/rendering/object.dart
Original file line number Diff line number Diff line change
Expand Up @@ -2295,12 +2295,6 @@ abstract class RenderObject extends AbstractNode with DiagnosticableTreeMixin im
return;
}
}
assert(() {
final AbstractNode? parent = this.parent;
if (parent is RenderObject)
return parent._needsCompositing;
return true;
}());
// parent is fine (or there isn't one), but we are dirty
if (owner != null)
owner!._nodesNeedingCompositingBitsUpdate.add(this);
Expand Down
23 changes: 16 additions & 7 deletions packages/flutter/lib/src/rendering/proxy_box.dart
Original file line number Diff line number Diff line change
Expand Up @@ -843,7 +843,7 @@ class RenderOpacity extends RenderProxyBox {
super(child);

@override
bool get isRepaintBoundary => child != null && (_alpha > 0);
bool get alwaysNeedsCompositing => child != null && (_alpha > 0);

@override
OffsetLayer updateCompositedLayer({required covariant OpacityLayer? oldLayer}) {
Expand Down Expand Up @@ -871,13 +871,13 @@ class RenderOpacity extends RenderProxyBox {
assert(value >= 0.0 && value <= 1.0);
if (_opacity == value)
return;
final bool wasRepaintBoundary = isRepaintBoundary;
final bool didNeedCompositing = alwaysNeedsCompositing;
final bool wasVisible = _alpha != 0;
_opacity = value;
_alpha = ui.Color.getAlphaFromOpacity(_opacity);
if (wasRepaintBoundary != isRepaintBoundary)
if (didNeedCompositing != alwaysNeedsCompositing)
markNeedsCompositingBitsUpdate();
markNeedsCompositedLayerUpdate();
markNeedsPaint();
if (wasVisible != (_alpha != 0) && !alwaysIncludeSemantics)
markNeedsSemanticsUpdate();
}
Expand All @@ -898,10 +898,19 @@ class RenderOpacity extends RenderProxyBox {

@override
void paint(PaintingContext context, Offset offset) {
if (_alpha == 0) {
return;
if (child != null) {
if (_alpha == 0) {
// No need to keep the layer. We'll create a new one if necessary.
layer = null;
return;
}
assert(needsCompositing);
layer = context.pushOpacity(offset, _alpha, super.paint, oldLayer: layer as OpacityLayer?);
assert(() {
layer!.debugCreator = debugCreator;
return true;
}());
}
super.paint(context, offset);
}

@override
Expand Down
5 changes: 0 additions & 5 deletions packages/flutter/lib/src/widgets/editable_text.dart
Original file line number Diff line number Diff line change
Expand Up @@ -2514,11 +2514,6 @@ class EditableTextState extends State<EditableText> with AutomaticKeepAliveClien
_showCaretOnScreenScheduled = true;
SchedulerBinding.instance.addPostFrameCallback((Duration _) {
_showCaretOnScreenScheduled = false;

// if cursor is inactive, e.g. while selecting text, do not jump away
if (!_cursorActive) {
return;
}
if (_currentCaretRect == null || !_scrollController.hasClients) {
return;
}
Expand Down
47 changes: 0 additions & 47 deletions packages/flutter/test/widgets/editable_text_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -5183,53 +5183,6 @@ void main() {
// toolbar. Until we change that, this test should remain skipped.
}, skip: kIsWeb); // [intended]


testWidgets('text selection handle visibility for long text', (WidgetTester tester) async {
// long text which is scrollable based on given box size
const String testText =
'Lorem ipsum dolor sit amet, consetetur sadipscing elitr, sed diam nonumy eirmod tempor invidunt ut labore et dolore magna aliquyam erat, sed diam voluptua. At vero eos et accusam et justo duo dolores et ea rebum. Stet clita kasd gubergren, no sea takimata sanctus est Lorem ipsum dolor sit amet.';
final TextEditingController controller =
TextEditingController(text: testText);
final ScrollController scrollController = ScrollController();

await tester.pumpWidget(MaterialApp(
home: Align(
alignment: Alignment.topLeft,
child: SizedBox(
width: 100,
height: 100,
child: SingleChildScrollView(
controller: scrollController,
child: EditableText(
controller: controller,
showSelectionHandles: true,
focusNode: FocusNode(),
style: Typography.material2018().black.subtitle1!,
cursorColor: Colors.blue,
backgroundCursorColor: Colors.grey,
selectionControls: materialTextSelectionControls,
keyboardType: TextInputType.multiline,
maxLines: null,
),
),
),
),
));

// scroll to a text that is outside of the inital visible rect
scrollController.jumpTo(151);
await tester.pump();

// long press on a word to trigger a select
await tester.longPressAt(const Offset(20, 15));
// wait for adjustments of scroll area
await tester.pump();
await tester.pumpAndSettle();

// assert not jumped to top
expect(scrollController.offset, equals(151));
});

const String testText = 'Now is the time for\n' // 20
'all good people\n' // 20 + 16 => 36
'to come to the aid\n' // 36 + 19 => 55
Expand Down
Loading

0 comments on commit 93a41bc

Please sign in to comment.