Aura: Show dialog box asking user's permisssion for screen sharing for gtalk

chrome/browser/renderer_host/chrome_render_view_host_observer.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/renderer_host/chrome_render_view_host_observer.h"
 
 #include "base/command_line.h"
+#include "base/utf_string_conversions.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/net/predictor.h"
 #include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/simple_message_box.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/extensions/extension_messages.h"
 #include "chrome/common/render_messages.h"
 #include "chrome/common/url_constants.h"
 #include "content/public/browser/child_process_security_policy.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/render_view_host_delegate.h"
+#include "content/public/browser/render_widget_host_view.h"
 #include "content/public/browser/site_instance.h"
+#include "grit/generated_resources.h"
+#include "ui/base/l10n/l10n_util.h"
 
 using content::ChildProcessSecurityPolicy;
 using content::RenderViewHost;
 using content::SiteInstance;
 
 ChromeRenderViewHostObserver::ChromeRenderViewHostObserver(
     RenderViewHost* render_view_host, chrome_browser_net::Predictor* predictor)
     : content::RenderViewHostObserver(render_view_host),
       predictor_(predictor) {
   SiteInstance* site_instance = render_view_host->GetSiteInstance();
@@ skipping 31 matching lines @@
      (url.SchemeIs(chrome::kHttpScheme) || url.SchemeIs(chrome::kHttpsScheme)))
     predictor_->PreconnectUrlAndSubresources(url);
 }
 
 bool ChromeRenderViewHostObserver::OnMessageReceived(
     const IPC::Message& message) {
   bool handled = true;
   IPC_BEGIN_MESSAGE_MAP(ChromeRenderViewHostObserver, message)
     IPC_MESSAGE_HANDLER(ChromeViewHostMsg_FocusedEditableNodeTouched,
                         OnFocusedEditableNodeTouched)
+    IPC_MESSAGE_HANDLER(ChromeViewHostMsg_GetTalkPermission,
+                        OnGetTalkPermission)
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()
   return handled;
 }
 
 void ChromeRenderViewHostObserver::InitRenderViewHostForExtensions() {
   const Extension* extension = GetExtension();
   if (!extension)
     return;
 
@@ skipping 87 matching lines @@
   if (process_manager)
     process_manager->UnregisterRenderViewHost(rvh);
 }
 
 void ChromeRenderViewHostObserver::OnFocusedEditableNodeTouched() {
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_FOCUSED_EDITABLE_NODE_TOUCHED,
       content::Source<RenderViewHost>(render_view_host()),
       content::NotificationService::NoDetails());
 }
+
+void ChromeRenderViewHostObserver::OnGetTalkPermission(
+    const ppapi::HostResource& resource) {
+#if defined(USE_AURA)
+  string16 title = l10n_util::GetStringUTF16(
+      IDS_GTALK_SCREEN_SHARE_DIALOG_TITLE);
+  string16 message = l10n_util::GetStringUTF16(
+      IDS_GTALK_SCREEN_SHARE_DIALOG_MESSAGE);
+  int32_t result = browser::ShowYesNoBox(
+      render_view_host()->GetView()->GetNativeView(), title, message);
+
+  Send(new ChromeViewMsg_GetTalkPermissionACK(

[jam, 2012/03/15 02:05:26]

what you're doing here is basically a sync IPC, so just make it one for
simplicity (it'll simplify http://codereview.chromium.org/9700028/ a lot).

normally we don't allow sync IPCs to the browser UI thread, but that's only for
messages that are also handled on windows (with npapi plugins). in this case,
you don't have to worry about that, so make it a sync IPC. see the comment in
browser_message_filter.cc for more background

[brettw, 2012/03/15 17:44:32]

I talked to John and we can keep this async. I think it is better async.

[varunjain, 2012/03/15 17:54:15]

Done.

[varunjain, 2012/03/15 17:54:15]

reverted to async

+      render_view_host()->GetRoutingID(), resource, result));
+#endif
+}