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

chrome/browser/renderer_host/chrome_render_view_host_observer.cc

Index: chrome/browser/renderer_host/chrome_render_view_host_observer.cc
diff --git a/chrome/browser/renderer_host/chrome_render_view_host_observer.cc b/chrome/browser/renderer_host/chrome_render_view_host_observer.cc
index 91da264e0ab545f3eb2f1a97974b80e51963fef4..d18fbb7122337845ec9c3cc7f1a79c0091516528 100644
--- a/chrome/browser/renderer_host/chrome_render_view_host_observer.cc
+++ b/chrome/browser/renderer_host/chrome_render_view_host_observer.cc
@@ -5,9 +5,11 @@
 #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"
@@ -18,6 +20,7 @@
 #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"
 
 using content::ChildProcessSecurityPolicy;
@@ -70,6 +73,8 @@ bool ChromeRenderViewHostObserver::OnMessageReceived(
   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;
@@ -177,3 +182,14 @@ void ChromeRenderViewHostObserver::OnFocusedEditableNodeTouched() {
       content::Source<RenderViewHost>(render_view_host()),
       content::NotificationService::NoDetails());
 }
+
+void ChromeRenderViewHostObserver::OnGetTalkPermission(
+    const ppapi::HostResource& resource) {
+  string16 title = ASCIIToUTF16("System Dialog");
+  string16 message = ASCIIToUTF16("Share your screen?");

[Josh Horwich, 2012/03/14 18:41:45]

Should we be using l10n_util::GetStringUTF16 for these (along with resources in
chrome/app/generated_resources.grd, as per
http://www.chromium.org/developers/design-documents/ui-localization) ?

[varunjain, 2012/03/14 18:49:08]

the talk code already has localized versions of this string. I am thinking if we
want to relocalize here or when we do the complete screen casting dialog, send
localized strings from the plugin?

[Avi (use Gerrit), 2012/03/14 21:03:00]

I'm not familiar with the design of this area, but you can't just use English
here as I earlier noted and Josh reasserted. If you have an idea how to
reasonably handle this, go for it.

+  int32_t result = browser::ShowYesNoBox(
+      render_view_host()->GetView()->GetNativeView(), title, message);
+
+  Send(new ChromeViewMsg_GetTalkPermissionACK(
+      render_view_host()->GetRoutingID(), resource, result));
+}