25772 Commits

Author SHA1 Message Date
Chinmay Garde
437c939632 [Impeller] Delete the archivist framework. (flutter/engine#50474)
It is unused and can be brought back from Git if needed. This unnecessarily adds to binary size estimates on standalone Impeller investigations.
2024-02-08 22:49:00 +00:00
skia-flutter-autoroll
dcf2e52abf Roll Fuchsia Linux SDK from CPiPDUwCrjHYf_lAR... to RK2irywND2DKT4l6C... (flutter/engine#50473)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/fuchsia-linux-sdk-flutter-engine
Please CC chinmaygarde@google.com,rmistry@google.com,zra@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
2024-02-08 22:08:06 +00:00
dependabot[bot]
27adeb7752 Bump actions/upload-artifact from 4.3.0 to 4.3.1 (flutter/engine#50388)
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4.3.0 to 4.3.1.
<details>
<summary>Release notes</summary>
<p><em>Sourced from <a href="https://github.com/actions/upload-artifact/releases">actions/upload-artifact's releases</a>.</em></p>
<blockquote>
<h2>v4.3.1</h2>
<ul>
<li>Bump <code>@​actions/artifacts</code> to latest version to include <a href="https://redirect.github.com/actions/toolkit/pull/1648">updated GHES host check</a></li>
</ul>
</blockquote>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a href="5d5d22a312"><code>5d5d22a</code></a> Merge pull request <a href="https://redirect.github.com/actions/upload-artifact/issues/515">#515</a> from actions/eggyhead/update-artifact-v2.1.1</li>
<li><a href="f1e993d966"><code>f1e993d</code></a> update artifact license</li>
<li><a href="4881bfd3f2"><code>4881bfd</code></a> updating dist:</li>
<li><a href="a30777e265"><code>a30777e</code></a> <a href="https://github.com/eggyhead"><code>@​eggyhead</code></a></li>
<li><a href="3a8048248f"><code>3a80482</code></a> Merge pull request <a href="https://redirect.github.com/actions/upload-artifact/issues/511">#511</a> from actions/robherley/migration-docs-typo</li>
<li><a href="9d63e3f2f8"><code>9d63e3f</code></a> Merge branch 'main' into robherley/migration-docs-typo</li>
<li><a href="dfa1ab292d"><code>dfa1ab2</code></a> fix typo with v3 artifact downloads in migration guide</li>
<li><a href="d00351bf69"><code>d00351b</code></a> Merge pull request <a href="https://redirect.github.com/actions/upload-artifact/issues/509">#509</a> from markmssd/patch-1</li>
<li><a href="707f5a7b71"><code>707f5a7</code></a> Update limitation of <code>10</code> artifacts upload to <code>500</code></li>
<li>See full diff in <a href="26f96dfa69...5d5d22a312">compare view</a></li>
</ul>
</details>
<br />

[![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=actions/upload-artifact&package-manager=github_actions&previous-version=4.3.0&new-version=4.3.1)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`.

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
- `@dependabot show <dependency name> ignore conditions` will show all of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

</details>
2024-02-08 21:45:05 +00:00
Chinmay Garde
a506103288 Sever the //flutter/vulkan dependency in Flutter tester. (flutter/engine#50454)
The previous code was setting up a proc table, then getting the address of the proc that was used to the setup that table, then setting up another proc table. Directly setup the final proc table and don't depend on //impeller/vulkan.

Part of https://github.com/flutter/flutter/issues/143127
2024-02-08 21:37:38 +00:00
gaaclarke
3296aee197 Updated CONTRIBUTING.md to have a testing section (flutter/engine#50469)
fixes https://github.com/flutter/flutter/issues/143087

## Pre-launch Checklist

- [x] I read the [Contributor Guide] and followed the process outlined
there for submitting PRs.
- [x] I read the [Tree Hygiene] wiki page, which explains my
responsibilities.
- [x] I read and followed the [Flutter Style Guide] and the [C++,
Objective-C, Java style guides].
- [x] I listed at least one issue that this PR fixes in the description
above.
- [x] I added new tests to check the change I am making or feature I am
adding, or the PR is [test-exempt]. See [testing the engine] for
instructions on writing and running engine tests.
- [x] I updated/added relevant documentation (doc comments with `///`).
- [x] I signed the [CLA].
- [x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel
on [Discord].

<!-- Links -->
[Contributor Guide]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#overview
[Tree Hygiene]: https://github.com/flutter/flutter/wiki/Tree-hygiene
[test-exempt]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#tests
[Flutter Style Guide]:
https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo
[C++, Objective-C, Java style guides]:
https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
[testing the engine]:
https://github.com/flutter/flutter/wiki/Testing-the-engine
[CLA]: https://cla.developers.google.com/
[flutter/tests]: https://github.com/flutter/tests
[breaking change policy]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#handling-breaking-changes
[Discord]: https://github.com/flutter/flutter/wiki/Chat
2024-02-08 13:18:23 -08:00
Rulong Chen(陈汝龙)
ae1b5bbb13 Fix Core Foundation object leaks. (flutter/engine#50460)
This fixes Core Foundation object leaks.

----
Unrelated, just as exciting: Today is the last day before the Chinese New Year, and I wish everyone a Happy New Year of the Dragon, Xin Nian Kuai Le! :)

[C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
2024-02-08 21:18:11 +00:00
Hassan Toor
650f4b9728 [web] - Fix inputmode on Android Firefox (flutter/engine#46901)
The wrong keyboard type shows when users specify the phone input type in the framework. 

This is happening because:

* We usually set `inputmode` for mobile text editing strategies.
* We only use Android's text editing strategy for Blink browser engines (not firefox), so we fall through to the Firefox text editing strategy which I believe should only be used for desktop firefox.
* So we're ending up in a case where Android + Firefox doesn't set the inputmode and it pops open the default alphanumeric keyboard

Fixes https://github.com/flutter/flutter/issues/136351
2024-02-08 20:10:48 +00:00
Brandon DeRosier
4eae677520 [Impeller] Vulkan: Don't fail initialization if stencil-only textures aren't supported. (flutter/engine#50455)
Closes https://github.com/flutter/flutter/issues/137108

We don't rely on stencil-only textures anymore in the 2D renderer.
2024-02-08 17:44:52 +00:00
Jonah Williams
f97711784b [Impeller] remove extra buffer usage in compute. (flutter/engine#50415)
Previously we were not able to effectively reserve host buffer regions, as these would map to host only memory and later be converted to device memory. Now that the host buffer is always device memory, we don't need to create compute specific buffers for the point_field_geometry.
2024-02-08 17:22:40 +00:00
gaaclarke
9fef46c3c8 [Impeller] blur: drop eighth downsample barrier (flutter/engine#50405)
issue: https://github.com/flutter/flutter/issues/142990

I pushed this value as low as I dared to go and ended up with a kernel
size of 41 (radius 20). This means the range of sigma ~60 to ~70 will
now be 1/16th scalar instead of 1/8th.

Here is the new downsample rate versus sigma on a 2x display.

## before
<img width="903" alt="Screenshot 2024-02-06 at 10 28 58 AM"
src="https://github.com/flutter/engine/assets/30870216/f94896d3-fe1c-40e5-8ff4-d50bafbd0b5e">

## after
<img width="901" alt="Screenshot 2024-02-06 at 10 26 49 AM"
src="https://github.com/flutter/engine/assets/30870216/bc5cced7-b8c4-4597-9dd4-c460b966176b">


## Pre-launch Checklist

- [x] I read the [Contributor Guide] and followed the process outlined
there for submitting PRs.
- [x] I read the [Tree Hygiene] wiki page, which explains my
responsibilities.
- [x] I read and followed the [Flutter Style Guide] and the [C++,
Objective-C, Java style guides].
- [x] I listed at least one issue that this PR fixes in the description
above.
- [ ] I added new tests to check the change I am making or feature I am
adding, or the PR is [test-exempt]. See [testing the engine] for
instructions on writing and running engine tests.
- [x] I updated/added relevant documentation (doc comments with `///`).
- [x] I signed the [CLA].
- [x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel
on [Discord].

<!-- Links -->
[Contributor Guide]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#overview
[Tree Hygiene]: https://github.com/flutter/flutter/wiki/Tree-hygiene
[test-exempt]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#tests
[Flutter Style Guide]:
https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo
[C++, Objective-C, Java style guides]:
https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
[testing the engine]:
https://github.com/flutter/flutter/wiki/Testing-the-engine
[CLA]: https://cla.developers.google.com/
[flutter/tests]: https://github.com/flutter/tests
[breaking change policy]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#handling-breaking-changes
[Discord]: https://github.com/flutter/flutter/wiki/Chat
2024-02-08 08:39:05 -08:00
skia-flutter-autoroll
e1dcfcdae4 Roll Skia from 8e08ece134da to 97f12df11af2 (1 revision) (flutter/engine#50461)
https://skia.googlesource.com/skia.git/+log/8e08ece134da..97f12df11af2

2024-02-08 skia-autoroll@skia-public.iam.gserviceaccount.com Roll Skia Infra from 35d65e587f8e to fac5758b02f3 (4 revisions)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/skia-flutter-autoroll
Please CC brianosman@google.com,bungeman@google.com,chinmaygarde@google.com,rmistry@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Skia: https://bugs.chromium.org/p/skia/issues/entry
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
2024-02-08 05:53:23 +00:00
Xilai Zhang
4c16dfe8cc [github actions] update license year (flutter/engine#50458)
context: https://github.com/flutter/engine/pull/50265#pullrequestreview-1868207906

minor fix to license year.
2024-02-08 04:40:13 +00:00
Robert Ancell
6f39b9941b Clear objects before setting them. (flutter/engine#50344)
The existing code worked, but it cleared the objects before calling
init_keyboard. This made it look like init_keyboard might leak the
existing values. If a third case was added later where init_keyboard was
called it would leak if the caller didn't clear the objects before
calling it.
2024-02-08 16:20:57 +13:00
Rulong Chen(陈汝龙)
e3b318920a [ios] Fix memory leak in ChildClippingView (flutter/engine#50389)
This fixes the memory leak in `ChildClippingView`.

[C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
2024-02-08 01:43:37 +00:00
Michael Goderbauer
c3d355cadd Report display sizes in physical pixels on MacOS (flutter/engine#50221)
Fixes https://github.com/flutter/flutter/issues/142629

`Display.size`'s [documentation](https://main-api.flutter.dev/flutter/dart-ui/Display/size.html) says (emphasize mine):

> The **physical** size of this display.

But we have actually been reporting the size in logical pixels - up until now!
2024-02-07 23:19:19 +00:00
zijiehe@
bcd7116bef [Fuchsia] Run tests with multiple packages (flutter/engine#50219)
This change enables tests requiring multiple packages in both debug and
release builders; though the AOT test cannot be executed.
Also 4 out of 5 suites are excluded from release builder since some of
the tests are failing; it's worth extra investigation.
MouseInputTest in theory is enabled, it indeed contains only disabled
tests.

Bug: https://github.com/flutter/flutter/issues/140179

## Pre-launch Checklist

- [x] I read the [Contributor Guide] and followed the process outlined
there for submitting PRs.
- [x] I read the [Tree Hygiene] wiki page, which explains my
responsibilities.
- [x] I read and followed the [Flutter Style Guide] and the [C++,
Objective-C, Java style guides].
- [x] I listed at least one issue that this PR fixes in the description
above.
- [x] I added new tests to check the change I am making or feature I am
adding, or the PR is [test-exempt]. See [testing the engine] for
instructions on writing and running engine tests.
- [x] I updated/added relevant documentation (doc comments with `///`).
- [x] I signed the [CLA].
- [x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel
on [Discord].

<!-- Links -->
[Contributor Guide]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#overview
[Tree Hygiene]: https://github.com/flutter/flutter/wiki/Tree-hygiene
[test-exempt]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#tests
[Flutter Style Guide]:
https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo
[C++, Objective-C, Java style guides]:
https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
[testing the engine]:
https://github.com/flutter/flutter/wiki/Testing-the-engine
[CLA]: https://cla.developers.google.com/
[flutter/tests]: https://github.com/flutter/tests
[breaking change policy]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#handling-breaking-changes
[Discord]: https://github.com/flutter/flutter/wiki/Chat
2024-02-07 15:17:31 -08:00
skia-flutter-autoroll
5131cf0219 Roll Skia from 8332438cbeb9 to 91a9154d0dfa (2 revisions) (flutter/engine#50452)
https://skia.googlesource.com/skia.git/+log/8332438cbeb9..91a9154d0dfa

2024-02-07 skia-autoroll@skia-public.iam.gserviceaccount.com Manual roll Skia Infra from e83737e1d145 to 35d65e587f8e (29 revisions)
2024-02-07 kschmi@microsoft.com Add contrast and gamma to SkSurfaceProps and use in text rendering

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/skia-flutter-autoroll
Please CC brianosman@google.com,bungeman@google.com,chinmaygarde@google.com,rmistry@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Skia: https://bugs.chromium.org/p/skia/issues/entry
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
2024-02-07 22:41:19 +00:00
Jonah Williams
a363e03859 [Impeller] move path out of PathBuilder in TakePath. (flutter/engine#50444)
We should move this path so we don't end up allocating/deallocating twice. 

Fixes https://github.com/flutter/flutter/issues/142873
2024-02-07 22:01:04 +00:00
skia-flutter-autoroll
c720373a9e Roll Skia from eacaa2d3600c to 8332438cbeb9 (6 revisions) (flutter/engine#50451)
https://skia.googlesource.com/skia.git/+log/eacaa2d3600c..8332438cbeb9

2024-02-07 borenet@google.com [recipes] Fix SCP for ChromeOS
2024-02-07 borenet@google.com [recipes] Remove dependency on python recipe module
2024-02-07 skia-autoroll@skia-public.iam.gserviceaccount.com Roll skottie-base from 906e4c4c9b17 to 67e2da56aec4
2024-02-07 skia-autoroll@skia-public.iam.gserviceaccount.com Roll shaders-base from 89826f3f6f5d to 45ec924a5da0
2024-02-07 johnstiles@google.com Reland "Remove GR_TEST_UTILS from module loader."
2024-02-07 skia-autoroll@skia-public.iam.gserviceaccount.com Roll jsfiddle-base from b575daa903da to db3abc597412

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/skia-flutter-autoroll
Please CC brianosman@google.com,bungeman@google.com,chinmaygarde@google.com,rmistry@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Skia: https://bugs.chromium.org/p/skia/issues/entry
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
2024-02-07 21:46:06 +00:00
Dan Field
9f9ab34867 [Impeller] Log non-default graphics backend usages, use IMPORTANT rather than ERROR (flutter/engine#50448)
Fixes https://github.com/flutter/flutter/issues/142488

- Only logs on iOS if Skia is used instead of Impeller.
- Logs on other platforms if Impeller is used instead of Skia.
- Uses "IMPORTANT" rather than "ERROR" for these logs. This will show up by default since flutter_tools sets ERROR and above as logs to show.
- Adds some tests.
- Makes INFO log print file paths the same as other verbosities.
2024-02-07 21:29:12 +00:00
skia-flutter-autoroll
354e532c5f Roll Fuchsia Linux SDK from kvEXgoydRYnW3UvX2... to CPiPDUwCrjHYf_lAR... (flutter/engine#50447)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/fuchsia-linux-sdk-flutter-engine
Please CC chinmaygarde@google.com,rmistry@google.com,zra@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
2024-02-07 21:04:04 +00:00
Matan Lurey
b79821c019 Fix Android scenario_app to actually run and block on CI (flutter/engine#50414)
20337eaea4
(now reverted) intentionally re-introduced a bug
(https://github.com/flutter/engine/pull/49938) on a now working test
runner, and verified that (with some fixes in this PR), we can now catch
regressions on CI with SkiaGold:

---

<details>

<summary>ExternalTextureTests_testCanvasSurface.png</summary>

This test shows that what previously was a picture has been reduced down
to a single stretched pixel.

**BEFORE**:


![ExternalTextureTests_testCanvasSurface](https://github.com/flutter/engine/assets/168174/1509ea21-2887-4a3b-b200-b857bbfb8304)

**AFTER**:


![ExternalTextureTests_testCanvasSurface](https://github.com/flutter/engine/assets/168174/d9ae19c8-dd39-4e7f-859c-5d82bd1ba0a3)

</details>

<details>

<summary>ExternalTextureTests_testRotatedMediaSurface_180.png</summary>

Similar to above, but shows _another_ bug (kClamp versus kRepeat)

**BEFORE**:


![ExternalTextureTests_testRotatedMediaSurface_180](https://github.com/flutter/engine/assets/168174/fee5bc8d-749f-4a59-976a-a573515fecea)

**AFTER**:


![ExternalTextureTests_testRotatedMediaSurface_180](https://github.com/flutter/engine/assets/168174/a5249f70-22b4-4d36-8b63-e88c8a6846fe)/cc

</details>

---

This PR now makes the `scenario_test` useful and blocking for Android
engine rolls:

- Add support for connecting to SkiaGold by depending on `goldctl` in
the appropriate configs.
- Add some useful local debugging flags for skipping SkiaGold or running
specific tests (not used by default).
- Failing to connect to Skia gold is now a test failure.
2024-02-07 12:58:41 -08:00
auto-submit[bot]
53ba0f96e4 Reverts "Roll Dart SDK from 452dd17120b7 to 322c81160fa9 (1 revision)" (flutter/engine#50450)
Reverts flutter/engine#50437

Initiated by: zanderso

Reason for reverting: Blocking the roll to the framework

Original PR Author: skia-flutter-autoroll

Reviewed By: {fluttergithubbot}

This change reverts the following previous change:
Original Description:

https://dart.googlesource.com/sdk.git/+log/452dd17120b7..322c81160fa9

2024-02-07 dart-internal-merge@dart-ci-internal.iam.gserviceaccount.com Version 3.4.0-117.0.dev

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/dart-sdk-flutter-engine
Please CC chinmaygarde@google.com,dart-vm-team@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter Engine: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
2024-02-07 20:28:18 +00:00
Chinmay Garde
c00c8fe2a0 [Impeller] Initialize the stencil reference value. (flutter/engine#50373)
Viewports, scissors, and stencil reference values are part of the dynamic state and must be initialized in the case where the caller doesn't set any value via the render pass.

Fixes https://github.com/flutter/flutter/issues/142943
2024-02-07 19:42:21 +00:00
gaaclarke
d5a2976258 Fixed CONTRIBUTING.md markdown lints (flutter/engine#50439)
## Pre-launch Checklist

- [x] I read the [Contributor Guide] and followed the process outlined
there for submitting PRs.
- [x] I read the [Tree Hygiene] wiki page, which explains my
responsibilities.
- [x] I read and followed the [Flutter Style Guide] and the [C++,
Objective-C, Java style guides].
- [x] I listed at least one issue that this PR fixes in the description
above.
- [x] I added new tests to check the change I am making or feature I am
adding, or the PR is [test-exempt]. See [testing the engine] for
instructions on writing and running engine tests.
- [x] I updated/added relevant documentation (doc comments with `///`).
- [x] I signed the [CLA].
- [x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel
on [Discord].

<!-- Links -->
[Contributor Guide]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#overview
[Tree Hygiene]: https://github.com/flutter/flutter/wiki/Tree-hygiene
[test-exempt]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#tests
[Flutter Style Guide]:
https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo
[C++, Objective-C, Java style guides]:
https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
[testing the engine]:
https://github.com/flutter/flutter/wiki/Testing-the-engine
[CLA]: https://cla.developers.google.com/
[flutter/tests]: https://github.com/flutter/tests
[breaking change policy]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#handling-breaking-changes
[Discord]: https://github.com/flutter/flutter/wiki/Chat
2024-02-07 10:27:07 -08:00
skia-flutter-autoroll
c52aac60fe Roll Skia from 7777976cd518 to 9b103a4148e2 (1 revision) (flutter/engine#50438)
https://skia.googlesource.com/skia.git/+log/7777976cd518..9b103a4148e2

2024-02-07 jvanverth@google.com Remove some uses of GRAPHITE_TEST_UTILS outside of src/gpu/graphite.

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/skia-flutter-autoroll
Please CC brianosman@google.com,bungeman@google.com,chinmaygarde@google.com,rmistry@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Skia: https://bugs.chromium.org/p/skia/issues/entry
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
2024-02-07 18:06:01 +00:00
skia-flutter-autoroll
4611aadcc5 Roll Dart SDK from 452dd17120b7 to 322c81160fa9 (1 revision) (flutter/engine#50437)
https://dart.googlesource.com/sdk.git/+log/452dd17120b7..322c81160fa9

2024-02-07 dart-internal-merge@dart-ci-internal.iam.gserviceaccount.com Version 3.4.0-117.0.dev

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/dart-sdk-flutter-engine
Please CC chinmaygarde@google.com,dart-vm-team@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter Engine: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
2024-02-07 17:54:18 +00:00
zijiehe@
9a5734f5a9 Redo "[Fuchsia] Execute most of the testing/fuchsia/test_suites.yaml on debug and release builds" (flutter/engine#50408)
Reverts flutter/engine#50407,
https://github.com/flutter/flutter/issues/142811, redo
https://github.com/flutter/engine/pull/50058.

Comparing to the original change, this redo disabled lto / link time
optimization for x64 release builder. It's a known issue that lto
increased the compilation time from <5 min to over 30 min. (See commit
a1b104c61b.)
Disabling lto is safe on x64 release since fuchsia assembly uses arm64
release build on smart display instead. But I will send a PSA to the
fuchsia team before submitting this change.

Following is the original change description.

This change implements a BundledTestRunner to run most of the tests in
testing/fuchsia/test_suites.yaml as ExecutableTestRunner.

- Tests with packages out of out/fuchsia_*_x64/ are ignored for now.
- Tests with extra test command line parameters are ignored for now.

The BundledTestRunner can share most of the logic in
ExecutableTestRunner and avoid reinventing the wheel.

This change also fixes the build break of fuchsia_tests in
fuchsia_release_x64 which allows tests to run on the build as well.

- Tests not built with AOT are filtered out with variant field in
test_suites.yaml.

Bug: https://github.com/flutter/flutter/issues/140179

## Pre-launch Checklist

- [x] I read the [Contributor Guide] and followed the process outlined
there for submitting PRs.
- [x] I read the [Tree Hygiene] wiki page, which explains my
responsibilities.
- [x] I read and followed the [Flutter Style Guide] and the [C++,
Objective-C, Java style guides].
- [x] I listed at least one issue that this PR fixes in the description
above.
- [x] I added new tests to check the change I am making or feature I am
adding, or the PR is [test-exempt]. See [testing the engine] for
instructions on writing and running engine tests.
- [x] I updated/added relevant documentation (doc comments with `///`).
- [x] I signed the [CLA].
- [x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel
on [Discord].

<!-- Links -->
[Contributor Guide]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#overview
[Tree Hygiene]: https://github.com/flutter/flutter/wiki/Tree-hygiene
[test-exempt]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#tests
[Flutter Style Guide]:
https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo
[C++, Objective-C, Java style guides]:
https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
[testing the engine]:
https://github.com/flutter/flutter/wiki/Testing-the-engine
[CLA]: https://cla.developers.google.com/
[flutter/tests]: https://github.com/flutter/tests
[breaking change policy]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#handling-breaking-changes
[Discord]: https://github.com/flutter/flutter/wiki/Chat
2024-02-07 09:50:23 -08:00
Zachary Anderson
1fc1563a3f Update NDK to version 26 (flutter/engine#48254)
This PR adjusts the GN build for a newer Android NDK. It relies on
https://github.com/flutter/buildroot/pull/822.
2024-02-07 08:42:08 -08:00
ᠵᠠᠭᠠᠨᠵᠠᠪ ᠪᠤᠷᠵᠢᠭᠤᠳ ᠰᠠᠴᠤᠷᠠᠭ
af574f2aae Fix: On the Web, cannot support multiline inputting when registering customized TextInputControl (flutter/engine#45522)
For [flutter/flutter/125875](https://github.com/flutter/flutter/issues/125875)

When registering customized TextInputControl, the _PlatformTextInputControl sends inputType = TextInputType.none to the engine. After receiving TextInputType.none, the engine on the Web will create a `<input>` element instead of  `<textarea>`. So there is no way to input \n(multiline).

This is my solution. I tested in Android Chrome, iOS Safari, and macOS Chrome. It works for me. But I'm not sure about other use cases. I pleasure, If someone gives me suggestions.
2024-02-07 15:15:00 +00:00
Brandon DeRosier
e4e1d5cc21 [Impeller] Always pop clips on restore. (flutter/engine#50419)
Fixes https://github.com/flutter/flutter/issues/142846

We need to always resolve clips when `Restore()` is called. Before, I just had it resolving the clips for canvas xformation stack entires created via `SaveLayer`.
2024-02-07 05:17:10 +00:00
skia-flutter-autoroll
6c88cab2c4 Roll Dart SDK from 35269fa71956 to 452dd17120b7 (1 revision) (flutter/engine#50424)
https://dart.googlesource.com/sdk.git/+log/35269fa71956..452dd17120b7

2024-02-06 dart-internal-merge@dart-ci-internal.iam.gserviceaccount.com Version 3.4.0-116.0.dev

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/dart-sdk-flutter-engine
Please CC chinmaygarde@google.com,dart-vm-team@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter Engine: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
2024-02-07 01:47:09 +00:00
skia-flutter-autoroll
3acebcbb84 Roll Skia from 44106ee8edea to fd55a6bd3580 (17 revisions) (flutter/engine#50420)
https://skia.googlesource.com/skia.git/+log/44106ee8edea..fd55a6bd3580

2024-02-06 johnstiles@google.com Update Tint method calls to match new style.
2024-02-06 herb@google.com Legacy flag SK_USE_FAST_UNPREMUL
2024-02-06 skia-autoroll@skia-public.iam.gserviceaccount.com Roll SK Tool from 4e1f58585b72 to 229ccfa616f1
2024-02-06 skia-autoroll@skia-public.iam.gserviceaccount.com Roll jsfiddle-base from fad68ece4009 to b575daa903da
2024-02-06 skia-autoroll@skia-public.iam.gserviceaccount.com Roll shaders-base from bf2a9ca94ef9 to 89826f3f6f5d
2024-02-06 skia-autoroll@skia-public.iam.gserviceaccount.com Manual roll Dawn from c9c20468546f to bdc7e32ee461 (27 revisions)
2024-02-06 skia-autoroll@skia-public.iam.gserviceaccount.com Roll debugger-app-base from d3314cdd050f to d2ed43ed5bb9
2024-02-06 herb@google.com Move rgbA_to_RGBA to specific opts
2024-02-06 skia-autoroll@skia-public.iam.gserviceaccount.com Roll skottie-base from f883f8608613 to 906e4c4c9b17
2024-02-06 skia-autoroll@skia-public.iam.gserviceaccount.com Roll skottie-base from 906e4c4c9b17 to f883f8608613
2024-02-06 skia-autoroll@skia-public.iam.gserviceaccount.com Roll shaders-base from 89826f3f6f5d to bf2a9ca94ef9
2024-02-06 michaelludwig@google.com [skif] Consolidate bounds analysis, refine flags
2024-02-06 kjlubick@google.com Remove old type union from GrGLInterface.h
2024-02-06 skia-autoroll@skia-public.iam.gserviceaccount.com Roll jsfiddle-base from b575daa903da to fad68ece4009
2024-02-06 skia-autoroll@skia-public.iam.gserviceaccount.com Roll debugger-app-base from d2ed43ed5bb9 to d3314cdd050f
2024-02-06 herb@google.com Implement bare bones unpremul conversions
2024-02-06 kjlubick@google.com Stop defining 'GR_TEST_UTILS' in G3

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/skia-flutter-autoroll
Please CC brianosman@google.com,bungeman@google.com,chinmaygarde@google.com,rmistry@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Skia: https://bugs.chromium.org/p/skia/issues/entry
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
2024-02-06 23:48:38 +00:00
Brandon DeRosier
06dc0e966f [Impeller] Fix pipeline attachment layout in CanRenderToTexture. (flutter/engine#50413)
The pipeline descriptor defaults changed, so we need to align the attachments to the RenderPass in this test.
Should fix https://github.com/flutter/flutter/issues/142868.
2024-02-06 23:13:04 +00:00
Ian Hickson
05e5a97357 Provide toStrings for Native objects (flutter/engine#50168)
This hides the "Instance of _Native" bit that you get in debug output otherwise.
2024-02-06 22:39:00 +00:00
keyonghan
680d87ab9d Add use_rbe to gclient variables for Framework Smoke Tests (flutter/engine#50403)
This is to enable rbe support. Part of https://github.com/flutter/flutter/issues/142986
2024-02-06 19:18:59 +00:00
auto-submit[bot]
6173975c27 Reverts "Revert "Revert "[Fuchsia] Execute most of the testing/fuchsia/test_suites.yaml on debug and release builds""" (flutter/engine#50407)
Reverts "Reland "[Fuchsia] Execute most of the testing/fuchsia/test_suites.yaml on debug and release builds"""

Reverts flutter/engine#50295

Initiated by: zanderso

Reason for reverting: Timing out on CI

Original PR Author: zijiehe-google-com

Reviewed By: {keyonghan}

This change reverts the following previous change:
Original Description:
Reverts flutter/engine#50291, https://github.com/flutter/flutter/issues/142811

Following is the original change description.

This change implements a BundledTestRunner to run most of the tests in testing/fuchsia/test_suites.yaml as ExecutableTestRunner.

- Tests with packages out of out/fuchsia_*_x64/ are ignored for now.
- Tests with extra test command line parameters are ignored for now.

The BundledTestRunner can share most of the logic in ExecutableTestRunner and avoid reinventing the wheel.

This change also fixes the build break of fuchsia_tests in fuchsia_release_x64 which allows tests to run on the build as well.

- Tests not built with AOT are filtered out with variant field in test_suites.yaml.

Bug: https://github.com/flutter/flutter/issues/140179

[C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
2024-02-06 18:49:25 +00:00
Matan Lurey
4ed520b2af Skip flaking test on Windows nobody is fixing. (flutter/engine#50401)
Filed https://github.com/flutter/flutter/issues/142991 to re-enable.
2024-02-06 10:23:06 -08:00
auto-submit[bot]
911b1400ea Reverts "[web] Fix Scene clip bounds. Trigger resize on DPR Change." (flutter/engine#50404)
Reverts flutter/engine#50161

Initiated by: jonahwilliams

Reason for reverting: This is causing what looks like bogus goldens on the framework -> engine roll: https://github.com/flutter/flutter/pull/142966

Original PR Author: ditman

Reviewed By: {yjbanov, mdebbar}

This change reverts the following previous change:
Original Description:
The Scene of the HTML renderer is passing incorrect size information to the engine, and when DPR<1, it can result in elements being culled off of the viewport.

In addition to that, when an app is embedded, not all changes in DPR cause a Resize event (only those some of the dimensions fails by a rounding error!), so this PR ensures that all DPR events in embedded trigger a resize event.

### Issues

Fixes https://github.com/flutter/flutter/issues/129182

### Testing

Looking good at: https://dit-astral-test.web.app

[C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
2024-02-06 17:35:47 +00:00
zijiehe@
76dce91cf2 Revert "Revert "[Fuchsia] Execute most of the testing/fuchsia/test_suites.yaml on debug and release builds"" (flutter/engine#50295)
Reverts flutter/engine#50291,
https://github.com/flutter/flutter/issues/142811

Following is the original change description.

This change implements a BundledTestRunner to run most of the tests in
testing/fuchsia/test_suites.yaml as ExecutableTestRunner.

- Tests with packages out of out/fuchsia_*_x64/ are ignored for now.
- Tests with extra test command line parameters are ignored for now.

The BundledTestRunner can share most of the logic in
ExecutableTestRunner and avoid reinventing the wheel.

This change also fixes the build break of fuchsia_tests in
fuchsia_release_x64 which allows tests to run on the build as well.

- Tests not built with AOT are filtered out with variant field in
test_suites.yaml.

Bug: https://github.com/flutter/flutter/issues/140179

## Pre-launch Checklist

- [x] I read the [Contributor Guide] and followed the process outlined
there for submitting PRs.
- [x] I read the [Tree Hygiene] wiki page, which explains my
responsibilities.
- [x] I read and followed the [Flutter Style Guide] and the [C++,
Objective-C, Java style guides].
- [x] I listed at least one issue that this PR fixes in the description
above.
- [x] I added new tests to check the change I am making or feature I am
adding, or the PR is [test-exempt]. See [testing the engine] for
instructions on writing and running engine tests.
- [x] I updated/added relevant documentation (doc comments with `///`).
- [x] I signed the [CLA].
- [x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel
on [Discord].

<!-- Links -->
[Contributor Guide]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#overview
[Tree Hygiene]: https://github.com/flutter/flutter/wiki/Tree-hygiene
[test-exempt]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#tests
[Flutter Style Guide]:
https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo
[C++, Objective-C, Java style guides]:
https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
[testing the engine]:
https://github.com/flutter/flutter/wiki/Testing-the-engine
[CLA]: https://cla.developers.google.com/
[flutter/tests]: https://github.com/flutter/tests
[breaking change policy]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#handling-breaking-changes
[Discord]: https://github.com/flutter/flutter/wiki/Chat
2024-02-06 09:34:53 -08:00
Chinmay Garde
9d32cac3ed [Impeller] Specify if Angle or SwiftShader is being used in the title. (flutter/engine#50376)
Minor QOL improvement. Tried taking a frame capture unsuccessfully before realizing that I was using SwiftShader.
2024-02-06 17:33:45 +00:00
Matan Lurey
5d1cc934ad Run all Android scenario_app tests, not just the smoke test. (flutter/engine#50400)
I was making sure that I could get at least one test running and passing
on CI (as expected), now turn all of them on.
2024-02-06 09:15:08 -08:00
skia-flutter-autoroll
473f0508b3 Roll Dart SDK from b62066b42af0 to 29265c94a6e8 (10 revisions) (flutter/engine#50398)
https://dart.googlesource.com/sdk.git/+log/b62066b42af0..29265c94a6e8

2024-02-06 dart-internal-merge@dart-ci-internal.iam.gserviceaccount.com Version 3.4.0-114.0.dev
2024-02-06 dart-internal-merge@dart-ci-internal.iam.gserviceaccount.com Version 3.4.0-113.0.dev
2024-02-06 dart-internal-merge@dart-ci-internal.iam.gserviceaccount.com Version 3.4.0-112.0.dev
2024-02-05 dart-internal-merge@dart-ci-internal.iam.gserviceaccount.com Version 3.4.0-111.0.dev
2024-02-05 dart-internal-merge@dart-ci-internal.iam.gserviceaccount.com Version 3.4.0-110.0.dev
2024-02-05 dart-internal-merge@dart-ci-internal.iam.gserviceaccount.com Version 3.4.0-109.0.dev
2024-02-05 dart-internal-merge@dart-ci-internal.iam.gserviceaccount.com Version 3.4.0-108.0.dev
2024-02-03 dart-internal-merge@dart-ci-internal.iam.gserviceaccount.com Version 3.4.0-107.0.dev
2024-02-03 dart-internal-merge@dart-ci-internal.iam.gserviceaccount.com Version 3.4.0-106.0.dev
2024-02-02 dart-internal-merge@dart-ci-internal.iam.gserviceaccount.com Version 3.4.0-105.0.dev

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/dart-sdk-flutter-engine
Please CC chinmaygarde@google.com,dart-vm-team@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter Engine: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
2024-02-06 16:48:31 +00:00
Matan Lurey
8db41b6544 Capture FAILURES!!! when running Android scenario_app tests. (flutter/engine#50255)
I expect `Linux linux_android_emulator_tests` and `Linux
linux_android_emulator_tests_api_33` to fail.

When they fail, I'll fix the assertion
(https://github.com/flutter/flutter/issues/142746) before merging this
PR.
2024-02-06 08:42:14 -08:00
Jonny Wang
eb4b07ee1d [fuchsia] Bump Fuchsia's API level to 16 (flutter/engine#50358)
Update Fuchsia's API level to 16.

b/322503140


## Pre-launch Checklist

- [x] I read the [Contributor Guide] and followed the process outlined
there for submitting PRs.
- [x] I read the [Tree Hygiene] wiki page, which explains my
responsibilities.
- [x] I read and followed the [Flutter Style Guide] and the [C++,
Objective-C, Java style guides].
- [x] I listed at least one issue that this PR fixes in the description
above.
- [x] I added new tests to check the change I am making or feature I am
adding, or the PR is [test-exempt]. See [testing the engine] for
instructions on writing and running engine tests.
- [x] I updated/added relevant documentation (doc comments with `///`).
- [x] I signed the [CLA].
- [x] All existing and new tests are passing.
2024-02-06 15:34:37 +00:00
Jonah Williams
a70478263d [Impeller] Cache RenderPass/Framebuffer objects on the resolve texture sources. (flutter/engine#50142)
Cache vk::RenderPass and vk::Framebuffer objects on the resolve texture of any render target attachments. Use these on the next frame unconditionally.

Fixes https://github.com/flutter/flutter/issues/141750
2024-02-06 04:35:05 +00:00
Jason Simmons
39a6ac669f [Impeller] Do not skip the GLES render pass if the command list is empty (flutter/engine#50381)
The render pass may need to apply other changes such as painting the clear color.

See https://github.com/flutter/flutter/issues/142639
2024-02-06 02:43:27 +00:00
skia-flutter-autoroll
dfd78684bd Roll Skia from cdf214adfb4d to c29a20702356 (54 revisions) (flutter/engine#50382)
Roll Skia from cdf214adfb4d to c29a20702356 (54 revisions)

https://skia.googlesource.com/skia.git/+log/cdf214adfb4d..c29a20702356

2024-02-06 brianosman@google.com Bring back DISABLE_LOWP
2024-02-06 skia-autoroll@skia-public.iam.gserviceaccount.com Roll ANGLE from f8c06f103a1f to 5a0615588a1a (10 revisions)
2024-02-05 lokokung@google.com Migrates graphite's main wait mechanism from using Tick to ProcessEvents.
2024-02-05 skia-autoroll@skia-public.iam.gserviceaccount.com Roll vulkan-deps from 3a55a3d5ee47 to 83223ec30684 (1 revision)
2024-02-05 fmalita@chromium.org [skottie] Fix supernormal spatial interpolation
2024-02-05 robertphillips@google.com [graphite] Add new Graphite Precompilation DMSink
2024-02-05 skia-autoroll@skia-public.iam.gserviceaccount.com Roll Dawn from dd852a700405 to c9c20468546f (26 revisions)
2024-02-05 skia-autoroll@skia-public.iam.gserviceaccount.com Roll SK Tool from 9a1a69ed50c4 to 5c8f3c7d6467
2024-02-05 kjlubick@google.com Fix skottie rules for G3
2024-02-05 kjlubick@google.com Temporarily disable animated_gif.cpp from Bazel test
2024-02-05 kjlubick@google.com Add bazel modules for Skottie and sksg
2024-02-03 skia-autoroll@skia-public.iam.gserviceaccount.com Roll vulkan-deps from a1def468c6b4 to 3a55a3d5ee47 (9 revisions)
2024-02-03 skia-autoroll@skia-public.iam.gserviceaccount.com Roll SK Tool from 8af152c5f7ce to 9a1a69ed50c4
2024-02-02 kjlubick@google.com Fix copypasta in public.bzl
2024-02-02 kjlubick@google.com Guard some gms/tests that require Skottie code
2024-02-02 skia-autoroll@skia-public.iam.gserviceaccount.com Roll jsfiddle-base from 969399a89649 to b575daa903da
2024-02-02 skia-autoroll@skia-public.iam.gserviceaccount.com Roll shaders-base from 9054c5988daf to 89826f3f6f5d
2024-02-02 skia-autoroll@skia-public.iam.gserviceaccount.com Roll skottie-base from 9ce4d2efff0c to 906e4c4c9b17
2024-02-02 skia-autoroll@skia-public.iam.gserviceaccount.com Roll debugger-app-base from 54bf3755c837 to d2ed43ed5bb9
2024-02-02 bungeman@google.com [unicode] Fix SkBreakIterator_libgrapheme
2024-02-02 skia-autoroll@skia-public.iam.gserviceaccount.com Roll vulkan-deps from 07589c29ccba to a1def468c6b4 (1 revision)
2024-02-02 kjlubick@google.com Make SVG backend explicitly depend on JPEG+PNG for OpenType
2024-02-02 kjlubick@google.com Add Bazel target for modular build of SkResources
2024-02-02 kjlubick@google.com Remove SkAnimCodecPlayer from the public API
2024-02-02 skia-autoroll@skia-public.iam.gserviceaccount.com Roll skottie-base from 5b5661dc98c7 to 9ce4d2efff0c
2024-02-02 skia-autoroll@skia-public.iam.gserviceaccount.com Roll shaders-base from e7695f0fdff5 to 9054c5988daf
2024-02-02 skia-autoroll@skia-public.iam.gserviceaccount.com Roll jsfiddle-base from f6c22747cda4 to 969399a89649
2024-02-02 skia-autoroll@skia-public.iam.gserviceaccount.com Roll debugger-app-base from 067d72e0dddf to 54bf3755c837
2024-02-02 johnstiles@google.com Fix fuzzer-discovered timeout by caching uniform information.
2024-02-02 bungeman@google.com [pdf] Properly handle drawing not on a page
2024-02-02 johnstiles@google.com Add method Type::isAllowedInUniform.
2024-02-02 brianosman@google.com Remove SK_DISABLE_LOWP_RASTER_PIPELINE
2024-02-02 drott@chromium.org [Fontations-backend] Minor: Remove return statement
2024-02-02 49699333+dependabot[bot]@users.noreply.github.com Bump follow-redirects from 1.14.8 to 1.15.5
2024-02-02 49699333+dependabot[bot]@users.noreply.github.com Bump socket.io-parser from 4.2.2 to 4.2.4 in /modules/canvaskit
2024-02-02 49699333+dependabot[bot]@users.noreply.github.com Bump jinja2 from 2.11.3 to 3.1.3
2024-02-02 kjlubick@google.com Update Bazel rules after vulkan_header update
2024-02-02 skia-autoroll@skia-public.iam.gserviceaccount.com Roll ANGLE from 1abfdc37320f to f8c06f103a1f (3 revisions)
2024-02-02 skia-autoroll@skia-public.iam.gserviceaccount.com Roll vulkan-deps from 82b9fb3ddb57 to 07589c29ccba (6 revisions)
2024-02-02 skia-autoroll@skia-public.iam.gserviceaccount.com Roll SK Tool from e83737e1d145 to b8aac8b590af
2024-02-02 skia-autoroll@skia-public.iam.gserviceaccount.com Roll Skia Infra from 6e97526ab534 to e83737e1d145 (14 revisions)
2024-02-02 skia-autoroll@skia-public.iam.gserviceaccount.com Roll SwiftShader from 78d1799ee43e to eb75201a4e03 (2 revisions)
2024-02-02 skia-autoroll@skia-public.iam.gserviceaccount.com Roll Dawn from 2c3a08ff64ba to dd852a700405 (8 revisions)
2024-02-02 bungeman@google.com Revert "[pdf] Properly handle drawing not on a page"
2024-02-02 johnstiles@google.com Add BUILD.gn cleanups to zlib.
2024-02-01 kjlubick@google.com Make SkScalarContext for Windows DirectWrite explicitly decode PNGs
...
2024-02-06 02:09:20 +00:00
David Iglesias
76129ffb8f [web] Fix Scene clip bounds. Trigger resize on DPR Change. (flutter/engine#50161)
The Scene of the HTML renderer is passing incorrect size information to the engine, and when DPR<1, it can result in elements being culled off of the viewport.

In addition to that, when an app is embedded, not all changes in DPR cause a Resize event (only those some of the dimensions fails by a rounding error!), so this PR ensures that all DPR events in embedded trigger a resize event.

### Issues

Fixes https://github.com/flutter/flutter/issues/129182

### Testing

Looking good at: https://dit-astral-test.web.app

[C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
2024-02-06 01:48:00 +00:00
Xilai Zhang
db877dd92a [github actions] add cherry pick workflow for engine repo (flutter/engine#50265)
Follow up from sync with @itsjustkevin : Add cherry pick github actions to flutter/engine repository.

Would have the same functionalities as https://github.com/flutter/flutter/blob/master/.github/workflows/easy-cp.yml.

@godofredoc would be great if we could add [actions bot token](https://valentine.corp.google.com/#/show/1702960394753966?tab=metadata) as a secret under the name FLUTTERACTIONSBOT_CP_TOKEN to the engine repository too. Thank you!
2024-02-06 01:31:54 +00:00