Stop running pub get at gclient sync time and fix build bugs

mojo/public/dart/third_party/shelf/test/pipeline_test.dart

Index: mojo/public/dart/third_party/shelf/test/pipeline_test.dart
diff --git a/mojo/public/dart/third_party/shelf/test/pipeline_test.dart b/mojo/public/dart/third_party/shelf/test/pipeline_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..314ba2076136bf48c345f27f8197916516023685
--- /dev/null
+++ b/mojo/public/dart/third_party/shelf/test/pipeline_test.dart
@@ -0,0 +1,90 @@
+// Copyright (c) 2014, the Dart project authors.  Please see the AUTHORS file
+// for details. All rights reserved. Use of this source code is governed by a
+// BSD-style license that can be found in the LICENSE file.
+
+library shelf.pipeline_test;
+
+import 'package:shelf/shelf.dart';
+import 'package:test/test.dart';
+
+import 'test_util.dart';
+
+void main() {
+  test('compose middleware with Pipeline', () {
+    int accessLocation = 0;
+
+    var middlewareA = createMiddleware(requestHandler: (request) {
+      expect(accessLocation, 0);
+      accessLocation = 1;
+      return null;
+    }, responseHandler: (response) {
+      expect(accessLocation, 4);
+      accessLocation = 5;
+      return response;
+    });
+
+    var middlewareB = createMiddleware(requestHandler: (request) {
+      expect(accessLocation, 1);
+      accessLocation = 2;
+      return null;
+    }, responseHandler: (response) {
+      expect(accessLocation, 3);
+      accessLocation = 4;
+      return response;
+    });
+
+    var handler = const Pipeline()
+        .addMiddleware(middlewareA)
+        .addMiddleware(middlewareB)
+        .addHandler((request) {
+      expect(accessLocation, 2);
+      accessLocation = 3;
+      return syncHandler(request);
+    });
+
+    return makeSimpleRequest(handler).then((response) {
+      expect(response, isNotNull);
+      expect(accessLocation, 5);
+    });
+  });
+
+  test('Pipeline can be used as middleware', () {
+    int accessLocation = 0;
+
+    var middlewareA = createMiddleware(requestHandler: (request) {
+      expect(accessLocation, 0);
+      accessLocation = 1;
+      return null;
+    }, responseHandler: (response) {
+      expect(accessLocation, 4);
+      accessLocation = 5;
+      return response;
+    });
+
+    var middlewareB = createMiddleware(requestHandler: (request) {
+      expect(accessLocation, 1);
+      accessLocation = 2;
+      return null;
+    }, responseHandler: (response) {
+      expect(accessLocation, 3);
+      accessLocation = 4;
+      return response;
+    });
+
+    var innerPipeline =
+        const Pipeline().addMiddleware(middlewareA).addMiddleware(middlewareB);
+
+    var handler = const Pipeline()
+        .addMiddleware(innerPipeline.middleware)
+        .addHandler((request) {
+      expect(accessLocation, 2);
+      accessLocation = 3;
+      return syncHandler(request);
+    });
+
+    return makeSimpleRequest(handler).then((response) {
+      expect(response, isNotNull);
+      expect(accessLocation, 5);
+    });
+  });
+}