Skip to content

Commit 344ecd1

Browse files
authored
Merge branch 'flutter:main' into feature/video-player/add-optional-web-options
2 parents e4655dc + 3eaad3d commit 344ecd1

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

51 files changed

+713
-254
lines changed

.ci.yaml

Lines changed: 2 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,6 @@ targets:
9797
version_file: flutter_master.version
9898

9999
- name: Linux repo_checks
100-
bringup: true # New target
101100
recipe: packages/packages
102101
timeout: 30
103102
properties:
@@ -246,11 +245,9 @@ targets:
246245
channel: master
247246

248247
- name: Linux_android custom_package_tests stable
249-
bringup: true # Blocked on https://github.com/flutter/flutter/issues/130071
250248
recipe: packages/packages
251249
timeout: 30
252250
properties:
253-
add_recipes_cq: "true"
254251
version_file: flutter_stable.version
255252
target_file: linux_custom_package_tests.yaml
256253
cores: "32"
@@ -282,7 +279,6 @@ targets:
282279
channel: stable
283280

284281
- name: Linux_android android_platform_tests_shard_1 master
285-
bringup: true # New target
286282
recipe: packages/packages
287283
timeout: 60
288284
properties:
@@ -293,7 +289,6 @@ targets:
293289
package_sharding: "--shardIndex 0 --shardCount 6"
294290

295291
- name: Linux_android android_platform_tests_shard_2 master
296-
bringup: true # New target
297292
recipe: packages/packages
298293
timeout: 60
299294
properties:
@@ -304,7 +299,6 @@ targets:
304299
package_sharding: "--shardIndex 1 --shardCount 6"
305300

306301
- name: Linux_android android_platform_tests_shard_3 master
307-
bringup: true # New target
308302
recipe: packages/packages
309303
timeout: 60
310304
properties:
@@ -315,7 +309,6 @@ targets:
315309
package_sharding: "--shardIndex 2 --shardCount 6"
316310

317311
- name: Linux_android android_platform_tests_shard_4 master
318-
bringup: true # New target
319312
recipe: packages/packages
320313
timeout: 60
321314
properties:
@@ -326,7 +319,6 @@ targets:
326319
package_sharding: "--shardIndex 3 --shardCount 6"
327320

328321
- name: Linux_android android_platform_tests_shard_5 master
329-
bringup: true # New target
330322
recipe: packages/packages
331323
timeout: 60
332324
properties:
@@ -337,18 +329,16 @@ targets:
337329
package_sharding: "--shardIndex 4 --shardCount 6"
338330

339331
- name: Linux_android android_platform_tests_shard_6 master
340-
bringup: true # New target
341332
recipe: packages/packages
342333
timeout: 60
343334
properties:
344335
target_file: android_platform_tests.yaml
345336
channel: master
346337
version_file: flutter_master.version
347338
cores: "32"
348-
package_sharding: "--shardIndex 6 --shardCount 6"
339+
package_sharding: "--shardIndex 5 --shardCount 6"
349340

350341
- name: Linux_android android_platform_tests_shard_1 stable
351-
bringup: true # New target
352342
recipe: packages/packages
353343
presubmit: false
354344
timeout: 60
@@ -360,7 +350,6 @@ targets:
360350
package_sharding: "--shardIndex 0 --shardCount 6"
361351

362352
- name: Linux_android android_platform_tests_shard_2 stable
363-
bringup: true # New target
364353
recipe: packages/packages
365354
presubmit: false
366355
timeout: 60
@@ -372,7 +361,6 @@ targets:
372361
package_sharding: "--shardIndex 1 --shardCount 6"
373362

374363
- name: Linux_android android_platform_tests_shard_3 stable
375-
bringup: true # New target
376364
recipe: packages/packages
377365
presubmit: false
378366
timeout: 60
@@ -384,7 +372,6 @@ targets:
384372
package_sharding: "--shardIndex 2 --shardCount 6"
385373

386374
- name: Linux_android android_platform_tests_shard_4 stable
387-
bringup: true # New target
388375
recipe: packages/packages
389376
presubmit: false
390377
timeout: 60
@@ -396,7 +383,6 @@ targets:
396383
package_sharding: "--shardIndex 3 --shardCount 6"
397384

398385
- name: Linux_android android_platform_tests_shard_5 stable
399-
bringup: true # New target
400386
recipe: packages/packages
401387
presubmit: false
402388
timeout: 60
@@ -408,7 +394,6 @@ targets:
408394
package_sharding: "--shardIndex 4 --shardCount 6"
409395

