New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 618326 link

Starred by 1 user
Status: Assigned
Owner:
xiy...@chromium.org
Cc:
█ dzhioev@chromium.org
dchan@chromium.org
█ krishna...@chromium.org
jdufault@chromium.org
achuith@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

KioskTest.LaunchAppNetworkDownConfigureNotAllowed fails

Project Member Reported by xidac...@chromium.org, Jun 8 2016

Comment 1 by xidac...@chromium.org, Jun 8 2016

Components: Tests>Flaky UI>Browser
Labels: OS-Chrome
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 8 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b9f57976622ff7f567255e4a751b0ec3ecc8665d

commit b9f57976622ff7f567255e4a751b0ec3ecc8665d
Author: xidachen <xidachen@chromium.org>
Date: Wed Jun 08 16:02:52 2016

Mark KioskTest.LaunchAppNetworkDownConfigureNotAllowed failure on Chrome OS

BUG=618326
TBR=jdufault@chromium.org

Review-Url: https://codereview.chromium.org/2048713004
Cr-Commit-Position: refs/heads/master@{#398576}

[modify] https://crrev.com/b9f57976622ff7f567255e4a751b0ec3ecc8665d/chrome/browser/chromeos/login/kiosk_browsertest.cc

Comment 3 by jdufault@chromium.org, Jun 8 2016

Cc: jdufault@chromium.org
Owner: xiy...@chromium.org

Comment 4 by xiy...@chromium.org, Jun 8 2016 

Looks like we should move |current_screen_| update before notifying observers [1]. I wonder how the test ever worked....


[1] https://cs.chromium.org/chromium/src/chrome/browser/ui/webui/chromeos/login/oobe_ui.cc?rcl=1465384673&l=613
Project Member

Comment 5 by bugdroid1@chromium.org, Jul 1 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5d8335a1118427d7a496cdf976433dd92169bcf1

commit 5d8335a1118427d7a496cdf976433dd92169bcf1
Author: xiyuan <xiyuan@chromium.org>
Date: Fri Jul 01 20:06:36 2016

kiosk: Fix OobeScreenWaiter get wrong current screen

The real test flakyness on splash screen wait time out should
be fixed with https://codereview.chromium.org/2107353002/

BUG=618326

Review-Url: https://codereview.chromium.org/2108843002
Cr-Commit-Position: refs/heads/master@{#403508}

[modify] https://crrev.com/5d8335a1118427d7a496cdf976433dd92169bcf1/chrome/browser/chromeos/login/kiosk_browsertest.cc
[modify] https://crrev.com/5d8335a1118427d7a496cdf976433dd92169bcf1/chrome/browser/ui/webui/chromeos/login/oobe_ui.cc

Comment 6 by trapti@chromium.org, Sep 7 2016

Cc: krishna...@chromium.org dchan@chromium.org scunning...@chromium.org
Test Case still missing in M53 
 
Columns
M	ChromeOS	Chrome	ARC	Type	Channel
53	8530.78.0	53.0.2785.93	3241884	release	beta

Comment 7 by scunning...@chromium.org, Nov 11 2016

Cc: -scunning...@chromium.org

Comment 8 by robliao@chromium.org, Sep 18 

Is there additional work that needs to be done here?

Comment 9 by robliao@chromium.org, Sep 19

Labels: Hotlist-UITriageDeferred
►

Sign in to add a comment