Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(276)

Side by Side Diff: editor/tools/plugins/com.google.dart.tools.debug.core/src/com/google/dart/tools/debug/core/breakpoints/DartBreakpoint.java

Issue 605833002: Show errors reported for invalid setBreakpoint calls. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Try both URLs. Write errors to the console. Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2012, the Dart project authors. 2 * Copyright (c) 2012, the Dart project authors.
3 * 3 *
4 * Licensed under the Eclipse Public License v1.0 (the "License"); you may not u se this file except 4 * Licensed under the Eclipse Public License v1.0 (the "License"); you may not u se this file except
5 * in compliance with the License. You may obtain a copy of the License at 5 * in compliance with the License. You may obtain a copy of the License at
6 * 6 *
7 * http://www.eclipse.org/legal/epl-v10.html 7 * http://www.eclipse.org/legal/epl-v10.html
8 * 8 *
9 * Unless required by applicable law or agreed to in writing, software distribut ed under the License 9 * Unless required by applicable law or agreed to in writing, software distribut ed under the License
10 * is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY K IND, either express 10 * is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY K IND, either express
(...skipping 126 matching lines...) Expand 10 before | Expand all | Expand 10 after
137 } 137 }
138 138
139 public String getFilePath() { 139 public String getFilePath() {
140 try { 140 try {
141 return (String) getMarker().getAttribute(FILE_PATH); 141 return (String) getMarker().getAttribute(FILE_PATH);
142 } catch (CoreException e) { 142 } catch (CoreException e) {
143 return null; 143 return null;
144 } 144 }
145 } 145 }
146 146
147 public String getActualFilePath() {
148 IFile file = getFile();
149 if (file != null) {
150 return file.getLocation().toOSString();
151 }
152 return getFilePath();
153 }
154
147 public int getLine() { 155 public int getLine() {
148 IMarker marker = getMarker(); 156 IMarker marker = getMarker();
149 157
150 if (marker != null) { 158 if (marker != null) {
151 return marker.getAttribute(IMarker.LINE_NUMBER, -1); 159 return marker.getAttribute(IMarker.LINE_NUMBER, -1);
152 } 160 }
153 161
154 return -1; 162 return -1;
155 } 163 }
156 164
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
192 public void updateLineNumber(int newLine) { 200 public void updateLineNumber(int newLine) {
193 try { 201 try {
194 getMarker().setAttribute(IMarker.LINE_NUMBER, newLine); 202 getMarker().setAttribute(IMarker.LINE_NUMBER, newLine);
195 } catch (CoreException e) { 203 } catch (CoreException e) {
196 // We make a best effort to update the breakpoint's line. 204 // We make a best effort to update the breakpoint's line.
197 205
198 } 206 }
199 } 207 }
200 208
201 } 209 }
OLDNEW
« no previous file with comments | « no previous file | editor/tools/plugins/com.google.dart.tools.debug.core/src/com/google/dart/tools/debug/core/server/ServerBreakpointManager.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698