Send correct termination reason when session is terminated by the host.

remoting/protocol/jingle_messages.h

Index: remoting/protocol/jingle_messages.h
diff --git a/remoting/protocol/jingle_messages.h b/remoting/protocol/jingle_messages.h
index eb1ec5c58a7da94ecd05a6c25cf6d87fc6442032..069d48f948879412f119d6232e3df31c59459b20 100644
--- a/remoting/protocol/jingle_messages.h
+++ b/remoting/protocol/jingle_messages.h
@@ -39,6 +39,7 @@ struct JingleMessage {
     UNKNOWN_REASON,
     SUCCESS,
     DECLINE,
+    GENERAL_ERROR,

[Wez, 2011/11/11 23:10:09]

How does a "general error" differ from an "unknown reason"?

[Sergey Ulanov, 2011/11/12 00:02:52]

It's basically the same. Jingle defines it as "The action is related to a
non-specific application error."

     INCOMPATIBLE_PARAMETERS,

[Wez, 2011/11/11 23:10:09]

These are all macro-style and don't have very consistent names. :(

[Sergey Ulanov, 2011/11/12 00:02:52]

That's required by style guide.
they correspond to names defined by jingle spec.

[Wez, 2011/11/15 02:30:05]

Style guide allows both forms and prefers non-macro-style unless it's for
consistency with existing code, though.

[Sergey Ulanov, 2011/11/15 02:37:21]

Chromium style guide (http://www.chromium.org/developers/coding-style) says:
  Though the style guide says to use kConstantNaming for enums now, we still use
MACRO_STYLE naming for enums for consistency with existing code.

It is worded ambiguously, but it think the right way to interpret is to always
use MACRO_STYLE for enums.

   };