Speech refactoring: Reimplemented SpeechRecognitionManagerImpl as a FSM. (CL1.7)

content/public/browser/speech_recognition_session_config.h

+// 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.
+
+#ifndef CONTENT_PUBLIC_BROWSER_SPEECH_RECOGNITION_SESSION_CONFIG_H_
+#define CONTENT_PUBLIC_BROWSER_SPEECH_RECOGNITION_SESSION_CONFIG_H_
+
+#include "base/basictypes.h"
+#include "content/common/content_export.h"
+#include "content/public/browser/speech_recognition_session_context.h"
+
+namespace net {
+class URLRequestContextGetter;
+}
+
+namespace content {
+
+// Configuration params for creating a new speech recognition session.
+struct CONTENT_EXPORT SpeechRecognitionSessionConfig {
+  SpeechRecognitionSessionConfig();
+  ~SpeechRecognitionSessionConfig();
+
+  std::string language;
+  std::string grammar;
+  std::string origin_url;
+  bool filter_profanities;
+  SpeechRecognitionSessionContext initial_context;
+  net::URLRequestContextGetter* url_request_context_getter;

[jam, 2012/04/24 15:56:32]

nit: why not hold this in scoped_refcounted?

[Primiano Tucci (use gerrit), 2012/04/25 11:30:03]

Ah, didn't noted that it is refcounted. But this will require to include
"net/url_request/url_request_context_getter.h" in the .cc file. Is it ok? After
all this struct is only used synchronously for initialization and is never held.

[jam, 2012/04/25 15:14:08]

yeah there are no issues with including other .h files from this cc file. it
seems safer to wrap this in a scoped_refptr

[Primiano Tucci (use gerrit), 2012/04/25 16:55:48]

Done.

+};
+
+}  // namespace content
+
+#endif  // CONTENT_PUBLIC_BROWSER_SPEECH_RECOGNITION_SESSION_CONFIG_H_