Add an Environment class that's exposed through RunnerSuite.

lib/src/runner/browser/browser_manager.dart

 // Copyright (c) 2015, 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 test.runner.browser.browser_manager;
 
 import 'dart:async';
 import 'dart:convert';
 
 import 'package:http_parser/http_parser.dart';
 import 'package:pool/pool.dart';
 
 import '../../backend/metadata.dart';
 import '../../backend/test_platform.dart';
 import '../../util/multi_channel.dart';
 import '../../util/remote_exception.dart';
 import '../../util/stack_trace_mapper.dart';
 import '../../utils.dart';
+import '../environment.dart';
 import '../load_exception.dart';
 import '../runner_suite.dart';
 import 'iframe_test.dart';
 
 /// A class that manages the connection to a single running browser.
 ///
 /// This is in charge of telling the browser which test suites to load and
 /// converting its responses into [Suite] objects.
 class BrowserManager {
   /// The browser that this is managing.
@@ skipping 15 matching lines @@
 
   /// The ID of the next suite to be loaded.
   ///
   /// This is used to ensure that the suites can be referred to consistently
   /// across the client and server.
   int _suiteId = 0;
 
   /// Whether the channel to the browser has closed.
   bool _closed = false;
 
+  /// The environment to attach to each suite.
+  _BrowserEnvironment _environment;
+
   /// Creates a new BrowserManager that communicates with [browser] over
   /// [webSocket].
   BrowserManager(this.browser, CompatibleWebSocket webSocket)
       : _channel = new MultiChannel(
           webSocket.map(JSON.decode),
           mapSink(webSocket, JSON.encode)) {
+    _environment = new _BrowserEnvironment(this);
     _channel.stream.listen(null, onDone: () => _closed = true);
   }
 
   /// Tells the browser the load a test suite from the URL [url].
   ///
   /// [url] should be an HTML page with a reference to the JS-compiled test
   /// suite. [path] is the path of the original test suite file, which is used
   /// for reporting. [metadata] is the parsed metadata for the test suite.
   ///
   /// If [mapper] is passed, it's used to map stack traces for errors coming
@@ skipping 63 matching lines @@
     }
 
     if (response["type"] == "error") {
       closeIframe();
       var asyncError = RemoteException.deserialize(response["error"]);
       await new Future.error(
           new LoadException(path, asyncError.error),
           asyncError.stackTrace);
     }
 
-    return new RunnerSuite(response["tests"].map((test) {
+    return new RunnerSuite(_environment, response["tests"].map((test) {
       var testMetadata = new Metadata.deserialize(test['metadata']);
       var testChannel = suiteChannel.virtualChannel(test['channel']);
       return new IframeTest(test['name'], testMetadata, testChannel,
           mapper: mapper);
     }), platform: browser, metadata: metadata, path: path,
         onClose: () => closeIframe());
   }
 }
+
+/// An implementation of [Environment] for the browser.
+///
+/// All methods forward directly to [BrowserManager].
+class _BrowserEnvironment implements Environment {
+  final BrowserManager _manager;

[kevmoo, 2015/07/30 01:37:24]

This field ends up being unused.

Uh...

[nweiz, 2015/07/30 19:41:50]

Right, the whole point of this CL is to set the stage for Environment to
actually get members later on.

+
+  _BrowserEnvironment(this._manager);
+}