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

Unified Diff: remoting/jingle_glue/xmpp_iq_request.h

Issue 7744041: Use new callbacks in the IqRequest interface. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 4 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « remoting/jingle_glue/mock_objects.h ('k') | remoting/jingle_glue/xmpp_iq_request.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/jingle_glue/xmpp_iq_request.h
diff --git a/remoting/jingle_glue/xmpp_iq_request.h b/remoting/jingle_glue/xmpp_iq_request.h
index 2e1b8b441829385eb4c6fd0535c0f8d20015ba86..ce8d61bbb9cf3e29b98ce9db3e7dd821eb03b396 100644
--- a/remoting/jingle_glue/xmpp_iq_request.h
+++ b/remoting/jingle_glue/xmpp_iq_request.h
@@ -20,15 +20,13 @@ namespace remoting {
class XmppIqRequest : public IqRequest, public buzz::XmppIqHandler {
public:
- typedef Callback1<const buzz::XmlElement*>::Type ReplyCallback;
-
XmppIqRequest(MessageLoop* message_loop, buzz::XmppClient* xmpp_client);
virtual ~XmppIqRequest();
// IqRequest interface.
virtual void SendIq(const std::string& type, const std::string& addressee,
buzz::XmlElement* iq_body) OVERRIDE;
- virtual void set_callback(ReplyCallback* callback) OVERRIDE;
+ virtual void set_callback(const ReplyCallback& callback) OVERRIDE;
// buzz::XmppIqHandler interface.
virtual void IqResponse(buzz::XmppIqCookie cookie,
@@ -44,7 +42,7 @@ class XmppIqRequest : public IqRequest, public buzz::XmppIqHandler {
MessageLoop* message_loop_;
buzz::XmppClient* xmpp_client_;
buzz::XmppIqCookie cookie_;
- scoped_ptr<ReplyCallback> callback_;
+ ReplyCallback callback_;
};
} // namespace remoting
« no previous file with comments | « remoting/jingle_glue/mock_objects.h ('k') | remoting/jingle_glue/xmpp_iq_request.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698