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

Unified Diff: third_party/WebKit/Source/modules/mediastream/UserMediaController.cpp

Issue 2614143002: Use a new Supplement constructor for LocalFrame's supplements (Closed)
Patch Set: Created 3 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/modules/mediastream/UserMediaController.cpp
diff --git a/third_party/WebKit/Source/modules/mediastream/UserMediaController.cpp b/third_party/WebKit/Source/modules/mediastream/UserMediaController.cpp
index 2a38d77fc245d25860d9a19e16d1e672f234b272..6259ee0f172ec279558d160a3c4ceded258d1547 100644
--- a/third_party/WebKit/Source/modules/mediastream/UserMediaController.cpp
+++ b/third_party/WebKit/Source/modules/mediastream/UserMediaController.cpp
@@ -31,23 +31,19 @@ const char* UserMediaController::supplementName() {
}
UserMediaController::UserMediaController(
+ LocalFrame& frame,
std::unique_ptr<UserMediaClient> client)
- : m_client(std::move(client)) {}
+ : Supplement<LocalFrame>(frame), m_client(std::move(client)) {}
DEFINE_TRACE(UserMediaController) {
Supplement<LocalFrame>::trace(visitor);
}
-UserMediaController* UserMediaController::create(
- std::unique_ptr<UserMediaClient> client) {
- return new UserMediaController(std::move(client));
-}
-
void provideUserMediaTo(LocalFrame& frame,
std::unique_ptr<UserMediaClient> client) {
UserMediaController::provideTo(
frame, UserMediaController::supplementName(),
- UserMediaController::create(std::move(client)));
+ new UserMediaController(frame, std::move(client)));
}
} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698