Use manifest for streamlined hosted app creation for title and URL.

content/browser/manifest/manifest_manager_host.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/manifest/manifest_manager_host.h"
 
 #include "content/common/manifest_manager_messages.h"
 #include "content/public/browser/render_frame_host.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/common/manifest.h"
@@ skipping 26 matching lines @@
   return it != pending_callbacks_.end() ? it->second : 0;
 }
 
 void ManifestManagerHost::RenderFrameDeleted(
     RenderFrameHost* render_frame_host) {
   CallbackMap* callbacks = GetCallbackMapForFrame(render_frame_host);
   if (!callbacks)
     return;
 
   // Before deleting the callbacks, make sure they are called with a failure
-  // state.
-  CallbackMap::const_iterator it(callbacks);
-  for (; !it.IsAtEnd(); it.Advance())
-    it.GetCurrentValue()->Run(Manifest());
+  // state. Do this in a block so the iterator is destroyed before |callbacks|.
+  {
+    CallbackMap::const_iterator it(callbacks);
+    for (; !it.IsAtEnd(); it.Advance())
+      it.GetCurrentValue()->Run(Manifest());
+  }
 
+  delete callbacks;
   pending_callbacks_.erase(render_frame_host);
 }
 
 void ManifestManagerHost::GetManifest(RenderFrameHost* render_frame_host,
                                       const GetManifestCallback& callback) {
   CallbackMap* callbacks = GetCallbackMapForFrame(render_frame_host);
   if (!callbacks) {
     callbacks = new CallbackMap();
     pending_callbacks_[render_frame_host] = callbacks;
   }
@@ skipping 64 matching lines @@
 
   callback->Run(manifest);
   callbacks->Remove(request_id);
   if (callbacks->IsEmpty()) {
     delete callbacks;
     pending_callbacks_.erase(render_frame_host);
   }
 }
 
 } // namespace content