Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(596)

Side by Side Diff: jingle/notifier/communicator/login_settings.h

Issue 6016001: Fix nits. Add 'const'. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Don't make accessors const Created 9 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « jingle/notifier/base/chrome_async_socket.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef JINGLE_NOTIFIER_COMMUNICATOR_LOGIN_SETTINGS_H_ 5 #ifndef JINGLE_NOTIFIER_COMMUNICATOR_LOGIN_SETTINGS_H_
6 #define JINGLE_NOTIFIER_COMMUNICATOR_LOGIN_SETTINGS_H_ 6 #define JINGLE_NOTIFIER_COMMUNICATOR_LOGIN_SETTINGS_H_
7 #include <string> 7 #include <string>
8 8
9 #include "jingle/notifier/communicator/xmpp_connection_generator.h" 9 #include "jingle/notifier/communicator/xmpp_connection_generator.h"
10 #include "talk/base/scoped_ptr.h" 10 #include "talk/base/scoped_ptr.h"
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
70 const ConnectionOptions& connection_options() const { 70 const ConnectionOptions& connection_options() const {
71 return *connection_options_.get(); 71 return *connection_options_.get();
72 } 72 }
73 73
74 void set_server_override(const net::HostPortPair& server); 74 void set_server_override(const net::HostPortPair& server);
75 void clear_server_override(); 75 void clear_server_override();
76 76
77 private: 77 private:
78 bool try_ssltcp_first_; 78 bool try_ssltcp_first_;
79 79
80 net::HostResolver* host_resolver_; 80 net::HostResolver* const host_resolver_;
81 net::CertVerifier* cert_verifier_; 81 net::CertVerifier* const cert_verifier_;
82 talk_base::scoped_array<ServerInformation> server_list_; 82 talk_base::scoped_array<ServerInformation> server_list_;
83 int server_count_; 83 int server_count_;
84 // Used to handle redirects 84 // Used to handle redirects
85 scoped_ptr<ServerInformation> server_override_; 85 scoped_ptr<ServerInformation> server_override_;
86 86
87 scoped_ptr<buzz::XmppClientSettings> user_settings_; 87 scoped_ptr<buzz::XmppClientSettings> user_settings_;
88 scoped_ptr<ConnectionOptions> connection_options_; 88 scoped_ptr<ConnectionOptions> connection_options_;
89 DISALLOW_COPY_AND_ASSIGN(LoginSettings); 89 DISALLOW_COPY_AND_ASSIGN(LoginSettings);
90 }; 90 };
91 } // namespace notifier 91 } // namespace notifier
92 #endif // JINGLE_NOTIFIER_COMMUNICATOR_LOGIN_SETTINGS_H_ 92 #endif // JINGLE_NOTIFIER_COMMUNICATOR_LOGIN_SETTINGS_H_
OLDNEW
« no previous file with comments | « jingle/notifier/base/chrome_async_socket.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698