Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(151)

Issue 5054004: chromeos: Fix screen locker tests. (Closed)

Created:
10 years, 1 month ago by Daniel Erat
Modified:
9 years, 7 months ago
Reviewers:
oshima
CC:
chromium-reviews, ben+cc_chromium.org, nkostylev+cc_chromium.org, davemoore+watch_chromium.org, Paweł Hajdan Jr., Scott Byer
Visibility:
Public.

Description

chromeos: Fix screen locker tests. These appear to have been following the pattern of: ScreenLocker::Show(); ... ui_test_utils::WaitForNotification( NotificationType::SCREEN_LOCK_STATE_CHANGED); I believe that this hangs until the test times out if the notification is sent before we return from Show() (since we won't yet be listening for the notification at the time). I'm changing them to: ScreenLocker::Show(); ... if (!chromeos::ScreenLocker::GetTester()->IsLocked()) ui_test_utils::WaitForNotification( NotificationType::SCREEN_LOCK_STATE_CHANGED); The old version looks like it started failing with my r66287, which I don't understand -- I thought that we were already generating the notification immediately as of r66131, which fixed a bug where the initial attempt to grab the pointer and keyboard always failed (thus ensuring that the notifications wouldn't be sent until the next time that the event loop was run). BUG=none TEST=all screen locker tests passed on my local machine Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=66340 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=66464

Patch Set 1 #

Patch Set 2 : uncomment formerly-failing expectation #

Unified diffs Side-by-side diffs Delta from patch set Stats (+18 lines, -8 lines) Patch
M chrome/browser/chromeos/login/screen_locker_browsertest.cc View 1 7 chunks +18 lines, -8 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Daniel Erat
10 years, 1 month ago (2010-11-16 23:19:16 UTC) #1
oshima
10 years, 1 month ago (2010-11-16 23:31:59 UTC) #2
oshima
lgtm
10 years, 1 month ago (2010-11-16 23:33:30 UTC) #3
oshima
10 years, 1 month ago (2010-11-17 17:58:22 UTC) #4
i'm going to reland this too.

Powered by Google App Engine
This is Rietveld 408576698