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

Side by Side Diff: media/base/message_loop_factory.h

Issue 6171009: Remove MessageLoop methods from Filter interface (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Apply CR suggestions & fix style issues. 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
OLDNEW
(Empty)
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #ifndef MEDIA_BASE_MESSAGE_LOOP_FACTORY_H
scherkus (not reviewing) 2011/01/13 23:52:50 nit: header guards end with trailing _
acolwell GONE FROM CHROMIUM 2011/01/14 01:14:12 Done.
6 #define MEDIA_BASE_MESSAGE_LOOP_FACTORY_H
7
8 #include <string>
9
10 #include "base/scoped_ptr.h"
11
12 class MessageLoop;
13
14 namespace media {
15
16 // Factory object that manages named MessageLoops.
17 class MessageLoopFactory {
18 public:
19 // Get the message loop associated with |name|. A new MessageLoop
20 // is created if the factory doesn't have one associated with |name|.
21 // Null is returned if |name| is null, |name| is an empty string, or a new
scherkus (not reviewing) 2011/01/13 23:52:50 Null -> NULL also |name| can no longer be NULL
acolwell GONE FROM CHROMIUM 2011/01/14 01:14:12 Done.
22 // MessageLoop needs to be created and a failure occurs during the
23 // creation process.
24 virtual MessageLoop* GetMessageLoop(const std::string& name) = 0;
25
26 protected:
27 // Only allow scoped_ptr<> to delete factory.
28 friend class scoped_ptr<MessageLoopFactory>;
29 virtual ~MessageLoopFactory();
30 };
31
32 } // namespace media
33
34 #endif // MEDIA_BASE_MESSAGE_LOOP_FACTORY_H
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698