Avoid passing a null uri to translateUri.

Avoid passing a null uri to translateUri.
This happens when reportDiagnostic is called from readScript.


BUG=
TEST=

Committed: https://code.google.com/p/dart/source/detail?r=9150

[karlklose, 2012-06-21 09:36:30]

LGTM.

[ahe, 2012-06-22 08:49:33]

http://codereview.chromium.org/10629002/diff/1/lib/compiler/implementation/ap...
File lib/compiler/implementation/apiimpl.dart (right):

http://codereview.chromium.org/10629002/diff/1/lib/compiler/implementation/ap...
lib/compiler/implementation/apiimpl.dart:99: if (span === null || span.uri ===
null) {
I would rather avoid creating source spans with uri == null. It is done in a few
places and should be easy to change to using null.

[Johnni Winther, 2012-06-22 11:40:55]

http://codereview.chromium.org/10629002/diff/1/lib/compiler/implementation/ap...
File lib/compiler/implementation/apiimpl.dart (right):

http://codereview.chromium.org/10629002/diff/1/lib/compiler/implementation/ap...
lib/compiler/implementation/apiimpl.dart:99: if (span === null || span.uri ===
null) {
On 2012/06/22 08:49:33, ahe wrote:
> I would rather avoid creating source spans with uri == null. It is done in a
few
> places and should be easy to change to using null.

Done.

[Johnni Winther, 2012-06-26 10:39:23]

PTAL

[ahe, 2012-06-26 11:38:41]

lgtm