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

Re-land Adding Link Semantics #12972

Merged
merged 14 commits into from
Oct 8, 2019
Merged

Re-land Adding Link Semantics #12972

merged 14 commits into from
Oct 8, 2019

Conversation

Piinks
Copy link
Contributor

@Piinks Piinks commented Oct 7, 2019

Re-lands #12453 which was reverted in #12815. The change was reverted due to the framework tree status preventing a manual engine roll.

Framework counter PR: flutter/flutter#41327 - will require a manual engine roll.

Part of flutter/flutter#36306

@Piinks
Copy link
Contributor Author

Piinks commented Oct 8, 2019

Landing on red to do a manual engine roll in flutter/flutter#41327

@Piinks Piinks merged commit 58d7b84 into flutter:master Oct 8, 2019
@Piinks Piinks deleted the relandLinkA11y branch October 8, 2019 18:28
@Piinks Piinks mentioned this pull request Oct 8, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 8, 2019
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Oct 8, 2019
git@github.com:flutter/engine.git/compare/5b3182294f63...c635d70

git log 5b31822..c635d70 --no-merges --oneline
2019-10-08 skia-flutter-autoroll@skia.org Roll src/third_party/skia 902cf1e12a74..1494a7f1ec03 (10 commits) (flutter/engine#13000)
2019-10-08 liyuqian@google.com SkSL precompile (flutter/engine#12412)
2019-10-08 chinmaygarde@google.com Compile sanitizer suppressions list and file bugs as necessary. (flutter/engine#12991)
2019-10-08 chinmaygarde@google.com Add a unit-test to verify that root surface transformation affect platform view coordinates. (flutter/engine#12783)
2019-10-08 katelovett@google.com Re-land Adding Link Semantics (flutter/engine#12972)


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 chinmaygarde@google.com 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
@tvolkert
Copy link
Contributor

One takeaway from this PR is that modifying code in dart:ui to refer to constants or enum values that were recently added (and thus haven't rolled through to Fuchsia) severely backs up the various rolls. And thus, we need better detection of breaking changes so that we issue soft breakages. In cases such as these, we should add the constant in one PR, wait for that to roll through, then add the code that uses the new constant later.

/cc @Hixie

Inconnu08 pushed a commit to Inconnu08/flutter that referenced this pull request Nov 26, 2019
git@github.com:flutter/engine.git/compare/5b3182294f63...c635d70

git log 5b31822..c635d70 --no-merges --oneline
2019-10-08 skia-flutter-autoroll@skia.org Roll src/third_party/skia 902cf1e12a74..1494a7f1ec03 (10 commits) (flutter/engine#13000)
2019-10-08 liyuqian@google.com SkSL precompile (flutter/engine#12412)
2019-10-08 chinmaygarde@google.com Compile sanitizer suppressions list and file bugs as necessary. (flutter/engine#12991)
2019-10-08 chinmaygarde@google.com Add a unit-test to verify that root surface transformation affect platform view coordinates. (flutter/engine#12783)
2019-10-08 katelovett@google.com Re-land Adding Link Semantics (flutter/engine#12972)


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 chinmaygarde@google.com 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
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants