Use WebContents instead of Browser to construct PermissionPrompt

chrome/browser/ui/cocoa/website_settings/permission_prompt_impl_views_mac.mm

Index: chrome/browser/ui/cocoa/website_settings/permission_prompt_impl_views_mac.mm
diff --git a/chrome/browser/ui/cocoa/website_settings/permission_prompt_impl_views_mac.mm b/chrome/browser/ui/cocoa/website_settings/permission_prompt_impl_views_mac.mm
index 55da7da287cac32566c3a51d66df51e6a79bb2a1..2b41deed065982f2f68e64d1d6c428a6975da068 100644
--- a/chrome/browser/ui/cocoa/website_settings/permission_prompt_impl_views_mac.mm
+++ b/chrome/browser/ui/cocoa/website_settings/permission_prompt_impl_views_mac.mm
@@ -32,8 +32,9 @@ views::BubbleBorder::Arrow PermissionPromptImpl::GetAnchorArrow() {
 }
 
 // static
-std::unique_ptr<PermissionPrompt> PermissionPrompt::Create(Browser* browser) {
+std::unique_ptr<PermissionPrompt> PermissionPrompt::Create(
+    content::WebContents* web_contents) {
   if (chrome::ToolkitViewsWebUIDialogsEnabled())
-    return base::WrapUnique(new PermissionPromptImpl(browser));
-  return base::MakeUnique<PermissionBubbleCocoa>(browser);
+    return base::WrapUnique(new PermissionPromptImpl(web_contents));
+  return base::MakeUnique<PermissionBubbleCocoa>(web_contents);

[tapted, 2016/09/15 03:43:52]

Can this just be changed to pass
`chrome::FindBrowserWithWebContents(web_contents)`? Seems most of the other
Cocoa changes aren't required then. And since PermissionBubbleCocoa* retains a
weak pointer to Browser* (and it would be invalid to create permission prompts
for WebContents not in a Browser*, such as for packaged apps) this seems like
the more sensible API.

Also if the current UI code doesn't have a dependency on WebContents, and it
doesn't need one, we shouldn't be adding one.

Although I suppose the same could be said for the views-backed
PermissionPromptImpl.. Do the downstream things for android need changes to the
toolkit-views implementation too? (That is, is it enough to just change the two
desktop implementations of PermissionPrompt::Create(..) rather than the
platform-specific prompt implementations themselves, which assume they are on a
Browser*?)

[lshang, 2016/09/15 09:45:06]

Thanks for your detailed comment Trent! 

I think your suggestion is a good idea, which can make this CL much simpler,
also avoid causing any potential bugs on desktop due to use WebContents instead
of Browser for implementation.

Done.

 }