Bug 165829

[launching] the source lookup tab is the new apply button

2 files changed, 2 insertions, 4 deletions

diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/sourcelookup/SourceLookupPanel.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/sourcelookup/SourceLookupPanel.java
index 6e90eaeae..46e4990c6 100644
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/sourcelookup/SourceLookupPanel.java
+++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/sourcelookup/SourceLookupPanel.java
@@ -357,9 +357,6 @@ public class SourceLookupPanel extends AbstractLaunchConfigurationTab implements
 					workingCopy.setAttribute(ILaunchConfiguration.ATTR_SOURCE_LOCATOR_MEMENTO, fLocator.getMemento());
 					workingCopy.setAttribute(ILaunchConfiguration.ATTR_SOURCE_LOCATOR_ID, fLocator.getId());
 				}
-				if(!workingCopy.isReadOnly()) {
-					workingCopy.doSave(); 
-				}
 			}
 			catch(CoreException e) {
 				DebugUIPlugin.log(e);
diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/sourcelookup/SourceLookupDialog.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/sourcelookup/SourceLookupDialog.java
index d479ad567..bdda0aa7f 100644
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/sourcelookup/SourceLookupDialog.java
+++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/sourcelookup/SourceLookupDialog.java
@@ -106,10 +106,11 @@ public class SourceLookupDialog extends TitleAreaDialog {
 		if(config != null) {
 			try {
 				copy = config.getWorkingCopy();
+				fPanel.performApply(copy);
+				copy.doSave();
 			} 
 			catch (CoreException e) {DebugUIPlugin.log(e);}
 		}
-		fPanel.performApply(copy);
 		super.okPressed();
 	}