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

Side by Side Diff: editor/tools/plugins/com.google.dart.server_test/src/com/google/dart/server/timing/AnalyzeEngineInServer.java

Issue 651243005: Add support for attaching observatory to the server (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 1 month 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) 2014, the Dart project authors. 2 * Copyright (c) 2014, 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 95 matching lines...) Expand 10 before | Expand all | Expand 10 after
106 String analysisServerPath = buildPath(svnDirectory, new String[] { 106 String analysisServerPath = buildPath(svnDirectory, new String[] {
107 "pkg", "analysis_server", "bin", "server.dart"}); 107 "pkg", "analysis_server", "bin", "server.dart"});
108 // TODO(paulberry): this won't run the server correctly because we aren't su pplying a package 108 // TODO(paulberry): this won't run the server correctly because we aren't su pplying a package
109 // root. The proper fix would be for the timing tests to run from a snapsho t (since this will 109 // root. The proper fix would be for the timing tests to run from a snapsho t (since this will
110 // give more accurate timing results) rather than from source. 110 // give more accurate timing results) rather than from source.
111 StdioServerSocket serverSocket = new StdioServerSocket( 111 StdioServerSocket serverSocket = new StdioServerSocket(
112 runtimePath, 112 runtimePath,
113 analysisServerPath, 113 analysisServerPath,
114 null, 114 null,
115 null, 115 null,
116 false,
116 false); 117 false);
117 server = new RemoteAnalysisServerImpl(serverSocket); 118 server = new RemoteAnalysisServerImpl(serverSocket);
118 server.start(); 119 server.start();
119 } 120 }
120 121
121 @Override 122 @Override
122 protected void oneTimeTearDown() { 123 protected void oneTimeTearDown() {
123 server.server_shutdown(); 124 server.server_shutdown();
124 } 125 }
125 126
(...skipping 64 matching lines...) Expand 10 before | Expand all | Expand 10 after
190 long endTime = System.currentTimeMillis() + timeout; 191 long endTime = System.currentTimeMillis() + timeout;
191 while (listener.isAnalyzing && System.currentTimeMillis() < endTime) { 192 while (listener.isAnalyzing && System.currentTimeMillis() < endTime) {
192 Thread.yield(); 193 Thread.yield();
193 } 194 }
194 if (listener.isAnalyzing) { 195 if (listener.isAnalyzing) {
195 System.out.println(System.currentTimeMillis() + " Timed out"); 196 System.out.println(System.currentTimeMillis() + " Timed out");
196 } 197 }
197 return listener.isAnalyzing; 198 return listener.isAnalyzing;
198 } 199 }
199 } 200 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698