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

Unified Diff: editor/tools/plugins/com.google.dart.tools.debug.ui/src/com/google/dart/tools/debug/ui/internal/client/MainDartJsLaunchConfigurationTab.java

Issue 9178032: Rename and move refactorings in the launch code. (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 8 years, 11 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 side-by-side diff with in-line comments
Download patch
Index: editor/tools/plugins/com.google.dart.tools.debug.ui/src/com/google/dart/tools/debug/ui/internal/client/MainDartJsLaunchConfigurationTab.java
===================================================================
--- editor/tools/plugins/com.google.dart.tools.debug.ui/src/com/google/dart/tools/debug/ui/internal/client/MainDartJsLaunchConfigurationTab.java (revision 3372)
+++ editor/tools/plugins/com.google.dart.tools.debug.ui/src/com/google/dart/tools/debug/ui/internal/client/MainDartJsLaunchConfigurationTab.java (working copy)
@@ -1,148 +0,0 @@
-/*
- * Copyright (c) 2011, the Dart project authors.
- *
- * Licensed under the Eclipse Public License v1.0 (the "License"); you may not
- * use this file except in compliance with the License. You may obtain a copy of
- * the License at
- *
- * http://www.eclipse.org/legal/epl-v10.html
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
- * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
- * License for the specific language governing permissions and limitations under
- * the License.
- */
-package com.google.dart.tools.debug.ui.internal.client;
-
-import com.google.dart.tools.debug.ui.internal.DartDebugUIPlugin;
-import com.google.dart.tools.debug.ui.internal.DartUtil;
-
-import org.eclipse.core.resources.IFile;
-import org.eclipse.core.resources.IResource;
-import org.eclipse.core.resources.IWorkspace;
-import org.eclipse.core.resources.ResourcesPlugin;
-import org.eclipse.core.runtime.IPath;
-import org.eclipse.core.runtime.Path;
-import org.eclipse.debug.core.ILaunchConfiguration;
-import org.eclipse.debug.core.ILaunchConfigurationWorkingCopy;
-import org.eclipse.debug.ui.AbstractLaunchConfigurationTab;
-import org.eclipse.swt.SWT;
-import org.eclipse.swt.graphics.Image;
-import org.eclipse.swt.widgets.Composite;
-import org.eclipse.swt.widgets.Control;
-
-/**
- * Main Dart Launch/Debug configuration tab
- */
-public class MainDartJsLaunchConfigurationTab extends AbstractLaunchConfigurationTab {
-
- private MainDartJsLaunchConfigurationPanel panel;
- private int initialLaunchType;
- private String initialResPath;
- private boolean initialExternal;
-
- @Override
- public void createControl(Composite parent) {
- panel = new MainDartJsLaunchConfigurationPanel(parent, SWT.NONE, this);
- setControl(panel);
-
- // TODO (danrubel) implement help
- // if (helpContextId != null) {
- // PlatformUI.getWorkbench().getHelpSystem().setHelp(group, helpContextId);
- // }
-
- // Use asyncExec(...) rather then scheduleUpdateJob()
- // so that this code will execute in Eclipse 3.5
- parent.getDisplay().asyncExec(new Runnable() {
- @Override
- public void run() {
- updateLaunchConfigurationDialog();
- }
- });
- }
-
- @Override
- public void dispose() {
- Control control = getControl();
- if (control != null) {
- control.dispose();
- setControl(null);
- }
- }
-
- /**
- * Answer the image to show in the configuration tab or <code>null</code> if none
- */
- @Override
- public Image getImage() {
- return DartDebugUIPlugin.getImage("dart_app.png");
- }
-
- /**
- * Answer the name to show in the configuration tab
- */
- @Override
- public String getName() {
- return "Main";
- }
-
- /**
- * Initialize the UI from the specified configuration
- */
- @Override
- public void initializeFrom(ILaunchConfiguration config) {
- initialLaunchType = DartUtil.getLaunchType(config);
- initialResPath = DartUtil.getResourcePath(config);
- initialExternal = DartUtil.isExternalBrowser(config);
-
- panel.setLaunchType(initialLaunchType);
- panel.setResourcePath(initialResPath);
- panel.setExternalBrowser(initialExternal);
- }
-
- /**
- * Determine if the configuration can be launched
- */
- @Override
- public boolean isValid(ILaunchConfiguration config) {
- IWorkspace workspace = ResourcesPlugin.getWorkspace();
- IPath path = new Path(DartUtil.getResourcePath(config));
- if (!workspace.validatePath(path.toString(), IResource.FILE).isOK()) {
- return false;
- }
- IFile file = workspace.getRoot().getFile(path);
- if (!file.exists()) {
- return false;
- }
- return DartUtil.isWebPage(file) || DartUtil.isDartApp(file);
- }
-
- /**
- * Store the value specified in the UI into the launch configuration
- */
- @Override
- public void performApply(ILaunchConfigurationWorkingCopy config) {
- initialLaunchType = panel.getLaunchType();
- initialResPath = panel.getResourcePath();
- initialExternal = panel.isExternalBrowser();
-
- config.setAttribute(ILaunchConstants.ATTR_LAUNCH_TYPE, initialLaunchType);
- config.setAttribute(ILaunchConstants.ATTR_RESOURCE_PATH, initialResPath);
- config.setAttribute(ILaunchConstants.ATTR_EXTERNAL_BROWSER, initialExternal);
- }
-
- @Override
- public void setDefaults(ILaunchConfigurationWorkingCopy config) {
- DartUtil.notYetImplemented(config);
- }
-
- /**
- * Called by the {@link MainDartJsLaunchConfigurationPanel} when the user has changed the panel
- * content.
- */
- void panelChanged() {
- setDirty(true);
- updateLaunchConfigurationDialog();
- }
-}

Powered by Google App Engine
This is Rietveld 408576698