[Web Intents] Display throbber when loading inline disposition.

chrome/browser/ui/intents/web_intent_inline_disposition_delegate.cc

 // Copyright (c) 2012 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 "chrome/browser/ui/intents/web_intent_inline_disposition_delegate.h"
 
 #include "base/logging.h"
+#include "chrome/browser/ui/intents/web_intent_picker.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/web_contents.h"
 
-WebIntentInlineDispositionDelegate::WebIntentInlineDispositionDelegate() {
+WebIntentInlineDispositionDelegate::WebIntentInlineDispositionDelegate(
+    WebIntentPicker* picker)
+    : picker_(picker) {
 }
 
 WebIntentInlineDispositionDelegate::~WebIntentInlineDispositionDelegate() {
 }
 
 bool WebIntentInlineDispositionDelegate::IsPopupOrPanel(
     const content::WebContents* source) const {
   return true;
 }
 
 bool WebIntentInlineDispositionDelegate::ShouldAddNavigationToHistory(
     const history::HistoryAddPageArgs& add_page_args,
     content::NavigationType navigation_type) {
   return false;
 }
 
 content::WebContents* WebIntentInlineDispositionDelegate::OpenURLFromTab(
     content::WebContents* source, const content::OpenURLParams& params) {
   DCHECK(source);  // Can only be invoked from inline disposition.
 
   // Load in place.
   source->GetController().LoadURL(params.url, content::Referrer(),
       content::PAGE_TRANSITION_START_PAGE, std::string());
 
   // Remove previous history entries - users should not navigate in intents.
   source->GetController().PruneAllButActive();
 
   return source;
 }
+
+void WebIntentInlineDispositionDelegate::LoadingStateChanged(
+    content::WebContents* source) {
+  if (!source->IsLoading()) {

[James Hawkins, 2012/04/11 18:05:46]

Optional nit: Save a line by removing braces.

[binji, 2012/04/12 17:48:49]

Done.

+    picker_->OnInlineDispositionWebContentsLoaded(source);
+  }
+}
+