New issue
Advanced search Search tips

Issue 814019 link

Starred by 2 users

Issue metadata

Status: Untriaged
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

mash: StatusAreaWidgetFocusTest.FocusOutObserver fails

Project Member Reported by jamescook@chromium.org, Feb 21 2018

Issue description

ash_unittests --mash --gtest_filter=StatusAreaWidget*

[ RUN      ] StatusAreaWidgetFocusTest.FocusOutObserver
../../ash/system/status_area_widget_unittest.cc:170: Failure
Expected equality of these values:
  status->system_tray()
    Which is: 0x2915fd114900
  focus_manager->GetFocusedView()
    Which is: 0x2915fce58000
../../ash/system/status_area_widget_unittest.cc:171: Failure
Expected equality of these values:
  1
  test_observer_->focus_out_count()
    Which is: 0
../../ash/system/status_area_widget_unittest.cc:178: Failure
Expected equality of these values:
  1
  test_observer_->focus_out_count()
    Which is: 0
../../ash/system/status_area_widget_unittest.cc:179: Failure
Expected equality of these values:
  1
  test_observer_->reverse_focus_out_count()
    Which is: 0

Focus isn't moving between views/widgets as expected.

Maybe related to the views-based lock screen switch?

(These tests were accidentally skipped on the waterfall for a couple months.)

 
Components: -Internals>MUS Internals>Services>WindowService
Components: -Internals>Services>WindowService Internals>Services>Ash
Labels: -Proj-Mustash-Mash
Project Member

Comment 3 by bugdroid1@chromium.org, Jun 12 2018

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

commit 37446a1d7863f1ac0a2b66f4e70c2a2ef0595a1e
Author: Tetsui Ohkubo <tetsui@chromium.org>
Date: Tue Jun 12 03:02:08 2018

Disable FocusOutObserverUnified in Mash.

StatusAreaWidgetFocusTest.FocusOutObserver is already disabled on Mash
ash_unittests because of focus traversal problem.
FocusOutObserverUnified is UnifiedSystemTray version of the test.

TBR=jamescook@chromium.org
TEST=none
BUG=814019

Change-Id: I206aeed174e0432415e73dd7ab628afab218c6d2
Reviewed-on: https://chromium-review.googlesource.com/1096581
Reviewed-by: Tetsui Ohkubo <tetsui@chromium.org>
Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org>
Cr-Commit-Position: refs/heads/master@{#566288}
[modify] https://crrev.com/37446a1d7863f1ac0a2b66f4e70c2a2ef0595a1e/testing/buildbot/filters/mash.ash_unittests.filter

Sign in to add a comment