From f4053b9e2d47c2aed60a701764934b4327aecd29 Mon Sep 17 00:00:00 2001
From: "Gregory.Shrago" <gregory.shrago@jetbrains.com>
Date: Thu, 25 Jul 2013 19:09:38 +0400
Subject: [PATCH 1/2]  IDEA-110933 Database table editor: in editing mode
 attempt to move focus to different view immediately moves it back

---
 .../intellij/openapi/fileEditor/impl/EditorsSplitters.java  | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/platform/platform-impl/src/com/intellij/openapi/fileEditor/impl/EditorsSplitters.java b/platform/platform-impl/src/com/intellij/openapi/fileEditor/impl/EditorsSplitters.java
index 7cb99dd389c4..f6bb0cc994e3 100644
--- a/platform/platform-impl/src/com/intellij/openapi/fileEditor/impl/EditorsSplitters.java
+++ b/platform/platform-impl/src/com/intellij/openapi/fileEditor/impl/EditorsSplitters.java
@@ -58,6 +58,7 @@ import javax.swing.*;
 import java.awt.*;
 import java.awt.datatransfer.DataFlavor;
 import java.awt.datatransfer.Transferable;
+import java.awt.event.ContainerEvent;
 import java.io.File;
 import java.util.*;
 import java.util.List;
