[Presentation API] Add layout test for connection.close() and fix test failures

content/renderer/presentation/presentation_connection_proxy.cc

Index: content/renderer/presentation/presentation_connection_proxy.cc
diff --git a/content/renderer/presentation/presentation_connection_proxy.cc b/content/renderer/presentation/presentation_connection_proxy.cc
index aebff14183ac5a7de998df1213d7c7f8da41c179..6378d3da66b4bdba382511774cf7bf20da8d1ede 100644
--- a/content/renderer/presentation/presentation_connection_proxy.cc
+++ b/content/renderer/presentation/presentation_connection_proxy.cc
@@ -67,8 +67,8 @@ void PresentationConnectionProxy::DidChangeState(
     source_connection_->didChangeState(
         blink::WebPresentationConnectionState::Connected);
   } else if (state == content::PRESENTATION_CONNECTION_STATE_CLOSED) {
-    source_connection_->didChangeState(
-        blink::WebPresentationConnectionState::Closed);
+    source_connection_->didClose(
+        blink::WebPresentationConnectionCloseReason::Closed, "");
   } else {
     NOTREACHED();
   }
@@ -76,8 +76,8 @@ void PresentationConnectionProxy::DidChangeState(
 
 void PresentationConnectionProxy::OnClose() {
   DCHECK(target_connection_ptr_);
-  source_connection_->didChangeState(
-      blink::WebPresentationConnectionState::Closed);
+  source_connection_->didClose(
+      blink::WebPresentationConnectionCloseReason::Closed, "");
   target_connection_ptr_->DidChangeState(

[mark a. foltz, 2017/03/06 18:57:55]

Part of the connection refactoring should better align the Web* and the mojo
APIs for PresentationConnection.   For example, this could be
target_connection_ptr_->DidClose().

Not asking for changes here- I'm starting a design doc with some initial
thoughts and will share it post-perf :)

[zhaobin, 2017/03/06 19:52:23]

Acknowledged.

       content::PRESENTATION_CONNECTION_STATE_CLOSED);
 }