[compiler] Delete extra map check in StringIterator.

test/unittests/compiler/js-builtin-reducer-unittest.cc

Index: test/unittests/compiler/js-builtin-reducer-unittest.cc
diff --git a/test/unittests/compiler/js-builtin-reducer-unittest.cc b/test/unittests/compiler/js-builtin-reducer-unittest.cc
index 419cc7224e95fe1b0a3f9bfaae0fe42232e06f6b..e0824375b61b7071c2884b8466c9e11de8c7ff73 100644
--- a/test/unittests/compiler/js-builtin-reducer-unittest.cc
+++ b/test/unittests/compiler/js-builtin-reducer-unittest.cc
@@ -33,8 +33,10 @@ class JSBuiltinReducerTest : public TypedGraphTest {
                     &machine);
     // TODO(titzer): mock the GraphReducer here for better unit testing.
     GraphReducer graph_reducer(zone(), graph());
+
     JSBuiltinReducer reducer(&graph_reducer, &jsgraph,
-                             JSBuiltinReducer::kNoFlags, nullptr);
+                             JSBuiltinReducer::kNoFlags, nullptr,
+                             Handle<Context>());

[Benedikt Meurer, 2016/11/03 19:43:03]

You can just get to the current native contecxt for the test via
native_context() here, which is somewhat safe, because passing a null handle is
not really ideal.

     return reducer.Reduce(node);
   }