Fix various nits in VM patch files.

sdk/lib/io/secure_socket.dart

Index: sdk/lib/io/secure_socket.dart
diff --git a/sdk/lib/io/secure_socket.dart b/sdk/lib/io/secure_socket.dart
index a523f20d6991099d4182a094023dd84b6bb66fbc..c6d3cb2d23885a742816a07e99c11cf600af9b54 100644
--- a/sdk/lib/io/secure_socket.dart
+++ b/sdk/lib/io/secure_socket.dart
@@ -446,12 +446,12 @@ class _RawSecureSocket extends Stream<RawSocketEvent>
       int requestedPort,
       this.is_server,
       this.context,
-      RawSocket this._socket,
+      this._socket,
       this._socketSubscription,
       this._bufferedData,
       this.requestClientCertificate,
       this.requireClientCertificate,
-      this.onBadCertificate(X509Certificate certificate),

[Vyacheslav Egorov (Google), 2017/02/17 13:08:52]

This was a more specific signature then onBadCertificate own type. Maybe own
type needs to be fixed?

[ahe, 2017/02/17 13:38:28]

I'm comfortable removing type annotations from Dart code, but adding types can
have unintended side-effects that I'd rather avoid. So honestly, I'd rather not
change the type on the field.

That being said, I don't think it's necessary. This constructor is private, and
the connect method has the proper signature.

+      this.onBadCertificate,
       List<String> supportedProtocols) {
     if (context == null) {
       context = SecurityContext.defaultContext;