Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

web: fix Paragraph.getBoxesForRange for zero-length ranges#13483

Merged
yjbanov merged 1 commit intoflutter:masterfrom
yjbanov:fix-zero-lengh-range
Nov 4, 2019
Merged

web: fix Paragraph.getBoxesForRange for zero-length ranges#13483
yjbanov merged 1 commit intoflutter:masterfrom
yjbanov:fix-zero-lengh-range

Conversation

@yjbanov
Copy link
Contributor

@yjbanov yjbanov commented Nov 1, 2019

This will fix text_test.dart over on the framework side.

Copy link
Contributor

@mdebbar mdebbar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks for fixing this!

@yjbanov
Copy link
Contributor Author

yjbanov commented Nov 4, 2019

Landing on red. LUCI is red on Fuchsia infra breakage.

@yjbanov yjbanov merged commit 6e0e227 into flutter:master Nov 4, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 5, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 5, 2019
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Nov 5, 2019
[email protected]:flutter/engine.git/compare/0a8bd9dd6fbb...e73c9c8

git log 0a8bd9d..e73c9c8 --no-merges --oneline
2019-11-04 [email protected] Roll src/third_party/skia 8e083eee8ece..4cc2dc64ff13 (1 commits) (flutter/engine#13543)
2019-11-04 [email protected] Roll fuchsia/sdk/core/mac-amd64 from ggsQ5... to XcaoM... (flutter/engine#13535)
2019-11-04 [email protected] fix getBoxesForRange for zero-length ranges (flutter/engine#13483)
2019-11-04 [email protected] Roll src/third_party/dart bbe2ac28c9..ab5cf0f854 (73 commits) (flutter/engine#13614)
2019-11-02 [email protected] Roll src/third_party/skia 283ec65f632a..8e083eee8ece (32 commits) (flutter/engine#13489)
2019-11-02 [email protected] Roll fuchsia/sdk/core/mac-amd64 from FKqQ_... to ggsQ5... (flutter/engine#13490)
2019-11-02 [email protected] Release shim bindings when detaching (flutter/engine#13432)
2019-11-01 [email protected] Add fuchsia.intl.PropertyProvider to our services on Fuchsia (flutter/engine#13486)
2019-11-01 [email protected] [dart] Makes the intl services available (flutter/engine#13460)
2019-11-01 [email protected] Roll fuchsia/sdk/core/linux-amd64 from 866GG... to dhwMR... (flutter/engine#13475)
2019-11-01 [email protected] Roll fuchsia/sdk/core/mac-amd64 from JngMB... to FKqQ_... (flutter/engine#13477)
2019-11-01 [email protected] Roll src/third_party/skia 809ec77893be..283ec65f632a (14 commits) (flutter/engine#13472)
2019-11-01 [email protected] Request a reattach when creating the text input plugin on Android (flutter/engine#13474)


If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC [email protected] on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
Inconnu08 pushed a commit to Inconnu08/flutter that referenced this pull request Nov 26, 2019
[email protected]:flutter/engine.git/compare/0a8bd9dd6fbb...e73c9c8

git log 0a8bd9d..e73c9c8 --no-merges --oneline
2019-11-04 [email protected] Roll src/third_party/skia 8e083eee8ece..4cc2dc64ff13 (1 commits) (flutter/engine#13543)
2019-11-04 [email protected] Roll fuchsia/sdk/core/mac-amd64 from ggsQ5... to XcaoM... (flutter/engine#13535)
2019-11-04 [email protected] fix getBoxesForRange for zero-length ranges (flutter/engine#13483)
2019-11-04 [email protected] Roll src/third_party/dart bbe2ac28c9..ab5cf0f854 (73 commits) (flutter/engine#13614)
2019-11-02 [email protected] Roll src/third_party/skia 283ec65f632a..8e083eee8ece (32 commits) (flutter/engine#13489)
2019-11-02 [email protected] Roll fuchsia/sdk/core/mac-amd64 from FKqQ_... to ggsQ5... (flutter/engine#13490)
2019-11-02 [email protected] Release shim bindings when detaching (flutter/engine#13432)
2019-11-01 [email protected] Add fuchsia.intl.PropertyProvider to our services on Fuchsia (flutter/engine#13486)
2019-11-01 [email protected] [dart] Makes the intl services available (flutter/engine#13460)
2019-11-01 [email protected] Roll fuchsia/sdk/core/linux-amd64 from 866GG... to dhwMR... (flutter/engine#13475)
2019-11-01 [email protected] Roll fuchsia/sdk/core/mac-amd64 from JngMB... to FKqQ_... (flutter/engine#13477)
2019-11-01 [email protected] Roll src/third_party/skia 809ec77893be..283ec65f632a (14 commits) (flutter/engine#13472)
2019-11-01 [email protected] Request a reattach when creating the text input plugin on Android (flutter/engine#13474)


If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC [email protected] on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
@yjbanov yjbanov deleted the fix-zero-lengh-range branch June 22, 2021 21:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants