Roll chromium to r491334 (#187)

This patch rolls chromium to r491334

This revision includes @pavelfeldman changes to make dialogs
work in headless.
This commit is contained in:
Andrey Lushnikov 2017-08-02 13:12:44 -07:00 committed by GitHub
parent 5d647bf1fa
commit dbe8660082
3 changed files with 6 additions and 9 deletions

View File

@ -29,7 +29,7 @@
"ws": "^3.0.0" "ws": "^3.0.0"
}, },
"puppeteer": { "puppeteer": {
"chromium_revision": "490379" "chromium_revision": "491334"
}, },
"devDependencies": { "devDependencies": {
"commonmark": "^0.27.0", "commonmark": "^0.27.0",

View File

@ -755,14 +755,14 @@ describe('Page', function() {
}); });
describe('Page.Events.Dialog', function() { describe('Page.Events.Dialog', function() {
it('should fire', function(done) { it('should fire', SX(async function() {
page.on('dialog', dialog => { page.on('dialog', dialog => {
expect(dialog.type).toBe('alert'); expect(dialog.type).toBe('alert');
expect(dialog.message()).toBe('yo'); expect(dialog.message()).toBe('yo');
done(); dialog.accept();
}); });
page.evaluate(() => alert('yo')); await page.evaluate(() => alert('yo'));
}); }));
// TODO Enable this when crbug.com/718235 is fixed. // TODO Enable this when crbug.com/718235 is fixed.
(headless ? xit : it)('should allow accepting prompts', SX(async function(done) { (headless ? xit : it)('should allow accepting prompts', SX(async function(done) {
page.on('dialog', dialog => { page.on('dialog', dialog => {

View File

@ -30,7 +30,4 @@ test(function () {
page.onConfirm = undefined; page.onConfirm = undefined;
assert_equals(page.onConfirm, undefined); assert_equals(page.onConfirm, undefined);
}, "page.onConfirm", { }, "page.onConfirm");
/* @see crbug.com/718235 */
expected_fail: true
});