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

third_party/WebKit/public/platform/modules/presentation/WebPresentationConnection.h

Index: third_party/WebKit/public/platform/modules/presentation/WebPresentationConnection.h
diff --git a/third_party/WebKit/public/platform/modules/presentation/WebPresentationConnection.h b/third_party/WebKit/public/platform/modules/presentation/WebPresentationConnection.h
index 69460de80b6baf0124042483b213550eb067ed7a..65f8cab88735cfb9cb28acbceaa0b5eb59c727d7 100644
--- a/third_party/WebKit/public/platform/modules/presentation/WebPresentationConnection.h
+++ b/third_party/WebKit/public/platform/modules/presentation/WebPresentationConnection.h
@@ -11,6 +11,7 @@
 
 namespace blink {
 
+enum class WebPresentationConnectionCloseReason;
 enum class WebPresentationConnectionState;
 class WebPresentationConnectionProxy;
 class WebString;
@@ -30,6 +31,10 @@ class WebPresentationConnection {
 
   // Notifies the connection about its state change.
   virtual void didChangeState(WebPresentationConnectionState) = 0;
+
+  // Notifies the connection about its state change to 'closed'.
+  virtual void didClose(WebPresentationConnectionCloseReason,

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

Can you remove the message argument since it's always empty?

My thought is to have an API with a did() method for each of the close reasons,
especially since there is no message for the "Closed" or "WentAway" reasons.

e.g.

didError(message) => Error
didClose() => Closed 
didGoAway() => WentAway

(Not asking for this change here though.)

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

Done.

+                        const WebString& message) = 0;
 };
 
 }  // namespace blink