@@ -827,6 +828,11 @@ public class EditorsSplitters extends JBPanel {
       if (component != null) {
         newWindow = findWindowWith(component);
       }
+      else if (cause instanceof ContainerEvent && cause.getID() == ContainerEvent.COMPONENT_REMOVED) {
+        // do not change current window in case of child removal as in JTable.removeEditor
+        // otherwise Escape in a toolwindow will not focus editor with JTable content
+        return;
+      }
 
       setCurrentWindow(newWindow);
       setCurrentWindow(newWindow, false);

From f197f9f472fe7c894bb988f4129b3724432566dd Mon Sep 17 00:00:00 2001
From: Dmitry Jemerov <yole@jetbrains.com>
Date: Thu, 25 Jul 2013 16:52:12 +0200
Subject: [PATCH 2/2] passing a name to ModuleBasedConfiguration constructor is
 optional; kill a lot of code for passing around empty names

---
 .../com/intellij/execution/applet/AppletConfiguration.java   | 4 ++--
 .../intellij/execution/applet/AppletConfigurationType.java   | 2 +-
 .../com/intellij/execution/remote/RemoteConfiguration.java   | 4 ++--
 .../intellij/execution/remote/RemoteConfigurationType.java   | 2 +-
 .../execution/configuration/AbstractRunConfiguration.java    | 5 +++++
 .../execution/configurations/ModuleBasedConfiguration.java   | 5 +++++
 6 files changed, 16 insertions(+), 6 deletions(-)

diff --git a/java/execution/impl/src/com/intellij/execution/applet/AppletConfiguration.java b/java/execution/impl/src/com/intellij/execution/applet/AppletConfiguration.java
index b8b935f8e426..8a51328e5f98 100644
--- a/java/execution/impl/src/com/intellij/execution/applet/AppletConfiguration.java
+++ b/java/execution/impl/src/com/intellij/execution/applet/AppletConfiguration.java
@@ -69,8 +69,8 @@ public class AppletConfiguration extends ModuleBasedConfiguration<JavaRunConfigu
   @NonNls
   protected static final String PARAMETER_ELEMENT_NAME = "parameter";
 
-  public AppletConfiguration(final String name, final Project project, ConfigurationFactory factory) {
-    super(name, new JavaRunConfigurationModule(project, false), factory);
+  public AppletConfiguration(final Project project, ConfigurationFactory factory) {
+    super(new JavaRunConfigurationModule(project, false), factory);
   }
 
   public void setMainClass(final PsiClass psiClass) {
diff --git a/java/execution/impl/src/com/intellij/execution/applet/AppletConfigurationType.java b/java/execution/impl/src/com/intellij/execution/applet/AppletConfigurationType.java
index 16431f362e7e..79a42c0118f8 100644
--- a/java/execution/impl/src/com/intellij/execution/applet/AppletConfigurationType.java
+++ b/java/execution/impl/src/com/intellij/execution/applet/AppletConfigurationType.java
@@ -36,7 +36,7 @@ public class AppletConfigurationType implements ConfigurationType {
   AppletConfigurationType() {
     myFactory = new ConfigurationFactoryEx(this) {
       public RunConfiguration createTemplateConfiguration(Project project) {
-        return new AppletConfiguration("", project, this);
+        return new AppletConfiguration(project, this);
       }
 
       @Override
diff --git a/java/execution/impl/src/com/intellij/execution/remote/RemoteConfiguration.java b/java/execution/impl/src/com/intellij/execution/remote/RemoteConfiguration.java
index 0b2ba56debbf..3b86726390a4 100644
--- a/java/execution/impl/src/com/intellij/execution/remote/RemoteConfiguration.java
+++ b/java/execution/impl/src/com/intellij/execution/remote/RemoteConfiguration.java
@@ -65,8 +65,8 @@ public class RemoteConfiguration extends ModuleBasedConfiguration<JavaRunConfigu
   public String HOST;
   public String PORT;
 
-  public RemoteConfiguration(final String name, final Project project, ConfigurationFactory configurationFactory) {
-    super(name, new JavaRunConfigurationModule(project, true), configurationFactory);
+  public RemoteConfiguration(final Project project, ConfigurationFactory configurationFactory) {
+    super(new JavaRunConfigurationModule(project, true), configurationFactory);
   }
 
   public RemoteConnection createRemoteConnection() {
diff --git a/java/execution/impl/src/com/intellij/execution/remote/RemoteConfigurationType.java b/java/execution/impl/src/com/intellij/execution/remote/RemoteConfigurationType.java
index 4e366606700f..87c18dc9d5f3 100644
--- a/java/execution/impl/src/com/intellij/execution/remote/RemoteConfigurationType.java
+++ b/java/execution/impl/src/com/intellij/execution/remote/RemoteConfigurationType.java
@@ -40,7 +40,7 @@ public class RemoteConfigurationType implements ConfigurationType {
   public RemoteConfigurationType() {
     myFactory = new ConfigurationFactory(this) {
       public RunConfiguration createTemplateConfiguration(Project project) {
-        return new RemoteConfiguration("", project, this);
+        return new RemoteConfiguration(project, this);
       }
 
     };
diff --git a/platform/lang-api/src/com/intellij/execution/configuration/AbstractRunConfiguration.java b/platform/lang-api/src/com/intellij/execution/configuration/AbstractRunConfiguration.java
index f9807582562c..424c62e03a2e 100644
--- a/platform/lang-api/src/com/intellij/execution/configuration/AbstractRunConfiguration.java
+++ b/platform/lang-api/src/com/intellij/execution/configuration/AbstractRunConfiguration.java
@@ -3,6 +3,7 @@ package com.intellij.execution.configuration;
 import com.intellij.execution.configurations.ConfigurationFactory;
 import com.intellij.execution.configurations.ModuleBasedConfiguration;
 import com.intellij.execution.configurations.RunConfigurationModule;
+import com.intellij.openapi.project.Project;
 
 import java.util.LinkedHashMap;
 import java.util.Map;
@@ -18,6 +19,10 @@ public abstract class AbstractRunConfiguration extends ModuleBasedConfiguration<
     super(name, configurationModule, factory);
   }
 
+  public AbstractRunConfiguration(Project project, ConfigurationFactory factory) {
+    super(new RunConfigurationModule(project), factory);
+  }
+
   public Map<String, String> getEnvs() {
     return myEnvs;
   }
diff --git a/platform/lang-api/src/com/intellij/execution/configurations/ModuleBasedConfiguration.java b/platform/lang-api/src/com/intellij/execution/configurations/ModuleBasedConfiguration.java
index e72537bd3ea0..952079303289 100644
--- a/platform/lang-api/src/com/intellij/execution/configurations/ModuleBasedConfiguration.java
+++ b/platform/lang-api/src/com/intellij/execution/configurations/ModuleBasedConfiguration.java
@@ -45,6 +45,11 @@ public abstract class ModuleBasedConfiguration<ConfigurationModule extends RunCo
     myModule = configurationModule;
   }
 
+  public ModuleBasedConfiguration(final ConfigurationModule configurationModule, final ConfigurationFactory factory) {
+    super("", configurationModule.getProject(), factory);
+    myModule = configurationModule;
+  }
+
   public abstract Collection<Module> getValidModules();
 
   public ConfigurationModule getConfigurationModule() {