From 39010ab847b68b0ba615bbdbef221d897a35ae05 Mon Sep 17 00:00:00 2001
From: Vyacheslav Gerasimov <vyacheslav.gerasimov@jetbrains.com>
Date: Mon, 6 Feb 2017 17:16:34 +0300
Subject: [PATCH] Fix broken isReferenceTo checking code for Android extensions

#KT-16132 Fixed
---
 .../idea/references/KtSimpleNameReference.kt  |  8 ++---
 .../SimpleNameReferenceExtension.kt           |  1 -
 .../AndroidSimpleNameReferenceExtension.kt    |  7 +----
 .../rename/commonElementId/commonElementId.kt | 20 +++++++++++++
 .../expected/commonElementId.kt               | 20 +++++++++++++
 .../expected/res/layout/layout.xml            | 17 +++++++++++
 .../commonElementId/res/layout/layout.xml     | 17 +++++++++++
 .../fqNameInAttr/expected/fqNameInAttr.kt     |  9 ++++++
 .../expected/res/layout/layout.xml            | 15 ++++++++++
 .../expected/fqNameInAttrFragment.kt          |  9 ++++++
 .../expected/res/layout/layout.xml            | 15 ++++++++++
 .../fqNameInTag/expected/fqNameInTag.kt       |  9 ++++++
 .../expected/res/layout/layout.xml            | 14 +++++++++
 .../expected/fqNameInTagFragment.kt           |  9 ++++++
 .../expected/res/layout/layout.xml            | 14 +++++++++
 .../rename/multiFile/expected/multiFile.kt    | 10 +++++++
 .../multiFile/expected/res/layout/layout.xml  | 29 +++++++++++++++++++
 .../multiFile/expected/res/layout/layout1.xml | 29 +++++++++++++++++++
 .../expected/multiFileFragment.kt             | 10 +++++++
 .../expected/res/layout/layout.xml            | 29 +++++++++++++++++++
 .../expected/res/layout/layout1.xml           | 29 +++++++++++++++++++
 .../simple/expected/res/layout/layout.xml     | 17 +++++++++++
 .../android/rename/simple/expected/simple.kt  | 12 ++++++++
 .../expected/res/layout/layout.xml            | 17 +++++++++++
 .../simpleFragment/expected/simpleFragment.kt | 11 +++++++
 .../simpleView/expected/res/layout/layout.xml | 17 +++++++++++
 .../rename/simpleView/expected/simpleView.kt  |  9 ++++++
 .../android/AbstractAndroidRenameTest.kt      | 27 +++++++++--------
 .../android/AndroidRenameTestGenerated.java   |  6 ++++
 29 files changed, 409 insertions(+), 27 deletions(-)
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/commonElementId/commonElementId.kt
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/commonElementId/expected/commonElementId.kt
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/commonElementId/expected/res/layout/layout.xml
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/commonElementId/res/layout/layout.xml
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInAttr/expected/fqNameInAttr.kt
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInAttr/expected/res/layout/layout.xml
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInAttrFragment/expected/fqNameInAttrFragment.kt
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInAttrFragment/expected/res/layout/layout.xml
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInTag/expected/fqNameInTag.kt
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInTag/expected/res/layout/layout.xml
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInTagFragment/expected/fqNameInTagFragment.kt
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInTagFragment/expected/res/layout/layout.xml
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFile/expected/multiFile.kt
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFile/expected/res/layout/layout.xml
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFile/expected/res/layout/layout1.xml
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFileFragment/expected/multiFileFragment.kt
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFileFragment/expected/res/layout/layout.xml
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFileFragment/expected/res/layout/layout1.xml
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/simple/expected/res/layout/layout.xml
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/simple/expected/simple.kt
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/simpleFragment/expected/res/layout/layout.xml
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/simpleFragment/expected/simpleFragment.kt
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/simpleView/expected/res/layout/layout.xml
 create mode 100644 plugins/android-extensions/android-extensions-idea/testData/android/rename/simpleView/expected/simpleView.kt