410396
- name: Linux_android android_platform_tests_shard_6 stable
411-
bringup: true # New target
412397
recipe: packages/packages
413398
presubmit: false
414399
timeout: 60
@@ -417,7 +402,7 @@ targets:
417402
channel: stable
418403
version_file: flutter_stable.version
419404
cores: "32"
420-
package_sharding: "--shardIndex 6 --shardCount 6"
405+
package_sharding: "--shardIndex 5 --shardCount 6"
421406

422407
### Web tasks ###
423408
- name: Linux_web web_build_all_packages master
@@ -438,7 +423,6 @@ targets:
438423
channel: stable
439424

440425
- name: Linux_web web_platform_tests_shard_1 master
441-
bringup: true # New target
442426
recipe: packages/packages
443427
timeout: 60
444428
properties:
@@ -449,7 +433,6 @@ targets:
449433
package_sharding: "--shardIndex 0 --shardCount 2"
450434

451435
- name: Linux_web web_platform_tests_shard_2 master
452-
bringup: true # New target
453436
recipe: packages/packages
454437
timeout: 60
455438
properties:
@@ -460,7 +443,6 @@ targets:
460443
package_sharding: "--shardIndex 1 --shardCount 2"
461444

462445
- name: Linux_web web_platform_tests_shard_1 stable
463-
bringup: true # New target
464446
recipe: packages/packages
465447
timeout: 60
466448
properties:
@@ -471,7 +453,6 @@ targets:
471453
package_sharding: "--shardIndex 0 --shardCount 2"
472454

473455
- name: Linux_web web_platform_tests_shard_2 stable
474-
bringup: true # New target
475456
recipe: packages/packages
476457
timeout: 60
477458
properties:

.ci/flutter_master.version

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
bc49cd1bcab51c491789e36dbbc4b53ca4d49dc6
1+
d55a7d89e05ede19676ef8065e4c72123107b6dd

.ci/targets/android_platform_tests.yaml

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,11 @@ tasks:
1111
# different exclusions.
1212
# TODO(stuartmorgan): Eliminate the native unit test exclusion, and combine
1313
# these steps.
14-
- name: native unit tests
15-
script: script/tool_runner.sh
16-
args: ["native-test", "--android", "--no-integration", "--exclude=script/configs/exclude_native_unit_android.yaml"]
14+
# TODO(stuartmorgan): Enable this once https://github.com/flutter/flutter/issues/130148
15+
# is resolved.
16+
#- name: native unit tests
17+
# script: script/tool_runner.sh
18+
# args: ["native-test", "--android", "--no-integration", "--exclude=script/configs/exclude_native_unit_android.yaml"]
1719
# TODO(stuartmorgan): Enable these once
1820
# https://github.com/flutter/flutter/issues/120736 is implemented.
1921
# See also https://github.com/flutter/flutter/issues/114373

.cirrus.yml

Lines changed: 6 additions & 59 deletions
Original file line numberDiff line numberDiff line change
@@ -65,20 +65,6 @@ task:
6565
# (on Flutter master).
6666
- name: repo_checks
6767
always:
68-
format_script: ./script/tool_runner.sh format --fail-on-change
69-
license_script: $PLUGIN_TOOL_COMMAND license-check
70-
# The major and minor version here should match the lowest version
71-
# analyzed in legacy_version_analyze.
72-
pubspec_script: ./script/tool_runner.sh pubspec-check --min-min-flutter-version=3.3.0 --allow-dependencies=script/configs/allowed_unpinned_deps.yaml --allow-pinned-dependencies=script/configs/allowed_pinned_deps.yaml
73-
readme_script:
74-
- ./script/tool_runner.sh readme-check
75-
# Re-run with --require-excerpts, skipping packages that still need
76-
# to be converted. Once https://github.com/flutter/flutter/issues/102679
77-
# has been fixed, this can be removed and there can just be a single
78-
# run with --require-excerpts and no exclusions.
79-
- ./script/tool_runner.sh readme-check --require-excerpts --exclude=script/configs/temp_exclude_excerpt.yaml
80-
dependabot_script: $PLUGIN_TOOL_COMMAND dependabot-check
81-
gradle_script: $PLUGIN_TOOL_COMMAND gradle-check
8268
version_script:
8369
# For pre-submit, pass the PR labels to the script to allow for
8470
# check overrides.
@@ -90,7 +76,6 @@ task:
9076
- else
9177
- ./script/tool_runner.sh version-check --check-for-missing-changes --pr-labels="$CIRRUS_PR_LABELS"
9278
- fi
93-
publishable_script: ./script/tool_runner.sh publish-check --allow-pre-release
9479
federated_safety_script:
9580
# This check is only meaningful for PRs, as it validates changes
9681
# rather than state.
@@ -157,43 +142,26 @@ task:
157142
cpu: 4
158143
memory: 16G
159144
matrix:
160-
### Platform-agnostic tasks ###
161-
- name: linux-custom_package_tests
162-
env:
163-
PATH: $PATH:/usr/local/bin
164-
# Master has been migrated to LUCI, but stable is blocked on
165-
# https://github.com/flutter/flutter/issues/130071
166-
CHANNEL: "stable"
167-
<< : *INSTALL_CHROME_LINUX
168-
local_tests_script:
169-
- ./script/tool_runner.sh custom-test
170145
### Android tasks ###
171146
- name: android-platform_tests
172147
# Don't run full platform tests on both channels in pre-submit.
173148
skip: $CIRRUS_PR != '' && $CHANNEL == 'stable'
174149
env:
175150
matrix:
176-
PACKAGE_SHARDING: "--shardIndex 0 --shardCount 8"
177-
PACKAGE_SHARDING: "--shardIndex 1 --shardCount 8"
178-
PACKAGE_SHARDING: "--shardIndex 2 --shardCount 8"
179-
PACKAGE_SHARDING: "--shardIndex 3 --shardCount 8"
180-
PACKAGE_SHARDING: "--shardIndex 4 --shardCount 8"
181-
PACKAGE_SHARDING: "--shardIndex 5 --shardCount 8"
182-
PACKAGE_SHARDING: "--shardIndex 6 --shardCount 8"
183-
PACKAGE_SHARDING: "--shardIndex 7 --shardCount 8"
151+
PACKAGE_SHARDING: "--shardIndex 0 --shardCount 6"
152+
PACKAGE_SHARDING: "--shardIndex 1 --shardCount 6"
153+
PACKAGE_SHARDING: "--shardIndex 2 --shardCount 6"
154+
PACKAGE_SHARDING: "--shardIndex 3 --shardCount 6"
155+
PACKAGE_SHARDING: "--shardIndex 4 --shardCount 6"
156+
PACKAGE_SHARDING: "--shardIndex 5 --shardCount 6"
184157
matrix:
185158
CHANNEL: "master"
186159
CHANNEL: "stable"
187160
MAPS_API_KEY: ENCRYPTED[d6583b08f79f91ea4844c77460f04539965e46ad2fd97fb7c062b4dfe88016228b86ebe8c220ab4187e0c4bd773dc1e7]
188161
GCLOUD_FIREBASE_TESTLAB_KEY: ENCRYPTED[1a2eebf9367197bbe812d9a0ea83a53a05aeba4bb5e4964fe6a69727883cd87e51238d39237b1f80b0894c48419ac268]
189-
build_script:
190-
- ./script/tool_runner.sh build-examples --apk
191-
lint_script:
192-
- ./script/tool_runner.sh lint-android # must come after build-examples
193162
native_unit_test_script:
194163
# Native integration tests are handled by Firebase Test Lab below, so
195164
# only run unit tests.
196-
# Must come after build-examples.
197165
- ./script/tool_runner.sh native-test --android --no-integration --exclude script/configs/exclude_native_unit_android.yaml
198166
firebase_test_lab_script:
199167
- if [[ -n "$GCLOUD_FIREBASE_TESTLAB_KEY" ]]; then
@@ -202,28 +170,7 @@ task:
202170
- else
203171
- echo "This user does not have permission to run Firebase Test Lab tests."
204172
- fi
205-
# Upload the full lint results to Cirrus to display in the results UI.
206-
always:
207-
android-lint_artifacts:
208-
path: "**/reports/lint-results-debug.xml"
209-
type: text/xml
210-
format: android-lint
211173
### Web tasks ###
212-
- name: web-platform_tests
213-
env:
214-
matrix:
215-
PACKAGE_SHARDING: "--shardIndex 0 --shardCount 2"
216-
PACKAGE_SHARDING: "--shardIndex 1 --shardCount 2"
217-
matrix:
218-
CHANNEL: "master"
219-
CHANNEL: "stable"
220-
<< : *INSTALL_CHROME_LINUX
221-
chromedriver_background_script:
222-
- $CHROMEDRIVER_EXECUTABLE --port=4444
223-
build_script:
224-
- ./script/tool_runner.sh build-examples --web
225-
drive_script:
226-
- ./script/tool_runner.sh drive-examples --web --exclude=script/configs/exclude_integration_web.yaml
227174
- name: web_benchmarks_test
228175
env:
229176
matrix:

