eb6d1e2579
With the current constant evaluator, the bug is benign because we only do dependency analysis within a library cycle, and the library cycle containing dart:core (which is the only library cycle with const external factories) contains no invocations of those const external factories. But in the new task model we do full dependency analysis across library cycles, so the dependency analysis needs to be correct in order for the const external factory to be safely invoked. R=scheglov@google.com Review URL: https://codereview.chromium.org//1134343003