Added support for other authentication mechanisms in jingle. This will allow ...

jingle/notifier/base/notifier_options.h

Index: jingle/notifier/base/notifier_options.h
===================================================================
--- jingle/notifier/base/notifier_options.h	(revision 77157)
+++ jingle/notifier/base/notifier_options.h	(working copy)
@@ -5,6 +5,8 @@
 #ifndef JINGLE_NOTIFIER_BASE_NOTIFIER_OPTIONS_H_
 #define JINGLE_NOTIFIER_BASE_NOTIFIER_OPTIONS_H_
 
+#include <string>
+
 #include "jingle/notifier/base/notification_method.h"
 #include "net/base/host_port_pair.h"
 
@@ -36,6 +38,10 @@
   // Indicates the method used by sync clients while sending and listening to
   // notifications.
   NotificationMethod notification_method;
+
+  // Specifies the auth mechanism to use ("X-GOOGLE-TOKEN", "X-OAUTH2" etc),
+  // If left empty, the underlying login code uses "X-GOOGLE-TOKEN" by default.
+  std::string auth_mechanism;

[akalin, 2011/03/08 22:55:37]

What do you think about using an enum to pass around instead?  It's a bit more
code (since you have to have an enum to string map somewhere) but it's clearer
in client code.  Also avoids empty-string-as-default.

[sanjeevr, 2011/03/08 23:01:41]

I used a string because it maps well to the SaslCookieMechanism class which also
takes in a string. That way, there are no new code changes needed to support a
new auth mechanism.  

[akalin, 2011/03/08 23:44:36]

Fair enough.  Although I expect we'll eventually standardize on oauth
eventually.

On 2011/03/08 23:01:41, sanjeevr wrote:

 };
 
 }  // namespace notifier