New issue
Advanced search Search tips

Issue 901218 link

Starred by 1 user
Status: Verified
Owner:
smbar...@chromium.org
Closed: Nov 4
Cc:
derat@chromium.org
nya@chromium.org
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

conn.Navigate failing in tast-tests

Project Member Reported by smbar...@chromium.org, Nov 2

Comment 1 by smbar...@chromium.org, Nov 2

Cc: derat@chromium.org nya@chromium.org
This regressed in 2e6fae72aa1b3fcff361ec9efe29e0058e4aab84 "chrome: JavaScript console logger."
Project Member

Comment 2 by bugdroid1@chromium.org, Nov 3 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/tast-tests/+/54ab8857a98180699c19eaf3c596d8efdfeb64c5

commit 54ab8857a98180699c19eaf3c596d8efdfeb64c5
Author: Stephen Barber <smbarber@chromium.org>
Date: Sat Nov 03 10:39:55 2018

tast-tests: chrome: call Page.Enable in newConn

Page.Enable() must be called before events will be delivered.

BUG= chromium:901218 
TEST=vm.CrostiniStartEverything

Change-Id: Id020ea53ebb329dfe44d7be9b1584a8caecad21d
Reviewed-on: https://chromium-review.googlesource.com/1313262
Commit-Ready: Stephen Barber <smbarber@chromium.org>
Tested-by: Stephen Barber <smbarber@chromium.org>
Reviewed-by: Dan Erat <derat@chromium.org>
Reviewed-by: Shuhei Takahashi <nya@chromium.org>

[modify] https://crrev.com/54ab8857a98180699c19eaf3c596d8efdfeb64c5/src/chromiumos/tast/local/chrome/conn.go

Comment 3 by smbarber@google.com, Nov 4

Status: Verified (was: Started)
Verified vm.CrostiniStartEverything (mostly) works again on 11226.0.0.
►

Sign in to add a comment