Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: chrome/content/composer.js

Issue 5728222025089024: Issue #447: EHH - TypeError: properties is undefined in composer.js (Closed)
Patch Set: Support and implementation changes Created June 19, 2014, 7:14 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | metadata.gecko » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/content/composer.js
===================================================================
--- a/chrome/content/composer.js
+++ b/chrome/content/composer.js
@@ -110,21 +110,18 @@
}
}
-var atomService = Cc["@mozilla.org/atom-service;1"].getService(Ci.nsIAtomService);
-var selectedAtom = atomService.getAtom("selected-false");
-var anchorAtom = atomService.getAtom("anchor");
-
-function TreeView_getRowProperties(row, properties) {
- if (!this.selection.isSelected(row))
- properties.AppendElement(selectedAtom);
+function TreeView_getRowProperties(row) {
+ let properties = "selected-" + this.selection.isSelected(row);
var item = this.getItemAtIndex(row);
if (item && (item.nodeData.expression != "*" || item.nodeData == nodeData))
- properties.AppendElement(anchorAtom);
+ properties += " anchor";
+
+ return properties;
}
-function TreeView_getCellProperties(row, col, properties) {
- this.getRowProperties(row, properties);
+function TreeView_getCellProperties(row, col) {
+ this.getRowProperties(row);
}
/*********************
« no previous file with comments | « no previous file | metadata.gecko » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld