Display origin url on the chooser permission dialog title

chrome/browser/ui/website_settings/chooser_bubble_controller.cc

Index: chrome/browser/ui/website_settings/chooser_bubble_controller.cc
diff --git a/chrome/browser/ui/website_settings/chooser_bubble_controller.cc b/chrome/browser/ui/website_settings/chooser_bubble_controller.cc
index 05113f0d6c2571cb833f93cf3d60a5da7b863d18..987835f7f3940237aa3efe3398fa3554136c7efe 100644
--- a/chrome/browser/ui/website_settings/chooser_bubble_controller.cc
+++ b/chrome/browser/ui/website_settings/chooser_bubble_controller.cc
@@ -6,6 +6,7 @@
 
 #include "chrome/browser/net/referrer.h"
 #include "chrome/browser/ui/browser_finder.h"
+#include "content/public/browser/render_frame_host.h"
 #include "content/public/browser/web_contents.h"
 
 ChooserBubbleController::ChooserBubbleController(
@@ -16,6 +17,11 @@ ChooserBubbleController::ChooserBubbleController(
 
 ChooserBubbleController::~ChooserBubbleController() {}
 
+GURL ChooserBubbleController::GetOriginUrl() const {
+  return const_cast<content::RenderFrameHost*>(owning_frame_)

[palmer, 2016/04/18 18:28:10]

Is this const_cast necessary? If so, why? I think it's because
|GetLastCommittedURL| (and GetLastCommittedOrigin) are not declared const? I
wonder if that could be changed/refactored?

[juncai, 2016/04/18 19:43:24]

Yes, it is necessary because GetLastCommittedOrigin() is not declared const. And
I asked this question on the chromium dev forum, they pointed out the
documentation:
http://www.chromium.org/developers/content-module/content-api
"don't add the const identifier to interfaces. For interfaces implemented by the
embedder, we can't make assumptions about what the embedder needs to implement
it. For interfaces implemented by content, the implementation details doesn't
have to be exposed."

+      ->GetLastCommittedURL();

[palmer, 2016/04/18 18:28:10]

Try GetLastCommittedOrigin.

[juncai, 2016/04/18 19:43:24]

Done.

+}
+
 void ChooserBubbleController::OpenHelpCenterUrl() const {
   browser_->OpenURL(content::OpenURLParams(
       GetHelpCenterUrl(), content::Referrer(), NEW_FOREGROUND_TAB,