Merge pull request #11256 from electron/fix-11244
Enable "<webview> tag found-in-page event emits when a request is made"
This commit is contained in:
commit
c1728720a7
1 changed files with 6 additions and 3 deletions
|
@ -936,9 +936,7 @@ describe('<webview> tag', function () {
|
||||||
})
|
})
|
||||||
})
|
})
|
||||||
|
|
||||||
// TODO(alexeykuzmin): Disabled during Chromium 61 upgrade.
|
describe('found-in-page event', () => {
|
||||||
// Fix it and enable.
|
|
||||||
xdescribe('found-in-page event', () => {
|
|
||||||
it('emits when a request is made', (done) => {
|
it('emits when a request is made', (done) => {
|
||||||
let requestId = null
|
let requestId = null
|
||||||
let activeMatchOrdinal = []
|
let activeMatchOrdinal = []
|
||||||
|
@ -961,6 +959,11 @@ describe('<webview> tag', function () {
|
||||||
webview.addEventListener('did-finish-load', listener2)
|
webview.addEventListener('did-finish-load', listener2)
|
||||||
webview.src = `file://${fixtures}/pages/content.html`
|
webview.src = `file://${fixtures}/pages/content.html`
|
||||||
document.body.appendChild(webview)
|
document.body.appendChild(webview)
|
||||||
|
// TODO(deepak1556): With https://codereview.chromium.org/2836973002
|
||||||
|
// focus of the webContents is required when triggering the api.
|
||||||
|
// Remove this workaround after determining the cause for
|
||||||
|
// incorrect focus.
|
||||||
|
webview.focus()
|
||||||
})
|
})
|
||||||
})
|
})
|
||||||
|
|
||||||
|
|
Loading…
Add table
Reference in a new issue