Issue #447: EHH - TypeError: properties is undefined in composer.js

chrome/content/composer.js

Index: chrome/content/composer.js
===================================================================
--- a/chrome/content/composer.js
+++ b/chrome/content/composer.js
@@ -110,21 +110,20 @@
   }
 }
 
-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) {
+function TreeView_getRowProperties(row) {

[Wladimir Palant, 2014/06/18 17:20:48]

Element Hiding Helper currently supports Firefox 20. Please update
metadata.gecko for this extension to match the version requirements of Adblock
Plus.

[saroyanm, 2014/06/19 07:24:26]

Done.

+  let property = "";
   if (!this.selection.isSelected(row))
-    properties.AppendElement(selectedAtom);
+    property += "selected-false";

[Wladimir Palant, 2014/06/18 17:20:48]

Given that we no longer need atoms for all properties, this can be simpler:

let properties = "selected-" + this.selection.isSelected(row);

Then you won't need to check whether property is empty below.

Note that I named the variable "properties" above, it should be plural.

[saroyanm, 2014/06/19 07:24:26]

Good point, 
Thanks. 

 
   var item = this.getItemAtIndex(row);
   if (item && (item.nodeData.expression != "*" || item.nodeData == nodeData))
-    properties.AppendElement(anchorAtom);
+    property += (property ? " anchor" : "anchor");
+
+  return property;
 }
 
-function TreeView_getCellProperties(row, col, properties) {
-  this.getRowProperties(row, properties);
+function TreeView_getCellProperties(row, col) {
+  this.getRowProperties(row);
 }
 
 /*********************