Skip to content

Commit cc773aa

Browse files
authored
Merge branch 'main' into add-browsertest-runners-to-ci
2 parents 94e7893 + d886392 commit cc773aa

File tree

9 files changed

+29
-12
lines changed

9 files changed

+29
-12
lines changed

.github/actions/on_host_tests/action.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ runs:
1414
using: "composite"
1515
steps:
1616
- name: Download Artifacts
17-
uses: actions/download-artifact@v4
17+
uses: actions/download-artifact@v5
1818
with:
1919
name: ${{ inputs.test_artifacts_key }}
2020
- name: Extract Artifacts

.github/actions/test_targets_json/action.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ runs:
44
using: "composite"
55
steps:
66
- name: Download Test Targets Json
7-
uses: actions/download-artifact@v4
7+
uses: actions/download-artifact@v5
88
with:
99
name: ${{ matrix.platform }}_test_targets_json
1010
path:

.github/actions/web_tests/action.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ runs:
1111
using: "composite"
1212
steps:
1313
- name: Download Artifacts
14-
uses: actions/download-artifact@v4
14+
uses: actions/download-artifact@v5
1515
with:
1616
name: ${{ inputs.test_artifacts_key }}
1717
- name: Extract Artifacts

.github/workflows/main.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -796,7 +796,7 @@ jobs:
796796
shell: bash
797797
- name: Download Test Results
798798
if: steps.filter.outputs.skipped != 'true'
799-
uses: actions/download-artifact@v4
799+
uses: actions/download-artifact@v5
800800
with:
801801
pattern: ${{ env.TEST_RESULTS_KEY }}*
802802
path: results/
@@ -848,7 +848,7 @@ jobs:
848848
filter: blob:none
849849
sparse-checkout: ${{ env.CI_ESSENTIALS }}
850850
- name: Download Test Results
851-
uses: actions/download-artifact@v4
851+
uses: actions/download-artifact@v5
852852
with:
853853
pattern: ${{ env.TEST_RESULTS_KEY }}*
854854
path: results/

.github/workflows/sync_chromium_branches.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,11 +47,13 @@ jobs:
4747
git remote add upstream https://chromium.googlesource.com/chromium/src
4848
git fetch --depth=1 upstream "refs/branch-heads/${{ matrix.branch.branch_num }}:refs/remotes/branch/${{ matrix.branch.branch_num }}"
4949
- name: Create missing Chromium branch
50+
id: create_missing_branch
5051
if: steps.checkout.outcome == 'failure'
5152
run: |
5253
git checkout "refs/remotes/branch/${{ matrix.branch.branch_num }}"
5354
ORIGINAL_TREE_HASH=$(git rev-parse "HEAD^{tree}")
5455
ORIGINAL_COMMIT_HASH=$(git rev-parse HEAD)
56+
echo "original_commit_hash=${ORIGINAL_COMMIT_HASH}" >> "$GITHUB_OUTPUT"
5557
NEW_ROOT_COMMIT_HASH=$(git commit-tree "${ORIGINAL_TREE_HASH}" -m "First commit for ${{ matrix.branch.milestone }}.")
5658
git checkout -b "chromium/${{ matrix.branch.milestone }}" "${NEW_ROOT_COMMIT_HASH}"
5759
git commit --amend --no-edit -c "${ORIGINAL_COMMIT_HASH}"
@@ -66,6 +68,12 @@ jobs:
6668
echo "Applying diff..."
6769
git apply --index chromium_diff.patch
6870
git commit -m "Rebase refresh on $(date +'%Y-%m-%d') for ${{ matrix.branch.milestone }}."
71+
72+
echo "LASTCHANGE=${{ steps.create_missing_branch.outputs.original_commit_hash }}" > cobalt/LASTCHANGE.chromium
73+
echo "LASTCHANGE_YEAR=$(date +'%Y')" >> cobalt/LASTCHANGE.chromium
74+
git add cobalt/LASTCHANGE.chromium
75+
git commit -m "Update LASTCHANGE.chromium for ${{ matrix.branch.milestone }}."
76+
6977
git push --force origin "chromium/${{ matrix.branch.milestone }}"
7078
DIFF_COMMIT_HASH=$(git rev-parse HEAD)
7179
echo "main_cherry_pick=true" >> "$GITHUB_OUTPUT"

DEPS

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -308,7 +308,7 @@ vars = {
308308
# Three lines of non-changing comments so that
309309
# the commit queue can handle CLs rolling V8
310310
# and whatever else without interference from each other.
311-
'v8_revision': '32952ed473705d2ab6127074f4e79d5127ef4145',
311+
'v8_revision': 'b40c5631f5d789a9370067e0f492795c0b05bbef',
312312
# Three lines of non-changing comments so that
313313
# the commit queue can handle CLs rolling ANGLE
314314
# and whatever else without interference from each other.

build/util/lastchange.gni

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,5 +12,9 @@ declare_args() {
1212
if (use_dummy_lastchange) {
1313
lastchange_file = "//build/util/LASTCHANGE.dummy"
1414
} else {
15+
if (is_cobalt) {
16+
lastchange_file = "//cobalt/LASTCHANGE.chromium"
17+
} else {
1518
lastchange_file = "//build/util/LASTCHANGE"
19+
}
1620
}

cobalt/LASTCHANGE.chromium

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
LASTCHANGE=f14590bbe48717072eb271d6c5ff6b7e88135a1d
2+
LASTCHANGE_YEAR=2026

cobalt/shell/browser/h5vcc_scheme_url_loader_factory.cc

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -215,14 +215,18 @@ class H5vccSchemeURLLoader : public network::mojom::URLLoader {
215215
mime_type_ = kMimeTypeVideoWebM;
216216
}
217217

218+
// Splash caching only serves the png and webm resources defined in
219+
// the resource map.
218220
const bool supports_splash_caching =
219221
(mime_type_ == kMimeTypeImagePng || mime_type_ == kMimeTypeVideoWebM) &&
220-
browser_context_;
222+
base::Contains(resource_map, key);
221223

222-
// Specify the built-in video/png if the cache is unavailable.
224+
// For png/webm, override resource_key with "fallback" query param.
225+
// If requested key is not found in cache, loader returns the fallback
226+
// resource.
223227
std::string fallback;
224-
if (net::GetValueForKeyInQuery(url_, "fallback", &fallback) &&
225-
supports_splash_caching) {
228+
if (supports_splash_caching &&
229+
net::GetValueForKeyInQuery(url_, "fallback", &fallback)) {
226230
LOG(INFO) << "Fallback splash: " << fallback;
227231
resource_key = std::move(fallback);
228232
}
@@ -235,8 +239,7 @@ class H5vccSchemeURLLoader : public network::mojom::URLLoader {
235239
LOG(WARNING) << "Resource not found: " << resource_key;
236240
}
237241

238-
// Only attempt to read files defined in the resource map from cache.
239-
if (supports_splash_caching && base::Contains(resource_map, key)) {
242+
if (supports_splash_caching && browser_context_) {
240243
ReadSplashCache(key);
241244
return;
242245
}

0 commit comments

Comments
 (0)