Mend CHANGE.log

2 files changed, 58 insertions, 11 deletions

diff --git a/ui/app/components/identicon.js b/ui/app/components/identicon.js
index ef625cc62..fd61b3125 100644
--- a/ui/app/components/identicon.js
+++ b/ui/app/components/identicon.js
@@ -1,8 +1,10 @@
 const Component = require('react').Component
 const h = require('react-hyperscript')
 const inherits = require('util').inherits
-const jazzicon = require('jazzicon')
 const findDOMNode = require('react-dom').findDOMNode
+const jazzicon = require('jazzicon')
+const iconFactoryGen = require('../../lib/icon-factory')
+const iconFactory = iconFactoryGen(jazzicon)
 
 module.exports = IdenticonComponent
 
@@ -35,21 +37,14 @@ IdenticonComponent.prototype.componentDidMount = function(){
   var address = state.address
 
   if (!address) return
-  var numericRepresentation = jsNumberForAddress(address)
 
   var container = findDOMNode(this)
-  // jazzicon with hack to fix inline svg error
+
   var diameter = state.diameter || this.defaultDiameter
-  var identicon = jazzicon(diameter, numericRepresentation)
-  var identiconSrc = identicon.innerHTML
-  var dataUri = 'data:image/svg+xml;charset=utf-8,'+encodeURIComponent(identiconSrc)
+  var dataUri = iconFactory.iconForAddress(address, diameter)
+
   var img = document.createElement('img')
   img.src = dataUri
   container.appendChild(img)
 }
 
-function jsNumberForAddress(address) {
-  var addr = address.slice(2, 10)
-  var seed = parseInt(addr, 16)
-  return seed
-}
diff --git a/ui/lib/icon-factory.js b/ui/lib/icon-factory.js
new file mode 100644
index 000000000..1b1df9490
--- /dev/null
+++ b/ui/lib/icon-factory.js
@@ -0,0 +1,52 @@
+var iconFactory
+
+module.exports = function(jazzicon) {
+  if (!iconFactory) {
+    iconFactory = new IconFactory(jazzicon)
+  }
+  return iconFactory
+}
+
+function IconFactory(jazzicon) {
+  this.jazzicon = jazzicon
+  this.cache = {}
+}
+
+IconFactory.prototype.iconForAddress = function(address, diameter) {
+  if (this.isCached(address, diameter)) {
+    return this.cache[address][diameter]
+  }
+
+  const dataUri = this.generateNewUri(address, diameter)
+  this.cacheIcon(address, diameter, dataUri)
+  return dataUri
+}
+
+IconFactory.prototype.generateNewUri = function(address, diameter) {
+  var numericRepresentation = jsNumberForAddress(address)
+  var identicon = this.jazzicon(diameter, numericRepresentation)
+  var identiconSrc = identicon.innerHTML
+  var dataUri = 'data:image/svg+xml;charset=utf-8,'+encodeURIComponent(identiconSrc)
+  return dataUri
+}
+
+IconFactory.prototype.cacheIcon = function(address, diameter, icon) {
+  if (!(address in this.cache)) {
+    var sizeCache = {}
+    sizeCache[diameter] = icon
+    return this.cache[address] = sizeCache
+
+  } else {
+    return this.cache[address][diameter] = icon
+  }
+}
+
+IconFactory.prototype.isCached = function(address, diameter) {
+  return address in this.cache && diameter in this.cache[address]
+}
+
+function jsNumberForAddress(address) {
+  var addr = address.slice(2, 10)
+  var seed = parseInt(addr, 16)
+  return seed
+}