Bug 1025552 - [Messages][Refactoring] Refactor attachment.js and specifically move rendering part to a separate module. r=julienw

I wanted to show that ```then``` callback on ```deferred.promise``` was only called once ```deferred.resolve``` is called and not earlier. Do you think it's redundant?