Skip to content

Commit 427d7a2

Browse files
authored
Revert "chore: tidy up headless-shell hacks (#33967)" (#34659)
1 parent 902e83f commit 427d7a2

File tree

3 files changed

+25
-8
lines changed

3 files changed

+25
-8
lines changed

packages/playwright-core/browsers.json

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,12 +7,24 @@
77
"installByDefault": true,
88
"browserVersion": "133.0.6943.35"
99
},
10+
{
11+
"name": "chromium-headless-shell",
12+
"revision": "1157",
13+
"installByDefault": true,
14+
"browserVersion": "133.0.6943.35"
15+
},
1016
{
1117
"name": "chromium-tip-of-tree",
1218
"revision": "1300",
1319
"installByDefault": false,
1420
"browserVersion": "134.0.6998.0"
1521
},
22+
{
23+
"name": "chromium-tip-of-tree-headless-shell",
24+
"revision": "1300",
25+
"installByDefault": false,
26+
"browserVersion": "134.0.6998.0"
27+
},
1628
{
1729
"name": "firefox",
1830
"revision": "1474",

packages/playwright-core/src/server/registry/index.ts

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -445,14 +445,7 @@ type BrowsersJSONDescriptor = {
445445
};
446446

447447
function readDescriptors(browsersJSON: BrowsersJSON): BrowsersJSONDescriptor[] {
448-
const headlessShells: BrowsersJSON['browsers'] = [];
449-
for (const browserName of ['chromium', 'chromium-tip-of-tree']) {
450-
headlessShells.push({
451-
...browsersJSON.browsers.find(browser => browser.name === browserName)!,
452-
name: `${browserName}-headless-shell`,
453-
});
454-
}
455-
return [...browsersJSON.browsers, ...headlessShells].map(obj => {
448+
return (browsersJSON['browsers']).map(obj => {
456449
const name = obj.name;
457450
const revisionOverride = (obj.revisionOverrides || {})[hostPlatform];
458451
const revision = revisionOverride || obj.revision;

utils/roll_browser.js

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -94,6 +94,18 @@ Example:
9494
console.log('\nUpdating browser version in browsers.json...');
9595
for (const descriptor of descriptors)
9696
descriptor.browserVersion = browserVersion;
97+
98+
// 4.1 chromium-headless-shell is equal to chromium version.
99+
if (browserName === 'chromium') {
100+
const headlessShellBrowser = await browsersJSON.browsers.find(b => b.name === 'chromium-headless-shell');
101+
headlessShellBrowser.revision = revision;
102+
headlessShellBrowser.browserVersion = browserVersion;
103+
} else if (browserName === 'chromium-tip-of-tree') {
104+
const tipOfTreeBrowser = await browsersJSON.browsers.find(b => b.name === 'chromium-tip-of-tree-headless-shell');
105+
tipOfTreeBrowser.revision = revision;
106+
tipOfTreeBrowser.browserVersion = browserVersion;
107+
}
108+
97109
fs.writeFileSync(path.join(CORE_PATH, 'browsers.json'), JSON.stringify(browsersJSON, null, 2) + '\n');
98110
}
99111

0 commit comments

Comments
 (0)