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

Unified Diff: content/browser/media/session/media_session_impl_visibility_browsertest.cc

Issue 2681863005: [Video] MediaSession API event handlers can resume background video. (Closed)
Patch Set: Tweaked the logic in HTMLME a bit Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/media/session/media_session_impl_visibility_browsertest.cc
diff --git a/content/browser/media/session/media_session_impl_visibility_browsertest.cc b/content/browser/media/session/media_session_impl_visibility_browsertest.cc
index 7703287553638c793a347919ef459b3d370a52b4..774183359402f0c4ea3d8c115c56a6061fb9a87a 100644
--- a/content/browser/media/session/media_session_impl_visibility_browsertest.cc
+++ b/content/browser/media/session/media_session_impl_visibility_browsertest.cc
@@ -164,7 +164,7 @@ class MediaSessionImplVisibilityBrowserTest
if (state_before_hide == state_after_hide) {
LOG(INFO) << "Waiting for 1 second and check session state is unchanged";
Wait(base::TimeDelta::FromSeconds(1));
- ASSERT_EQ(media_session_->audio_focus_state_, state_after_hide);
+ ASSERT_EQ(state_after_hide, media_session_->audio_focus_state_);
} else {
LOG(INFO) << "Waiting for Session to change";
WaitForMediaSessionState(state_after_hide);

Powered by Google App Engine
This is Rietveld 408576698