[Presentation API] Handle multiple Presentation URLs in PresentationRequest::getAvailability()

content/renderer/presentation/presentation_dispatcher_unittest.cc

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <memory>
 #include <utility>
 
 #include "base/run_loop.h"
 #include "content/public/test/test_browser_thread_bundle.h"
 #include "content/renderer/presentation/presentation_dispatcher.h"
@@ skipping 29 matching lines @@
 // - Messages received
 // - Discarding queued messages when the frame navigates
 // - Screen availability not supported
 // - Default presentation starting
 
 namespace content {
 
 class MockPresentationAvailabilityObserver
     : public WebPresentationAvailabilityObserver {
  public:
-  explicit MockPresentationAvailabilityObserver(const WebVector<WebURL>& urls)
-      : urls_(urls) {}
+  explicit MockPresentationAvailabilityObserver(const std::vector<GURL>& gurls)

[mark a. foltz, 2017/01/17 21:04:24]

Nit: s/gurls/urls/ for consistency

[zhaobin, 2017/01/18 03:38:57]

Done.

+      : urls_(gurls) {}
   ~MockPresentationAvailabilityObserver() override {}
 
   MOCK_METHOD1(availabilityChanged, void(bool is_available));
   const WebVector<WebURL>& urls() const override { return urls_; }
 
  private:
   const WebVector<WebURL> urls_;
 };
 
 class MockPresentationService : public PresentationService {
@@ skipping 36 matching lines @@
   // *Internal method is to work around lack of support for move-only types in
   // GMock.
   void ListenForConnectionMessages(
       PresentationSessionInfoPtr session_info) override {
     ListenForConnectionMessagesInternal(session_info.get());
   }
   MOCK_METHOD1(ListenForConnectionMessagesInternal,
                void(PresentationSessionInfo* session_info));
 };
 
+class MockPresentationAvailabilityCallbacks
+    : public blink::WebCallbacks<bool, const blink::WebPresentationError&> {
+ public:
+  MOCK_METHOD1(onSuccess, void(bool value));
+  MOCK_METHOD1(onError, void(const blink::WebPresentationError&));
+};
+
 class TestWebPresentationConnectionCallback
     : public WebPresentationConnectionCallback {
  public:
   TestWebPresentationConnectionCallback(WebURL url, WebString id)
       : url_(url), id_(id), callback_called_(false) {}
   ~TestWebPresentationConnectionCallback() override {
     EXPECT_TRUE(callback_called_);
   }
 
   void onSuccess(const WebPresentationSessionInfo& info) override {
@@ skipping 47 matching lines @@
           mojo::MakeRequest(&presentation_service_));
     }
   }
 
   MockPresentationService* mock_presentation_service_;
   std::unique_ptr<mojo::Binding<PresentationService>> mock_binding_;
 };
 
 class PresentationDispatcherTest : public ::testing::Test {
  public:
+  enum class URLState { Available, Unavailable, Unsupported, Unknown };
+
   PresentationDispatcherTest()
       : gurl1_(GURL("https://www.example.com/1.html")),
         gurl2_(GURL("https://www.example.com/2.html")),
-        gurls_({gurl1_, gurl2_}),
+        gurl3_(GURL("https://www.example.com/3.html")),
+        gurl4_(GURL("https://www.example.com/4.html")),
+        gurls_({gurl1_, gurl2_, gurl3_, gurl4_}),
         url1_(WebURL(gurl1_)),
         url2_(WebURL(gurl2_)),
+        url3_(WebURL(gurl3_)),
+        url4_(WebURL(gurl4_)),
         urls_(WebVector<WebURL>(gurls_)),
         presentation_id_(WebString::fromUTF8("test-id")),
         array_buffer_(WebArrayBuffer::create(4, 1)),
-        observer_(urls_),
+        observer_(gurls_),
+        mock_observer1_({gurl1_, gurl2_, gurl3_}),
+        mock_observer2_({gurl2_, gurl3_, gurl4_}),
+        mock_observer3_({gurl2_, gurl3_}),
+        mock_observers_({&mock_observer1_, &mock_observer2_, &mock_observer3_}),
         dispatcher_(&presentation_service_) {}
+
   ~PresentationDispatcherTest() override {}
 
   void SetUp() override {
     // Set some test data.
     *array_buffer_data() = 42;
   }
 
   uint8_t* array_buffer_data() {
     return static_cast<uint8_t*>(array_buffer_.data());
   }
 
+  void ChangeURLState(const GURL& url, URLState state) {
+    switch (state) {
+      case URLState::Available:
+        dispatcher_.OnScreenAvailabilityUpdated(url, true);
+        break;
+      case URLState::Unavailable:
+        dispatcher_.OnScreenAvailabilityUpdated(url, false);
+        break;
+      case URLState::Unsupported:
+        dispatcher_.OnScreenAvailabilityNotSupported(url);
+        break;
+      case URLState::Unknown:
+        break;
+    }
+  }
+
+  // Tests that PresenationService is called for getAvailability(urls), after
+  // |urls| change state to |states|.
+  void TestGetAvailability(
+      const std::vector<GURL>& urls,
+      const std::vector<URLState>& states,
+      MockPresentationAvailabilityCallbacks* mock_callback) {
+    DCHECK_EQ(urls.size(), states.size());
+
+    for (const auto& url : urls) {
+      EXPECT_CALL(presentation_service_, ListenForScreenAvailability(url))
+          .Times(1);
+    }
+
+    base::RunLoop run_loop;
+    // This function takes ownership of |mock_callback|.

[mark a. foltz, 2017/01/17 21:04:24]

This comment probably belongs with the function-level comment above.

[zhaobin, 2017/01/18 03:38:57]

Done.

+    client()->getAvailability(urls, base::WrapUnique(mock_callback));
+    for (size_t i = 0; i < urls.size(); i++)
+      ChangeURLState(urls[i], states[i]);
+
+    run_loop.RunUntilIdle();
+  }
+
+  blink::WebPresentationClient* client() { return &dispatcher_; }
+
  protected:
   const GURL gurl1_;
   const GURL gurl2_;
+  const GURL gurl3_;
+  const GURL gurl4_;
   const std::vector<GURL> gurls_;
   const WebURL url1_;
   const WebURL url2_;
+  const WebURL url3_;
+  const WebURL url4_;
   const WebVector<WebURL> urls_;
   const WebString presentation_id_;
   const WebArrayBuffer array_buffer_;
   MockPresentationAvailabilityObserver observer_;
+  MockPresentationAvailabilityObserver mock_observer1_;
+  MockPresentationAvailabilityObserver mock_observer2_;
+  MockPresentationAvailabilityObserver mock_observer3_;
+  std::vector<MockPresentationAvailabilityObserver*> mock_observers_;
+
   MockPresentationService presentation_service_;
   TestPresentationDispatcher dispatcher_;
 
  private:
   content::TestBrowserThreadBundle thread_bundle_;
 };
 
 TEST_F(PresentationDispatcherTest, TestStartSession) {
   base::RunLoop run_loop;
 
   EXPECT_CALL(presentation_service_, StartSession(gurls_, _))
       .WillOnce(Invoke([this](
           const std::vector<GURL>& presentation_urls,
           const PresentationService::StartSessionCallback& callback) {
         PresentationSessionInfoPtr session_info(PresentationSessionInfo::New());
         session_info->url = gurl1_;
         session_info->id = presentation_id_.utf8();
         callback.Run(std::move(session_info), PresentationErrorPtr());
       }));
-  dispatcher_.startSession(
+  client()->startSession(
       urls_, base::MakeUnique<TestWebPresentationConnectionCallback>(
                  url1_, presentation_id_));
   run_loop.RunUntilIdle();
 }
 
 TEST_F(PresentationDispatcherTest, TestStartSessionError) {
   WebString error_message = WebString::fromUTF8("Test error message");
   base::RunLoop run_loop;
 
   EXPECT_CALL(presentation_service_, StartSession(gurls_, _))
       .WillOnce(Invoke([this, &error_message](
           const std::vector<GURL>& presentation_urls,
           const PresentationService::StartSessionCallback& callback) {
         PresentationErrorPtr error(PresentationError::New());
         error->error_type = PresentationErrorType::NO_AVAILABLE_SCREENS;
         error->message = error_message.utf8();
         callback.Run(PresentationSessionInfoPtr(), std::move(error));
       }));
-  dispatcher_.startSession(
+  client()->startSession(
       urls_,
       base::MakeUnique<TestWebPresentationConnectionErrorCallback>(
           WebPresentationError::ErrorTypeNoAvailableScreens, error_message));
   run_loop.RunUntilIdle();
 }
 
 TEST_F(PresentationDispatcherTest, TestJoinSessionError) {
   WebString error_message = WebString::fromUTF8("Test error message");
   base::RunLoop run_loop;
 
@@ skipping 109 matching lines @@
 TEST_F(PresentationDispatcherTest, TestTerminateSession) {
   base::RunLoop run_loop;
   EXPECT_CALL(presentation_service_,
               Terminate(gurl1_, presentation_id_.utf8()));
   dispatcher_.terminateSession(url1_, presentation_id_);
   run_loop.RunUntilIdle();
 }
 
 TEST_F(PresentationDispatcherTest, TestListenForScreenAvailability) {
   base::RunLoop run_loop1;
-  EXPECT_CALL(presentation_service_, ListenForScreenAvailability(gurl1_));
+  for (const auto& gurl : gurls_)
+    EXPECT_CALL(presentation_service_, ListenForScreenAvailability(gurl));
   dispatcher_.getAvailability(
       urls_, base::MakeUnique<WebPresentationAvailabilityCallbacks>());
   dispatcher_.OnScreenAvailabilityUpdated(url1_, true);
   run_loop1.RunUntilIdle();
 
   base::RunLoop run_loop2;
-  EXPECT_CALL(presentation_service_, ListenForScreenAvailability(gurl1_));
-  dispatcher_.startListening(&observer_);
+  client()->startListening(&observer_);
   run_loop2.RunUntilIdle();
 
   base::RunLoop run_loop3;
   EXPECT_CALL(observer_, availabilityChanged(false));
   dispatcher_.OnScreenAvailabilityUpdated(url1_, false);
   EXPECT_CALL(observer_, availabilityChanged(true));
   dispatcher_.OnScreenAvailabilityUpdated(url1_, true);
-  EXPECT_CALL(presentation_service_,
-              StopListeningForScreenAvailability(gurl1_));
-  dispatcher_.stopListening(&observer_);
+  for (const auto& gurl : gurls_) {
+    EXPECT_CALL(presentation_service_,
+                StopListeningForScreenAvailability(gurl));
+  }
+  client()->stopListening(&observer_);
   run_loop3.RunUntilIdle();
 
   // After stopListening(), |observer_| should no longer be notified.
   base::RunLoop run_loop4;
   EXPECT_CALL(observer_, availabilityChanged(false)).Times(0);
   dispatcher_.OnScreenAvailabilityUpdated(url1_, false);
   run_loop4.RunUntilIdle();
 }
 
 TEST_F(PresentationDispatcherTest, TestSetDefaultPresentationUrls) {
   base::RunLoop run_loop;
   EXPECT_CALL(presentation_service_, SetDefaultPresentationUrls(gurls_));
   dispatcher_.setDefaultPresentationUrls(urls_);
   run_loop.RunUntilIdle();
 }
 
+TEST_F(PresentationDispatcherTest, GetAvailabilityOneUrlNoAvailabilityChange) {
+  auto* mock_callback =
+      new testing::StrictMock<MockPresentationAvailabilityCallbacks>();
+  TestGetAvailability({url1_}, {URLState::Unknown}, mock_callback);
+}
+
+TEST_F(PresentationDispatcherTest, GetAvailabilityOneUrlBecomesAvailable) {
+  auto* mock_callback = new MockPresentationAvailabilityCallbacks();
+  EXPECT_CALL(*mock_callback, onSuccess(true));
+
+  TestGetAvailability({url1_}, {URLState::Available}, mock_callback);
+}
+
+TEST_F(PresentationDispatcherTest, GetAvailabilityOneUrlBecomesUnavailable) {
+  auto* mock_callback = new MockPresentationAvailabilityCallbacks();
+  EXPECT_CALL(*mock_callback, onSuccess(false));
+
+  TestGetAvailability({url1_}, {URLState::Unavailable}, mock_callback);
+}
+
+TEST_F(PresentationDispatcherTest, GetAvailabilityOneUrlBecomesNotSupported) {
+  auto* mock_callback = new MockPresentationAvailabilityCallbacks();
+  EXPECT_CALL(*mock_callback, onError(_));
+  EXPECT_CALL(presentation_service_,
+              StopListeningForScreenAvailability(gurl1_));
+
+  TestGetAvailability({url1_}, {URLState::Unsupported}, mock_callback);
+}
+
+TEST_F(PresentationDispatcherTest,
+       GetAvailabilityMultipleUrlsAllBecomesAvailable) {
+  auto* mock_callback = new MockPresentationAvailabilityCallbacks();
+  EXPECT_CALL(*mock_callback, onSuccess(true)).Times(1);
+
+  TestGetAvailability({url1_, url2_},
+                      {URLState::Available, URLState::Available},
+                      mock_callback);
+}
+
+TEST_F(PresentationDispatcherTest,
+       GetAvailabilityMultipleUrlsAllBecomesUnavailable) {
+  auto* mock_callback = new MockPresentationAvailabilityCallbacks();
+  EXPECT_CALL(*mock_callback, onSuccess(false)).Times(1);
+
+  TestGetAvailability({url1_, url2_},
+                      {URLState::Unavailable, URLState::Unavailable},
+                      mock_callback);
+}
+
+TEST_F(PresentationDispatcherTest,
+       GetAvailabilityMultipleUrlsAllBecomesUnsupported) {
+  auto* mock_callback = new MockPresentationAvailabilityCallbacks();
+  EXPECT_CALL(*mock_callback, onError(_)).Times(1);
+  EXPECT_CALL(presentation_service_,
+              StopListeningForScreenAvailability(gurl1_));
+  EXPECT_CALL(presentation_service_,
+              StopListeningForScreenAvailability(gurl2_));
+
+  TestGetAvailability({url1_, url2_},
+                      {URLState::Unsupported, URLState::Unsupported},
+                      mock_callback);
+}
+
+TEST_F(PresentationDispatcherTest,
+       GetAvailabilityReturnsDirectlyForAlreadyListeningUrls) {
+  auto* mock_callback_1 = new MockPresentationAvailabilityCallbacks();
+  EXPECT_CALL(*mock_callback_1, onSuccess(false)).Times(1);
+
+  std::vector<URLState> state_seq = {URLState::Unavailable, URLState::Available,
+                                     URLState::Unavailable};
+  TestGetAvailability({url1_, url2_, url3_}, state_seq, mock_callback_1);
+
+  auto* mock_callback_2 = new MockPresentationAvailabilityCallbacks();
+  EXPECT_CALL(*mock_callback_2, onSuccess(true)).Times(1);
+
+  base::RunLoop run_loop;
+  client()->getAvailability(mock_observer3_.urls(),
+                            base::WrapUnique(mock_callback_2));
+  run_loop.RunUntilIdle();
+}
+
+TEST_F(PresentationDispatcherTest, StartListeningListenToEachURLOnce) {
+  for (const auto& gurl : gurls_) {
+    EXPECT_CALL(presentation_service_, ListenForScreenAvailability(gurl))
+        .Times(1);
+  }
+
+  base::RunLoop run_loop;
+  for (auto* mock_observer : mock_observers_) {
+    client()->getAvailability(
+        mock_observer->urls(),
+        base::MakeUnique<WebPresentationAvailabilityCallbacks>());
+    client()->startListening(mock_observer);
+  }
+  run_loop.RunUntilIdle();
+}
+
+TEST_F(PresentationDispatcherTest, StopListeningListenToEachURLOnce) {
+  for (const auto& gurl : gurls_) {
+    EXPECT_CALL(presentation_service_, ListenForScreenAvailability(gurl))
+        .Times(1);
+    EXPECT_CALL(presentation_service_, StopListeningForScreenAvailability(gurl))
+        .Times(1);
+  }
+
+  // Set up |availability_set_| and |listening_status_|
+  base::RunLoop run_loop;
+  for (auto* mock_observer : mock_observers_) {
+    client()->getAvailability(
+        mock_observer->urls(),
+        base::MakeUnique<WebPresentationAvailabilityCallbacks>());
+  }
+  // Clean up callbacks.
+  ChangeURLState(gurl2_, URLState::Unavailable);
+
+  for (auto* mock_observer : mock_observers_)
+    client()->startListening(mock_observer);
+  for (auto* mock_observer : mock_observers_)
+    client()->stopListening(mock_observer);
+
+  run_loop.RunUntilIdle();
+}
+
+TEST_F(PresentationDispatcherTest,
+       StopListeningDoesNotStopIfURLListenedByOthers) {
+  for (const auto& gurl : gurls_) {
+    EXPECT_CALL(presentation_service_, ListenForScreenAvailability(gurl))
+        .Times(1);
+  }
+  EXPECT_CALL(presentation_service_, StopListeningForScreenAvailability(gurl1_))
+      .Times(1);
+  EXPECT_CALL(presentation_service_, StopListeningForScreenAvailability(gurl2_))
+      .Times(0);
+  EXPECT_CALL(presentation_service_, StopListeningForScreenAvailability(gurl3_))
+      .Times(0);
+
+  // Set up |availability_set_| and |listening_status_|
+  base::RunLoop run_loop;
+  for (auto& mock_observer : mock_observers_) {
+    client()->getAvailability(
+        mock_observer->urls(),
+        base::MakeUnique<WebPresentationAvailabilityCallbacks>());
+  }
+
+  for (auto* mock_observer : mock_observers_)
+    client()->startListening(mock_observer);
+
+  EXPECT_CALL(mock_observer1_, availabilityChanged(false));
+  EXPECT_CALL(mock_observer2_, availabilityChanged(false));
+
+  // Clean up callbacks.
+  ChangeURLState(gurl2_, URLState::Unavailable);
+  client()->stopListening(&mock_observer1_);
+  run_loop.RunUntilIdle();
+}
+
+TEST_F(PresentationDispatcherTest,
+       OnScreenAvailabilityUpdatedInvokesAvailabilityChanged) {
+  for (const auto& gurl : gurls_) {
+    EXPECT_CALL(presentation_service_, ListenForScreenAvailability(gurl))
+        .Times(1);
+  }
+  EXPECT_CALL(mock_observer1_, availabilityChanged(true));
+
+  base::RunLoop run_loop;
+  for (auto* mock_observer : mock_observers_) {
+    client()->getAvailability(
+        mock_observer->urls(),
+        base::MakeUnique<WebPresentationAvailabilityCallbacks>());
+    client()->startListening(mock_observer);
+  }
+
+  ChangeURLState(gurl1_, URLState::Available);
+  run_loop.RunUntilIdle();
+
+  EXPECT_CALL(mock_observer1_, availabilityChanged(false));
+
+  base::RunLoop run_loop_2;
+  ChangeURLState(gurl1_, URLState::Unavailable);
+  run_loop_2.RunUntilIdle();
+}
+
+TEST_F(PresentationDispatcherTest,
+       OnScreenAvailabilityUpdatedInvokesMultipleAvailabilityChanged) {
+  for (const auto& gurl : gurls_) {
+    EXPECT_CALL(presentation_service_, ListenForScreenAvailability(gurl))
+        .Times(1);
+  }
+  for (auto* mock_observer : mock_observers_)
+    EXPECT_CALL(*mock_observer, availabilityChanged(true));
+
+  base::RunLoop run_loop;
+  for (auto* mock_observer : mock_observers_) {
+    client()->getAvailability(
+        mock_observer->urls(),
+        base::MakeUnique<WebPresentationAvailabilityCallbacks>());
+    client()->startListening(mock_observer);
+  }
+
+  ChangeURLState(gurl2_, URLState::Available);
+  run_loop.RunUntilIdle();
+
+  for (auto* mock_observer : mock_observers_)
+    EXPECT_CALL(*mock_observer, availabilityChanged(false));
+
+  base::RunLoop run_loop_2;
+  ChangeURLState(gurl2_, URLState::Unavailable);
+  run_loop_2.RunUntilIdle();
+}
+
 }  // namespace content