Roll to pickup pool changes

packages/matcher/lib/src/string_matchers.dart

 // Copyright (c) 2012, 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.
 
 import 'interfaces.dart';
 
 /// Returns a matcher which matches if the match argument is a string and
 /// is equal to [value] when compared case-insensitively.
 Matcher equalsIgnoringCase(String value) => new _IsEqualIgnoringCase(value);
 
@@ skipping 100 matching lines @@
 
 Matcher stringContainsInOrder(List<String> substrings) =>
     new _StringContainsInOrder(substrings);
 
 class _StringContainsInOrder extends _StringMatcher {
   final List<String> _substrings;
 
   const _StringContainsInOrder(this._substrings);
 
   bool matches(item, Map matchState) {
-    if (!(item is String)) {
+    if (item is String) {
+      var from_index = 0;
+      for (var s in _substrings) {
+        from_index = item.indexOf(s, from_index);
+        if (from_index < 0) return false;
+      }
+      return true;
+    } else {
       return false;
     }
-    var from_index = 0;
-    for (var s in _substrings) {
-      from_index = item.indexOf(s, from_index);
-      if (from_index < 0) return false;
-    }
-    return true;
   }
 
   Description describe(Description description) => description.addAll(
       'a string containing ', ', ', ' in order', _substrings);
 }
 
 /// Returns a matcher that matches if the match argument is a string and
 /// matches the regular expression given by [re].
 ///
 /// [re] can be a [RegExp] instance or a [String]; in the latter case it will be
@@ skipping 50 matching lines @@
     } else {
       result.write(character);
       skipSpace = false;
     }
   }
   return result.toString().trim();
 }
 
 bool _isWhitespace(String ch) =>
     ch == ' ' || ch == '\n' || ch == '\r' || ch == '\t';