Fix static analysis warnings in Observatory

runtime/observatory/lib/src/elements/script_inset.dart

 // Copyright (c) 2013, 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 script_inset_element;
 
 import 'dart:async';
 import 'dart:html';
 import 'observatory_element.dart';
 import 'service_ref.dart';
@@ skipping 108 matching lines @@
   Breakpoint bpt;
   int priority = 1;
 
   BreakpointAnnotation(this.bpt) {
     var script = bpt.location.script;
     if (bpt.location.tokenPos != null) {
       var pos = bpt.location.tokenPos;
       line = script.tokenToLine(pos);
       columnStart = script.tokenToCol(pos) - 1;  // tokenToCol is 1-origin.
     } else {
-      line = bpt.location.line;
-      columnStart = bpt.location.column;
-      if (columnStart == null) {
-        columnStart = 0;
-      }

[turnidge, 2015/09/15 16:01:03]

bpt.location can have two types: SourceLocation and UnresolvedSourceLocation. 
If you remove this code, you will break breakpoint annotations for some
unresolved breakpoints, I think.

[Cutch, 2015/09/15 17:29:57]

Yep, you are right. I've fixed the static warnings in a different way. PTAL.

+      line = 0;
+      columnStart = 0;
     }
     var length = script.guessTokenLength(line, columnStart);
     if (length == null) {
       length = 1;
     }
     columnStop = columnStart + length;
   }
 
   void applyStyleTo(element) {
     if (element == null) {
@@ skipping 868 matching lines @@
 @CustomTag('source-inset')
 class SourceInsetElement extends PolymerElement {
   SourceInsetElement.created() : super.created();
 
   @published SourceLocation location;
   @published String height = null;
   @published int currentPos;
   @published bool inDebuggerContext = false;
   @published ObservableList variables;
 }
-