Remove many (all?) test visual artifacts

Remove many (all?) test visual artifacts

Use utils.makeContainer many more places.
Update some components that render directly to body to take 
optional target and use that in tests.

https://code.launchpad.net/~bcsaller/juju-gui/containagedon/+merge/162551

(do not edit description out of merge proposal)

[bcsaller, 2013-05-06 01:02:53]

Please take a look.

[gary.poster, 2013-05-06 03:34:07]

Appspot diff is bad.  I looked at it on LP.  LGTM!  Thanks for cleanups.

Looks like you can remove this?

612	+      //mask.remove(true);

Gary

[bcsaller, 2013-05-06 03:36:36]

Thanks for the review, I removed the mask comment. The diff here looks ok to me,
but I didn't look at all the files.

https://codereview.appspot.com/9216043/diff/1/test/test_login.js
File test/test_login.js (left):

https://codereview.appspot.com/9216043/diff/1/test/test_login.js#oldcode117
test/test_login.js:117: 
Got it, thanks

[bac, 2013-05-06 12:43:51]

LGTM but please fix the indentation in index.html.  FWIW the diff is broken for
me to.  I'm not sure why this problem happens some time.

[bcsaller, 2013-05-06 14:28:31]

Thanks for review, fixed index as asked.

[bcsaller, 2013-05-06 14:33:41]

*** Submitted:

Remove many (all?) test visual artifacts

Use utils.makeContainer many more places.
Update some components that render directly to body to take 
optional target and use that in tests.

R=gary.poster, bac
CC=
https://codereview.appspot.com/9216043