Introducing new data types and IPC messages for scripted JS speech recognition APIs (Speech CL2.0)

content/public/common/speech_recognition_error.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_COMMON_SPEECH_RECOGNITION_ERROR_H_
 #define CONTENT_PUBLIC_COMMON_SPEECH_RECOGNITION_ERROR_H_
 
 namespace content {
 
 // This enumeration follows the values described here:
@@ skipping 19 matching lines @@
 enum SpeechAudioErrorDetails {
   SPEECH_AUDIO_ERROR_DETAILS_NONE = 0,
   SPEECH_AUDIO_ERROR_DETAILS_NO_MIC,
   SPEECH_AUDIO_ERROR_DETAILS_IN_USE
 };
 
 struct CONTENT_EXPORT SpeechRecognitionError {
   SpeechRecognitionErrorCode code;
   SpeechAudioErrorDetails details;
 
+  SpeechRecognitionError()
+      : code(SPEECH_RECOGNITION_ERROR_NONE),
+        details(SPEECH_AUDIO_ERROR_DETAILS_NONE) {
+  }
   SpeechRecognitionError(SpeechRecognitionErrorCode code_value)

[hans, 2012/04/26 16:23:25]

i probably missed this before, but i believe this constructor should be marked
"explicit"

[Primiano Tucci (use gerrit), 2012/04/27 09:27:13]

uh, right.

       : code(code_value),
-        details(SPEECH_AUDIO_ERROR_DETAILS_NONE) {}
+        details(SPEECH_AUDIO_ERROR_DETAILS_NONE) {
+  }
   SpeechRecognitionError(SpeechRecognitionErrorCode code_value,
                          SpeechAudioErrorDetails details_value)
       : code(code_value),
-        details(details_value) {}
+        details(details_value) {
+  }
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_COMMON_SPEECH_RECOGNITION_ERROR_H_