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

content/public/common/speech_recognition_error.h

Index: content/public/common/speech_recognition_error.h
diff --git a/content/public/common/speech_recognition_error.h b/content/public/common/speech_recognition_error.h
index 10af480d9abf18c661655175c1bb2c71ebb71e95..e00a84e77dc1e6b27fd8120221d237f1b07d8de1 100644
--- a/content/public/common/speech_recognition_error.h
+++ b/content/public/common/speech_recognition_error.h
@@ -4,6 +4,7 @@
 
 #ifndef CONTENT_PUBLIC_COMMON_SPEECH_RECOGNITION_ERROR_H_
 #define CONTENT_PUBLIC_COMMON_SPEECH_RECOGNITION_ERROR_H_
+#pragma once
 
 namespace content {
 
@@ -37,13 +38,19 @@ struct CONTENT_EXPORT SpeechRecognitionError {
   SpeechRecognitionErrorCode code;
   SpeechAudioErrorDetails details;
 
-  SpeechRecognitionError(SpeechRecognitionErrorCode code_value)
+  SpeechRecognitionError()

[Satish, 2012/04/27 10:04:41]

style guide forbids inlining constructors in header files, so can you add a .cc
and move the body of these into that?

[hans, 2012/04/27 10:53:30]

I think in practice it's fine to avoid the extra .cc file as long as the
style-checker plugin doesn't complain.

See for example jam's second comment in
http://codereview.chromium.org/9972011/diff/45003/content/public/browser/spee...
where he questions the need to out-of-line the ctor/dtor.

The style guide rule is there to avoid inlining complex constructors.

[Satish, 2012/04/27 11:08:02]

Thats what I thought earlier, but the style guide above says "Even for simple
classes, putting the function in the .cc file helps avoid this when someone
later adds a new variable to the class." - which is true. I'm happy either way
but want to be consistent, and per style guide consistent means not having any
inline constructors.

jam@, do you have a strong preference to inline constructors? If so, should we
allow it in the style guide and not leave it as is?

[jam, 2012/04/27 14:43:26]

For files in content/public we have tried to avoid cc files as much as possible,
only adding them so that clank doesn't complain. The reasoning is that it's a
public directory, so (ideally) should just be headers

[Satish, 2012/04/27 14:45:34]

Cool, makes sense. Is it ok to mention this in the style guide as the one
exception to the inlining rule?

+      : code(SPEECH_RECOGNITION_ERROR_NONE),
+        details(SPEECH_AUDIO_ERROR_DETAILS_NONE) {
+  }
+  explicit SpeechRecognitionError(SpeechRecognitionErrorCode code_value)
       : 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