diff --git a/idea/idea-analysis/src/org/jetbrains/kotlin/idea/references/KtSimpleNameReference.kt b/idea/idea-analysis/src/org/jetbrains/kotlin/idea/references/KtSimpleNameReference.kt
index 62d83187d45..6d0d0637858 100644
--- a/idea/idea-analysis/src/org/jetbrains/kotlin/idea/references/KtSimpleNameReference.kt
+++ b/idea/idea-analysis/src/org/jetbrains/kotlin/idea/references/KtSimpleNameReference.kt
@@ -72,12 +72,8 @@ class KtSimpleNameReference(expression: KtSimpleNameExpression) : KtSimpleRefere
     }
 
     override fun isReferenceTo(element: PsiElement?): Boolean {
-        if (element != null) {
-            if (!canBeReferenceTo(element)) return false
-
-            for (extension in Extensions.getArea(element.project).getExtensionPoint(SimpleNameReferenceExtension.EP_NAME).extensions) {
-                if (extension.isReferenceTo(this, element)) return true
-            }
+        if (element != null && !canBeReferenceTo(element)) {
+            return false
         }
 
         return super.isReferenceTo(element)
diff --git a/idea/idea-analysis/src/org/jetbrains/kotlin/plugin/references/SimpleNameReferenceExtension.kt b/idea/idea-analysis/src/org/jetbrains/kotlin/plugin/references/SimpleNameReferenceExtension.kt
index e5bde18c4dc..5084cd1f44b 100644
--- a/idea/idea-analysis/src/org/jetbrains/kotlin/plugin/references/SimpleNameReferenceExtension.kt
+++ b/idea/idea-analysis/src/org/jetbrains/kotlin/plugin/references/SimpleNameReferenceExtension.kt
@@ -27,6 +27,5 @@ interface SimpleNameReferenceExtension {
                 ExtensionPointName.create("org.jetbrains.kotlin.simpleNameReferenceExtension")
     }
 
-    fun isReferenceTo(reference: KtSimpleNameReference, element: PsiElement): Boolean
     fun handleElementRename(reference: KtSimpleNameReference, psiFactory: KtPsiFactory, newElementName: String): PsiElement?
 }
\ No newline at end of file
diff --git a/plugins/android-extensions/android-extensions-idea/src/org/jetbrains/kotlin/android/synthetic/idea/AndroidSimpleNameReferenceExtension.kt b/plugins/android-extensions/android-extensions-idea/src/org/jetbrains/kotlin/android/synthetic/idea/AndroidSimpleNameReferenceExtension.kt
index bb9fb4081a4..2522b06bbfa 100644
--- a/plugins/android-extensions/android-extensions-idea/src/org/jetbrains/kotlin/android/synthetic/idea/AndroidSimpleNameReferenceExtension.kt
+++ b/plugins/android-extensions/android-extensions-idea/src/org/jetbrains/kotlin/android/synthetic/idea/AndroidSimpleNameReferenceExtension.kt
@@ -25,16 +25,11 @@ import org.jetbrains.kotlin.android.synthetic.androidIdToName
 import org.jetbrains.kotlin.idea.references.KtSimpleNameReference
 import org.jetbrains.kotlin.plugin.references.SimpleNameReferenceExtension
 import org.jetbrains.kotlin.psi.KtDotQualifiedExpression
+import org.jetbrains.kotlin.psi.KtProperty
 import org.jetbrains.kotlin.psi.KtPsiFactory
 
 class AndroidSimpleNameReferenceExtension : SimpleNameReferenceExtension {
 
-    override fun isReferenceTo(reference: KtSimpleNameReference, element: PsiElement) = when {
-        element is ValueResourceElementWrapper && AndroidResourceUtil.isIdDeclaration(element) -> true
-        isLayoutPackageIdentifier(reference) -> true
-        else -> false
-    }
-
     private fun isLayoutPackageIdentifier(reference: KtSimpleNameReference): Boolean {
         val probablyVariant = reference.element?.parent as? KtDotQualifiedExpression ?: return false
         val probablyKAS = probablyVariant.receiverExpression as? KtDotQualifiedExpression ?: return false
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/commonElementId/commonElementId.kt b/plugins/android-extensions/android-extensions-idea/testData/android/rename/commonElementId/commonElementId.kt
new file mode 100644
index 00000000000..7edff23d868
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/commonElementId/commonElementId.kt
@@ -0,0 +1,20 @@
+package com.myapp
+
+import android.app.Activity
+import android.os.Bundle
+import java.io.File
+import kotlinx.android.synthetic.main.layout.*
+
+// KT-16132 Renaming property provided by kotlinx leads to renaming another members
+
+object Loginer {
+    fun login() { }
+}
+
+public class MyActivity : Activity() {
+    override fun onCreate(savedInstanceState: Bundle?) {
+        <caret>login.setOnClickListener {
+            Loginer.login()
+        }
+    }
+}
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/commonElementId/expected/commonElementId.kt b/plugins/android-extensions/android-extensions-idea/testData/android/rename/commonElementId/expected/commonElementId.kt
new file mode 100644
index 00000000000..f387ae77593
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/commonElementId/expected/commonElementId.kt
@@ -0,0 +1,20 @@
+package com.myapp
+
+import android.app.Activity
+import android.os.Bundle
+import java.io.File
+import kotlinx.android.synthetic.main.layout.*
+
+// KT-16132 Renaming property provided by kotlinx leads to renaming another members
+
+object Loginer {
+    fun login() { }
+}
+
+public class MyActivity : Activity() {
+    override fun onCreate(savedInstanceState: Bundle?) {
+        NEWNAME.setOnClickListener {
+            Loginer.login()
+        }
+    }
+}
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/commonElementId/expected/res/layout/layout.xml b/plugins/android-extensions/android-extensions-idea/testData/android/rename/commonElementId/expected/res/layout/layout.xml
new file mode 100644
index 00000000000..8cf28d45f8a
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/commonElementId/expected/res/layout/layout.xml
@@ -0,0 +1,17 @@
+<FrameLayout xmlns:android="http://schemas.android.com/apk/res/android"
+             xmlns:tools="http://schemas.android.com/tools"
+             android:id="@+id/item_detail_container"
+             android:layout_width="match_parent"
+             android:layout_height="match_parent"
+             tools:context=".ItemDetailActivity"
+             tools:ignore="MergeRootFrame" >
+
+
+
+    <Button
+        android:id="@+id/NEWNAME"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:text="Sign in" />
+
+</FrameLayout>
\ No newline at end of file
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/commonElementId/res/layout/layout.xml b/plugins/android-extensions/android-extensions-idea/testData/android/rename/commonElementId/res/layout/layout.xml
new file mode 100644
index 00000000000..4d73173b521
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/commonElementId/res/layout/layout.xml
@@ -0,0 +1,17 @@
+<FrameLayout xmlns:android="http://schemas.android.com/apk/res/android"
+             xmlns:tools="http://schemas.android.com/tools"
+             android:id="@+id/item_detail_container"
+             android:layout_width="match_parent"
+             android:layout_height="match_parent"
+             tools:context=".ItemDetailActivity"
+             tools:ignore="MergeRootFrame" >
+
+
+
+    <Button
+        android:id="@+id/login"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:text="Sign in" />
+
+</FrameLayout>
\ No newline at end of file
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInAttr/expected/fqNameInAttr.kt b/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInAttr/expected/fqNameInAttr.kt
new file mode 100644
index 00000000000..87db619b6f3
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInAttr/expected/fqNameInAttr.kt
@@ -0,0 +1,9 @@
+package com.myapp
+
+import android.app.Activity
+import kotlinx.android.synthetic.main.layout.*
+
+class MyActivity: Activity() {
+    val button = this.NEWNAME
+}
+
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInAttr/expected/res/layout/layout.xml b/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInAttr/expected/res/layout/layout.xml
new file mode 100644
index 00000000000..26cafba11d8
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInAttr/expected/res/layout/layout.xml
@@ -0,0 +1,15 @@
+<FrameLayout xmlns:android="http://schemas.android.com/apk/res/android"
+             xmlns:tools="http://schemas.android.com/tools"
+             android:layout_width="match_parent"
+             android:layout_height="match_parent"
+             tools:context=".ItemDetailActivity"
+             tools:ignore="MergeRootFrame" >
+
+    <view
+        class="org.my.cool.Button"
+        android:id="@+id/NEWNAME"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:text="Sign in" />
+
+</FrameLayout>
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInAttrFragment/expected/fqNameInAttrFragment.kt b/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInAttrFragment/expected/fqNameInAttrFragment.kt
new file mode 100644
index 00000000000..0df90969f43
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInAttrFragment/expected/fqNameInAttrFragment.kt
@@ -0,0 +1,9 @@
+package com.myapp
+
+import android.app.Fragment
+import kotlinx.android.synthetic.main.layout.*
+
+class MyFragment: Fragment() {
+    val button = this.NEWNAME
+}
+
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInAttrFragment/expected/res/layout/layout.xml b/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInAttrFragment/expected/res/layout/layout.xml
new file mode 100644
index 00000000000..26cafba11d8
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInAttrFragment/expected/res/layout/layout.xml
@@ -0,0 +1,15 @@
+<FrameLayout xmlns:android="http://schemas.android.com/apk/res/android"
+             xmlns:tools="http://schemas.android.com/tools"
+             android:layout_width="match_parent"
+             android:layout_height="match_parent"
+             tools:context=".ItemDetailActivity"
+             tools:ignore="MergeRootFrame" >
+
+    <view
+        class="org.my.cool.Button"
+        android:id="@+id/NEWNAME"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:text="Sign in" />
+
+</FrameLayout>
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInTag/expected/fqNameInTag.kt b/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInTag/expected/fqNameInTag.kt
new file mode 100644
index 00000000000..87db619b6f3
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInTag/expected/fqNameInTag.kt
@@ -0,0 +1,9 @@
+package com.myapp
+
+import android.app.Activity
+import kotlinx.android.synthetic.main.layout.*
+
+class MyActivity: Activity() {
+    val button = this.NEWNAME
+}
+
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInTag/expected/res/layout/layout.xml b/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInTag/expected/res/layout/layout.xml
new file mode 100644
index 00000000000..38a26d0980e
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInTag/expected/res/layout/layout.xml
@@ -0,0 +1,14 @@
+<FrameLayout xmlns:android="http://schemas.android.com/apk/res/android"
+             xmlns:tools="http://schemas.android.com/tools"
+             android:layout_width="match_parent"
+             android:layout_height="match_parent"
+             tools:context=".ItemDetailActivity"
+             tools:ignore="MergeRootFrame" >
+
+    <org.my.cool.Button
+        android:id="@+id/NEWNAME"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:text="Sign in" />
+
+</FrameLayout>
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInTagFragment/expected/fqNameInTagFragment.kt b/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInTagFragment/expected/fqNameInTagFragment.kt
new file mode 100644
index 00000000000..0df90969f43
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInTagFragment/expected/fqNameInTagFragment.kt
@@ -0,0 +1,9 @@
+package com.myapp
+
+import android.app.Fragment
+import kotlinx.android.synthetic.main.layout.*
+
+class MyFragment: Fragment() {
+    val button = this.NEWNAME
+}
+
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInTagFragment/expected/res/layout/layout.xml b/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInTagFragment/expected/res/layout/layout.xml
new file mode 100644
index 00000000000..38a26d0980e
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInTagFragment/expected/res/layout/layout.xml
@@ -0,0 +1,14 @@
+<FrameLayout xmlns:android="http://schemas.android.com/apk/res/android"
+             xmlns:tools="http://schemas.android.com/tools"
+             android:layout_width="match_parent"
+             android:layout_height="match_parent"
+             tools:context=".ItemDetailActivity"
+             tools:ignore="MergeRootFrame" >
+
+    <org.my.cool.Button
+        android:id="@+id/NEWNAME"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:text="Sign in" />
+
+</FrameLayout>
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFile/expected/multiFile.kt b/plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFile/expected/multiFile.kt
new file mode 100644
index 00000000000..c5da8a04289
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFile/expected/multiFile.kt
@@ -0,0 +1,10 @@
+package com.myapp
+
+import android.app.Activity
+import kotlinx.android.synthetic.main.layout.*
+
+class MyActivity: Activity() {
+    val button = this.NEWNAME
+    val button1 = this.loginButton
+}
+
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFile/expected/res/layout/layout.xml b/plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFile/expected/res/layout/layout.xml
new file mode 100644
index 00000000000..cba6d098afb
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFile/expected/res/layout/layout.xml
@@ -0,0 +1,29 @@
+<FrameLayout xmlns:android="http://schemas.android.com/apk/res/android"
+             xmlns:tools="http://schemas.android.com/tools"
+             android:id="@+id/item_detail_container"
+             android:layout_width="match_parent"
+             android:layout_height="match_parent"
+             tools:context=".ItemDetailActivity"
+             tools:ignore="MergeRootFrame" >
+
+
+    <TextView
+        android:id="@+id/textView1"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:text="Enter your password" />
+
+    <EditText
+        android:id="@+id/password"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:ems="10"
+        android:inputType="textPassword" />
+
+    <Button
+        android:id="@+id/NEWNAME"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:text="Sign in" />
+
+</FrameLayout>
\ No newline at end of file
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFile/expected/res/layout/layout1.xml b/plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFile/expected/res/layout/layout1.xml
new file mode 100644
index 00000000000..7c48f2b615c
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFile/expected/res/layout/layout1.xml
@@ -0,0 +1,29 @@
+<FrameLayout xmlns:android="http://schemas.android.com/apk/res/android"
+             xmlns:tools="http://schemas.android.com/tools"
+             android:id="@+id/frameLayout"
+             android:layout_width="match_parent"
+             android:layout_height="match_parent"
+             tools:context=".ItemDetailActivity"
+             tools:ignore="MergeRootFrame" >
+
+
+    <TextView
+        android:id="@+id/passwordField"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:text="Enter your password" />
+
+    <EditText
+        android:id="@+id/passwordCaption"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:ems="10"
+        android:inputType="textPassword" />
+
+    <Button
+        android:id="@+id/loginButton"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:text="Sign in" />
+
+</FrameLayout>
\ No newline at end of file
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFileFragment/expected/multiFileFragment.kt b/plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFileFragment/expected/multiFileFragment.kt
new file mode 100644
index 00000000000..741345bf569
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFileFragment/expected/multiFileFragment.kt
@@ -0,0 +1,10 @@
+package com.myapp
+
+import android.app.Fragment
+import kotlinx.android.synthetic.main.layout.*
+
+class MyFragment: Fragment() {
+    val button = this.NEWNAME
+    val button1 = this.loginButton
+}
+
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFileFragment/expected/res/layout/layout.xml b/plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFileFragment/expected/res/layout/layout.xml
new file mode 100644
index 00000000000..cba6d098afb
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFileFragment/expected/res/layout/layout.xml
@@ -0,0 +1,29 @@
+<FrameLayout xmlns:android="http://schemas.android.com/apk/res/android"
+             xmlns:tools="http://schemas.android.com/tools"
+             android:id="@+id/item_detail_container"
+             android:layout_width="match_parent"
+             android:layout_height="match_parent"
+             tools:context=".ItemDetailActivity"
+             tools:ignore="MergeRootFrame" >
+
+
+    <TextView
+        android:id="@+id/textView1"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:text="Enter your password" />
+
+    <EditText
+        android:id="@+id/password"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:ems="10"
+        android:inputType="textPassword" />
+
+    <Button
+        android:id="@+id/NEWNAME"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:text="Sign in" />
+
+</FrameLayout>
\ No newline at end of file
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFileFragment/expected/res/layout/layout1.xml b/plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFileFragment/expected/res/layout/layout1.xml
new file mode 100644
index 00000000000..7c48f2b615c
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/multiFileFragment/expected/res/layout/layout1.xml
@@ -0,0 +1,29 @@
+<FrameLayout xmlns:android="http://schemas.android.com/apk/res/android"
+             xmlns:tools="http://schemas.android.com/tools"
+             android:id="@+id/frameLayout"
+             android:layout_width="match_parent"
+             android:layout_height="match_parent"
+             tools:context=".ItemDetailActivity"
+             tools:ignore="MergeRootFrame" >
+
+
+    <TextView
+        android:id="@+id/passwordField"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:text="Enter your password" />
+
+    <EditText
+        android:id="@+id/passwordCaption"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:ems="10"
+        android:inputType="textPassword" />
+
+    <Button
+        android:id="@+id/loginButton"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:text="Sign in" />
+
+</FrameLayout>
\ No newline at end of file
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/simple/expected/res/layout/layout.xml b/plugins/android-extensions/android-extensions-idea/testData/android/rename/simple/expected/res/layout/layout.xml
new file mode 100644
index 00000000000..8cf28d45f8a
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/simple/expected/res/layout/layout.xml
@@ -0,0 +1,17 @@
+<FrameLayout xmlns:android="http://schemas.android.com/apk/res/android"
+             xmlns:tools="http://schemas.android.com/tools"
+             android:id="@+id/item_detail_container"
+             android:layout_width="match_parent"
+             android:layout_height="match_parent"
+             tools:context=".ItemDetailActivity"
+             tools:ignore="MergeRootFrame" >
+
+
+
+    <Button
+        android:id="@+id/NEWNAME"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:text="Sign in" />
+
+</FrameLayout>
\ No newline at end of file
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/simple/expected/simple.kt b/plugins/android-extensions/android-extensions-idea/testData/android/rename/simple/expected/simple.kt
new file mode 100644
index 00000000000..7c5d7d6cda2
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/simple/expected/simple.kt
@@ -0,0 +1,12 @@
+package com.myapp
+
+import android.app.Activity
+import android.os.Bundle
+import java.io.File
+import kotlinx.android.synthetic.main.layout.*
+
+public class MyActivity : Activity() {
+    override fun onCreate(savedInstanceState: Bundle?) {}
+    val button = NEWNAME
+}
+
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/simpleFragment/expected/res/layout/layout.xml b/plugins/android-extensions/android-extensions-idea/testData/android/rename/simpleFragment/expected/res/layout/layout.xml
new file mode 100644
index 00000000000..8cf28d45f8a
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/simpleFragment/expected/res/layout/layout.xml
@@ -0,0 +1,17 @@
+<FrameLayout xmlns:android="http://schemas.android.com/apk/res/android"
+             xmlns:tools="http://schemas.android.com/tools"
+             android:id="@+id/item_detail_container"
+             android:layout_width="match_parent"
+             android:layout_height="match_parent"
+             tools:context=".ItemDetailActivity"
+             tools:ignore="MergeRootFrame" >
+
+
+
+    <Button
+        android:id="@+id/NEWNAME"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:text="Sign in" />
+
+</FrameLayout>
\ No newline at end of file
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/simpleFragment/expected/simpleFragment.kt b/plugins/android-extensions/android-extensions-idea/testData/android/rename/simpleFragment/expected/simpleFragment.kt
new file mode 100644
index 00000000000..781026d0469
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/simpleFragment/expected/simpleFragment.kt
@@ -0,0 +1,11 @@
+package com.myapp
+
+import android.app.Fragment
+import java.io.File
+import kotlinx.android.synthetic.main.layout.*
+
+public class MyFragment : Fragment() {
+    override fun onResume() {}
+    val button = NEWNAME
+}
+
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/simpleView/expected/res/layout/layout.xml b/plugins/android-extensions/android-extensions-idea/testData/android/rename/simpleView/expected/res/layout/layout.xml
new file mode 100644
index 00000000000..8cf28d45f8a
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/simpleView/expected/res/layout/layout.xml
@@ -0,0 +1,17 @@
+<FrameLayout xmlns:android="http://schemas.android.com/apk/res/android"
+             xmlns:tools="http://schemas.android.com/tools"
+             android:id="@+id/item_detail_container"
+             android:layout_width="match_parent"
+             android:layout_height="match_parent"
+             tools:context=".ItemDetailActivity"
+             tools:ignore="MergeRootFrame" >
+
+
+
+    <Button
+        android:id="@+id/NEWNAME"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:text="Sign in" />
+
+</FrameLayout>
\ No newline at end of file
diff --git a/plugins/android-extensions/android-extensions-idea/testData/android/rename/simpleView/expected/simpleView.kt b/plugins/android-extensions/android-extensions-idea/testData/android/rename/simpleView/expected/simpleView.kt
new file mode 100644
index 00000000000..cbf5b8ed14d
--- /dev/null
+++ b/plugins/android-extensions/android-extensions-idea/testData/android/rename/simpleView/expected/simpleView.kt
@@ -0,0 +1,9 @@
+package com.myapp
+
+import android.view.View
+import kotlinx.android.synthetic.main.layout.view.*
+
+fun View.a() {
+    val button = NEWNAME
+}
+
diff --git a/plugins/android-extensions/android-extensions-idea/tests/org/jetbrains/kotlin/android/AbstractAndroidRenameTest.kt b/plugins/android-extensions/android-extensions-idea/tests/org/jetbrains/kotlin/android/AbstractAndroidRenameTest.kt
index 24697a2978d..2cacc3a5017 100644
--- a/plugins/android-extensions/android-extensions-idea/tests/org/jetbrains/kotlin/android/AbstractAndroidRenameTest.kt
+++ b/plugins/android-extensions/android-extensions-idea/tests/org/jetbrains/kotlin/android/AbstractAndroidRenameTest.kt
@@ -17,9 +17,11 @@
 package org.jetbrains.kotlin.android
 
 import com.intellij.codeInsight.TargetElementUtil
+import com.intellij.openapi.vfs.LocalFileSystem
 import com.intellij.psi.impl.source.tree.injected.InjectedLanguageUtil
 import com.intellij.refactoring.rename.RenameProcessor
 import com.intellij.psi.impl.source.xml.XmlAttributeValueImpl
+import com.intellij.testFramework.PlatformTestUtil
 import org.jetbrains.kotlin.psi.KtNameReferenceExpression
 
 abstract class AbstractAndroidRenameTest : KotlinAndroidTestCase() {
@@ -27,22 +29,19 @@ abstract class AbstractAndroidRenameTest : KotlinAndroidTestCase() {
     private val NEW_ID_NAME = "@+id/$NEW_NAME"
 
     fun doTest(path: String) {
-        val f = myFixture!!
         getResourceDirs(path).forEach { myFixture.copyDirectoryToProject(it.name, it.name) }
-        val virtualFile = f.copyFileToProject(path + getTestName(true) + ".kt", "src/" + getTestName(true) + ".kt")
-        f.configureFromExistingVirtualFile(virtualFile)
-
-        val completionEditor = InjectedLanguageUtil.getEditorForInjectedLanguageNoCommit(f.editor, f.file)
-
-        val element = TargetElementUtil.findTargetElement(
-                completionEditor,
-                TargetElementUtil.REFERENCED_ELEMENT_ACCEPTED or TargetElementUtil.ELEMENT_NAME_ACCEPTED) as XmlAttributeValueImpl
-
-        RenameProcessor(f.project, element, NEW_ID_NAME, false, true).run()
-
-        val expression = TargetElementUtil.findReference(f.editor, f.caretOffset)!!.element as KtNameReferenceExpression
-        assertEquals(NEW_NAME, expression.getReferencedName())
+        val virtualFile = myFixture.copyFileToProject("$path${getTestName(true)}.kt", "src/${getTestName(true)}.kt")
+        myFixture.configureFromExistingVirtualFile(virtualFile)
+        myFixture.renameElement(myFixture.elementAtCaret, NEW_ID_NAME)
+        myFixture.checkResultByFile("expected/${getTestName(true)}.kt")
+        assertResourcesEqual("$path/expected/res")
     }
 
+    fun assertResourcesEqual(expectedPath: String) {
+        PlatformTestUtil.assertDirectoriesEqual(LocalFileSystem.getInstance().findFileByPath(expectedPath), getResourceDirectory())
+    }
+
+    fun getResourceDirectory() = LocalFileSystem.getInstance().findFileByPath(myFixture.tempDirPath + "/res")
+
     override fun getTestDataPath() = KotlinAndroidTestCaseBase.getPluginTestDataPathBase() + "/rename/" + getTestName(true) + "/"
 }
diff --git a/plugins/android-extensions/android-extensions-idea/tests/org/jetbrains/kotlin/android/AndroidRenameTestGenerated.java b/plugins/android-extensions/android-extensions-idea/tests/org/jetbrains/kotlin/android/AndroidRenameTestGenerated.java
index f48b0e2205d..50457fff799 100644
--- a/plugins/android-extensions/android-extensions-idea/tests/org/jetbrains/kotlin/android/AndroidRenameTestGenerated.java
+++ b/plugins/android-extensions/android-extensions-idea/tests/org/jetbrains/kotlin/android/AndroidRenameTestGenerated.java
@@ -36,6 +36,12 @@ public class AndroidRenameTestGenerated extends AbstractAndroidRenameTest {
         KotlinTestUtils.assertAllTestsPresentByMetadata(this.getClass(), new File("plugins/android-extensions/android-extensions-idea/testData/android/rename"), Pattern.compile("^([^\\.]+)$"), TargetBackend.ANY, false);
     }
 
+    @TestMetadata("commonElementId")
+    public void testCommonElementId() throws Exception {
+        String fileName = KotlinTestUtils.navigationMetadata("plugins/android-extensions/android-extensions-idea/testData/android/rename/commonElementId/");
+        doTest(fileName);
+    }
+
     @TestMetadata("fqNameInAttr")
     public void testFqNameInAttr() throws Exception {
         String fileName = KotlinTestUtils.navigationMetadata("plugins/android-extensions/android-extensions-idea/testData/android/rename/fqNameInAttr/");