am e1b8e967: Merge "Add comment about ColorInfo structure" into studio-1.3-dev automerge: 0cec8b3 automerge: 36c4260

* commit 'e1b8e967a19e8779da7cd10f44f61e4dd36bc4ea':
  Add comment about ColorInfo structure

1 files changed, 4 insertions, 0 deletions

diff --git a/android/src/com/android/tools/idea/editors/theme/attributes/AttributesTableModel.java b/android/src/com/android/tools/idea/editors/theme/attributes/AttributesTableModel.java
index 5dc3e2759bf..93257cc42e1 100644
--- a/android/src/com/android/tools/idea/editors/theme/attributes/AttributesTableModel.java
+++ b/android/src/com/android/tools/idea/editors/theme/attributes/AttributesTableModel.java
@@ -532,6 +532,10 @@ public class AttributesTableModel extends AbstractTableModel implements CellSpan
       }
 
       final boolean succeeded;
+      // Color editing may return reference value, which can be the same as previous value
+      // in this cell, but updating table is still required because value that reference points
+      // to was changed. To preserve this information, ColorEditor returns ColorInfo data
+      // structure with value and boolean flag which shows whether reload should be forced.
       if (value instanceof ColorInfo) {
         ColorInfo info = (ColorInfo) value;
         succeeded = setAttributeValue(info.getResourceValue(), info.isForceReload());