SpeechInputExtensionManager now interface with SpeechRecognitionManagerDelegate

content/public/browser/speech_recognition_session_context.h

Index: content/public/browser/speech_recognition_session_context.h
diff --git a/content/public/browser/speech_recognition_session_context.h b/content/public/browser/speech_recognition_session_context.h
index 4bfcc2939945c03c210986bfe449801cde55183f..dc4eb0457be5b07058f824b0ba53f222a9d95d06 100644
--- a/content/public/browser/speech_recognition_session_context.h
+++ b/content/public/browser/speech_recognition_session_context.h
@@ -6,6 +6,7 @@
 #define CONTENT_PUBLIC_BROWSER_SPEECH_RECOGNITION_SESSION_CONTEXT_H_
 #pragma once
 
+#include "base/string16.h"
 #include "content/common/content_export.h"
 #include "ui/gfx/rect.h"
 
@@ -21,18 +22,29 @@ namespace content {
 // SpeechRecognitionManager::GetSessionContext and
 // SpeechRecognitionManager::LookupSessionByContext methods).
 struct CONTENT_EXPORT SpeechRecognitionSessionContext {
-  SpeechRecognitionSessionContext()
-      : render_process_id(0),
-        render_view_id(0),
-        render_request_id(0),
-        js_handle_id(0) {}
-  ~SpeechRecognitionSessionContext() {}
+  SpeechRecognitionSessionContext();
+  ~SpeechRecognitionSessionContext();
 
   int render_process_id;
   int render_view_id;
   int render_request_id;
   int js_handle_id;
+
+  // Determines whether recognition shall use an active bubble on a page element
+  // (which coordinates are passed in |element_rect|) or a background tray icon.
+  bool use_bubble_on_element;

[Satish, 2012/05/20 21:40:26]

similar to jam's comment below, the concept of bubble should really be confined
to the chrome/ layer. I'm guessing in this case webkit is setting this flag to
true for x-webkit-speech api and false for JS API - if this is correct the
variable should probably be named accordingly (e.g. invoked_by_page_element) and
the chrome/ layer decides that it maps to whether it should show a bubble
attached to the element or not

[Primiano Tucci (use gerrit), 2012/05/21 16:03:13]

Right, renamed to requested_by_page_element

+
+  // The coordinates of the page element for placing the bubble (valid only when
+  // |use_bubble_on_element| = true).
   gfx::Rect element_rect;
+
+  // Determines whether a security notification balloon shall pop-up together
+  // with the tray icon (valid only when |use_bubble_on_element| = false).
+  bool show_security_balloon;

[jam, 2012/05/17 16:45:16]

this is a chrome concept and doesn't belong in content, i.e. different embedders
of content may treat security UI differently

[Primiano Tucci (use gerrit), 2012/05/21 16:03:13]

Renamed and added a TODO, since it will be removed very soon (but in the
meanwhile I need it in this CL to keep the behavior of speech input extensions
consistent).

+
+  // A texual description of the context (website, extension name) that is
+  // requesting recognition, for prompting security notifications to the user.
+  string16 context_name;
 };
 
 }  // namespace content