Make 'reload' in skydb work again

Make 'reload' in skydb work again

Looks like it broke with:
https://codereview.chromium.org/741453002

I don't really like this fix, (it just leaks
the SkyApplication object) but I don't have
a better one at the moment.

I suspect the problem may actually be in
ApplicationManager::LoadWithContentHandler
in its attempt to re-use connections, unclear.

The bug was that you would launch skydb and then
try to 'reload' the url and the reload would
be ignored every other attempt.

Reload just tells the view to embed the url
again.  The ViewManager was telling its
client OnEmbed, but that message was never
arriving every other time.  After many printfs
I traced this to the DocumentView being destroyed
(which calls SkyApplication::OnViewDestroyed)
and causing the SkyApplication to disappear
right before the ViewManager tried to talk to it.

R=aa@chromium.org
BUG=

[eseidel, 2014-12-05 23:08:18]

eseidel@chromium.org changed reviewers:
+ qsr@chromium.org

[eseidel, 2014-12-05 23:08:19]

Looks like this broke with https://codereview.chromium.org/741453002

[Aaron Boodman, 2014-12-05 23:29:50]

https://codereview.chromium.org/780413002/diff/1/sky/viewer/content_handler_i...
File sky/viewer/content_handler_impl.cc (right):

https://codereview.chromium.org/780413002/diff/1/sky/viewer/content_handler_i...
sky/viewer/content_handler_impl.cc:60: // TODO(eseidel): We can't destroy
ourselves synchronously here
This seems like a reasonable fix for now. I don't think counting embeds is the
right way to manage app lifetime... but I don't think we know what the right way
is, either.

For more context, see:

https://codereview.chromium.org/776553003/diff/1/mojo/services/html_viewer/ht...
and 
https://groups.google.com/a/chromium.org/forum/#!topic/mojo-dev/d4G1y3d9zZc

[eseidel, 2014-12-05 23:33:04]

It appears that the code as-written creates a new SkyApplication each time
content is loaded.  So leaking isn't good.  But the existing tear-down also
doesn't make much sense since the Applications themselves are never reused...

[Aaron Boodman, 2014-12-06 00:02:30]

lgtm

[qsr, 2014-12-08 14:39:56]

On 2014/12/06 00:02:30, Aaron Boodman wrote:
> lgtm

 if we remove the cleanup code, let's remove everything:
https://codereview.chromium.org/783973002