.github/autosubmit.yml

Lines changed: 0 additions & 7 deletions
This file was deleted.

.github/workflows/scorecards-analysis.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,6 @@ jobs:
4949

5050
# Upload the results to GitHub's code scanning dashboard.
5151
- name: "Upload to code-scanning"
52-
uses: github/codeql-action/upload-sarif@29b1f65c5e92e24fe6b6647da1eaabe529cec70f # v1.0.26
52+
uses: github/codeql-action/upload-sarif@46ed16ded91731b2df79a2893d3aea8e9f03b5c4 # v1.0.26
5353
with:
5454
sarif_file: results.sarif
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,8 @@
1+
## 0.5.0+1
2+
3+
* Bumps androidx.annotation:annotation from 1.5.0 to 1.6.0.
4+
* Adds a dependency on kotlin-bom to align versions of Kotlin transitive dependencies.
5+
16
## 0.5.0
27

38
* Implements file_selector_platform_interface for Android.

packages/file_selector/file_selector_android/android/build.gradle

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,10 +38,14 @@ android {
3838
}
3939

4040
dependencies {
41-
implementation 'androidx.annotation:annotation:1.5.0'
41+
implementation 'androidx.annotation:annotation:1.6.0'
4242
testImplementation 'junit:junit:4.13.2'
4343
testImplementation 'org.mockito:mockito-inline:5.1.0'
4444
testImplementation 'androidx.test:core:1.3.0'
45+
46+
// org.jetbrains.kotlin:kotlin-bom artifact purpose is to align kotlin stdlib and related code versions.
47+
// See: https://youtrack.jetbrains.com/issue/KT-55297/kotlin-stdlib-should-declare-constraints-on-kotlin-stdlib-jdk8-and-kotlin-stdlib-jdk7
48+
implementation(platform("org.jetbrains.kotlin:kotlin-bom:1.8.10"))
4549
}
4650

4751
lintOptions {

packages/file_selector/file_selector_android/pubspec.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ name: file_selector_android
22
description: Android implementation of the file_selector package.
33
repository: https://github.com/flutter/packages/tree/main/packages/file_selector/file_selector_android
44
issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+file_selector%22
5-
version: 0.5.0
5+
version: 0.5.0+1
66

77
environment:
88
sdk: ">=2.18.0 <4.0.0"

packages/flutter_markdown/CHANGELOG.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,8 @@
1+
## 0.6.17
2+
3+
* Introduces a new `MarkdownElementBuilder.visitElementAfterWithContext()` method passing the widget `BuildContext` and
4+
the parent text's `TextStyle`.
5+
16
## 0.6.16
27

38
* Adds `tableVerticalAlignment` property to allow aligning table cells vertically.

packages/flutter_markdown/lib/src/builder.dart

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -74,6 +74,13 @@ class _InlineElement {
7474

7575
/// A delegate used by [MarkdownBuilder] to control the widgets it creates.
7676
abstract class MarkdownBuilderDelegate {
77+
/// Returns the [BuildContext] of the [MarkdownWidget].
78+
///
79+
/// The context will be passed down to the
80+
/// [MarkdownElementBuilder.visitElementBefore] method and allows elements to
81+
/// get information from the context.
82+
BuildContext get context;
83+
7784
/// Returns a gesture recognizer to use for an `a` element with the given
7885
/// text, `href` attribute, and title.
7986
GestureRecognizer createLink(String text, String? href, String title);
@@ -454,8 +461,12 @@ class MarkdownBuilder implements md.NodeVisitor {
454461
}
455462

456463
if (builders.containsKey(tag)) {
457-
final Widget? child =
458-
builders[tag]!.visitElementAfter(element, styleSheet.styles[tag]);
464+
final Widget? child = builders[tag]!.visitElementAfterWithContext(
465+
delegate.context,
466+
element,
467+
styleSheet.styles[tag],
468+
parent.style,
469+
);
459470
if (child != null) {
460471
if (current.children.isEmpty) {
461472
current.children.add(child);

0 commit comments

Comments
 (0)