Refactor IntentPickerBubbleView::ShowBubble()

chrome/browser/ui/views/intent_picker_bubble_view.cc

Index: chrome/browser/ui/views/intent_picker_bubble_view.cc
diff --git a/chrome/browser/ui/views/intent_picker_bubble_view.cc b/chrome/browser/ui/views/intent_picker_bubble_view.cc
index 0d46d817d4822b33e4f3f64474e5f87bd3a8bc7d..1a622f16829dbd40cd982759628e01347e654f27 100644
--- a/chrome/browser/ui/views/intent_picker_bubble_view.cc
+++ b/chrome/browser/ui/views/intent_picker_bubble_view.cc
@@ -50,11 +50,10 @@ enum class Option : int { ALWAYS = -2, JUST_ONCE };
 
 // static
 void IntentPickerBubbleView::ShowBubble(
-    content::NavigationHandle* handle,
+    content::WebContents* contents,

[djacobo_, 2016/09/14 17:16:53]

nit: other parts of the class refer to |web_contents| instead of just
|contents|, I think it would be a bit cleaner to use |web_contents| in all the
class.

[Yusuke Sato, 2016/09/14 18:15:55]

Done.

     const std::vector<NameAndIcon>& app_info,
     const ThrottleCallback& throttle_cb) {
-  Browser* browser =
-      chrome::FindBrowserWithWebContents(handle->GetWebContents());
+  Browser* browser = chrome::FindBrowserWithWebContents(contents);
   if (!browser) {
     throttle_cb.Run(kAppTagNoneSelected,
                     arc::ArcNavigationThrottle::CloseReason::ERROR);
@@ -67,8 +66,8 @@ void IntentPickerBubbleView::ShowBubble(
     return;
   }
 
-  IntentPickerBubbleView* delegate = new IntentPickerBubbleView(
-      app_info, throttle_cb, handle->GetWebContents());
+  IntentPickerBubbleView* delegate =
+      new IntentPickerBubbleView(app_info, throttle_cb, contents);
   delegate->set_margins(gfx::Insets());
   delegate->set_parent_window(browser_view->GetNativeWindow());
   views::Widget* widget =