From 71aa97f70cfd5c1b847c5e31cc76eb788cac231d Mon Sep 17 00:00:00 2001
From: Eugene Kudelevsky <Eugene.Kudelevsky@jetbrains.com>
Date: Tue, 18 Dec 2012 17:03:14 +0400
Subject: [PATCH] logging for IDEA-88322

---
 .../augment/AndroidPsiAugmentProvider.java    | 28 +++++++++++++++++--
 1 file changed, 26 insertions(+), 2 deletions(-)

diff --git a/plugins/android/src/org/jetbrains/android/augment/AndroidPsiAugmentProvider.java b/plugins/android/src/org/jetbrains/android/augment/AndroidPsiAugmentProvider.java
index 117713c57c0f..2dc90f6b259d 100644
--- a/plugins/android/src/org/jetbrains/android/augment/AndroidPsiAugmentProvider.java
+++ b/plugins/android/src/org/jetbrains/android/augment/AndroidPsiAugmentProvider.java
@@ -1,6 +1,7 @@
 package org.jetbrains.android.augment;
 
 import com.android.resources.ResourceType;
+import com.intellij.openapi.diagnostic.Logger;
 import com.intellij.openapi.project.DumbService;
 import com.intellij.psi.PsiClass;
 import com.intellij.psi.PsiElement;
@@ -21,25 +22,41 @@ import java.util.*;
  * @author Eugene.Kudelevsky
  */
 public class AndroidPsiAugmentProvider extends PsiAugmentProvider {
+  private static final Logger LOG = Logger.getInstance("#org.jetbrains.android.augment.AndroidPsiAugmentProvider");
+
   @SuppressWarnings("unchecked")
   @NotNull
   @Override
   public <Psi extends PsiElement> List<Psi> getAugments(@NotNull PsiElement element, @NotNull Class<Psi> type) {
     if ((type != PsiClass.class && type != PsiField.class) ||
-        !(element instanceof PsiExtensibleClass) ||
-        DumbService.isDumb(element.getProject())) {
+        !(element instanceof PsiExtensibleClass)) {
       return Collections.emptyList();
     }
     final PsiExtensibleClass aClass = (PsiExtensibleClass)element;
     final String className = aClass.getName();
+    final boolean rClassAugment = AndroidUtils.R_CLASS_NAME.equals(className)
+                                  && type == PsiClass.class;
+
+    if (DumbService.isDumb(element.getProject())) {
+      if (rClassAugment) {
+        LOG.info("R_CLASS_AUGMENT: empty because of dumb mode");
+      }
+      return Collections.emptyList();
+    }
 
     final AndroidFacet facet = AndroidFacet.getInstance(element);
     if (facet == null) {
+      if (rClassAugment) {
+        LOG.info("R_CLASS_AUGMENT: empty because no facet");
+      }
       return Collections.emptyList();
     }
 
     final PsiFile containingFile = element.getContainingFile();
     if (containingFile == null) {
+      if (rClassAugment) {
+        LOG.info("R_CLASS_AUGMENT: empty because of no containing file");
+      }
       return Collections.emptyList();
     }
 
@@ -56,6 +73,9 @@ public class AndroidPsiAugmentProvider extends PsiAugmentProvider {
             result.add((Psi)resClass);
           }
         }
+        if (rClassAugment) {
+          LOG.info("R_CLASS_AUGMENT: " + result.size() + " classes added");
+        }
         return result;
       }
       else if (AndroidUtils.MANIFEST_CLASS_NAME.equals(className) &&
@@ -63,6 +83,10 @@ public class AndroidPsiAugmentProvider extends PsiAugmentProvider {
         return Arrays.asList((Psi)new PermissionClass(facet, aClass),
                              (Psi)new PermissionGroupClass(facet, aClass));
       }
+
+      if (rClassAugment) {
+        LOG.info("R_CLASS_AUGMENT: empty because containing file is not actual R.java file");
+      }
     }
     else if (type == PsiField.class && !(aClass instanceof AndroidLightClass)) {
       // extend existing inner classes, not provided by this augment (ex. they can be generated by maven)