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

Side by Side Diff: extensions/browser/api/display_source/wifi_display/wifi_display_media_service_impl.cc

Issue 2851173004: Eliminate bind callback that doesn't take a BindSourceInfo parameter. (Closed)
Patch Set: . Created 3 years, 7 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
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 #include "extensions/browser/api/display_source/wifi_display/wifi_display_media_ service_impl.h" 5 #include "extensions/browser/api/display_source/wifi_display/wifi_display_media_ service_impl.h"
6 6
7 #include <string> 7 #include <string>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/big_endian.h" 10 #include "base/big_endian.h"
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
44 void WiFiDisplayMediaServiceImpl::Create( 44 void WiFiDisplayMediaServiceImpl::Create(
45 mojom::WiFiDisplayMediaServiceRequest request) { 45 mojom::WiFiDisplayMediaServiceRequest request) {
46 DCHECK_CURRENTLY_ON(content::BrowserThread::IO); 46 DCHECK_CURRENTLY_ON(content::BrowserThread::IO);
47 auto* impl = new WiFiDisplayMediaServiceImpl(); 47 auto* impl = new WiFiDisplayMediaServiceImpl();
48 impl->binding_ = 48 impl->binding_ =
49 mojo::MakeStrongBinding(base::WrapUnique(impl), std::move(request)); 49 mojo::MakeStrongBinding(base::WrapUnique(impl), std::move(request));
50 } 50 }
51 51
52 // static 52 // static
53 void WiFiDisplayMediaServiceImpl::BindToRequest( 53 void WiFiDisplayMediaServiceImpl::BindToRequest(
54 const service_manager::BindSourceInfo& source_info,
54 mojom::WiFiDisplayMediaServiceRequest request) { 55 mojom::WiFiDisplayMediaServiceRequest request) {
55 BrowserThread::PostTask(BrowserThread::IO, FROM_HERE, 56 BrowserThread::PostTask(BrowserThread::IO, FROM_HERE,
56 base::Bind(WiFiDisplayMediaServiceImpl::Create, 57 base::Bind(WiFiDisplayMediaServiceImpl::Create,
57 base::Passed(std::move(request)))); 58 base::Passed(std::move(request))));
58 } 59 }
59 60
60 WiFiDisplayMediaServiceImpl::WiFiDisplayMediaServiceImpl() 61 WiFiDisplayMediaServiceImpl::WiFiDisplayMediaServiceImpl()
61 : last_send_code_(net::OK), weak_factory_(this) {} 62 : last_send_code_(net::OK), weak_factory_(this) {}
62 63
63 WiFiDisplayMediaServiceImpl::~WiFiDisplayMediaServiceImpl() {} 64 WiFiDisplayMediaServiceImpl::~WiFiDisplayMediaServiceImpl() {}
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
135 binding_->Close(); 136 binding_->Close();
136 return; 137 return;
137 } 138 }
138 DCHECK(!write_buffers_.empty()); 139 DCHECK(!write_buffers_.empty());
139 write_buffers_.pop(); 140 write_buffers_.pop();
140 if (!write_buffers_.empty()) 141 if (!write_buffers_.empty())
141 Send(); 142 Send();
142 } 143 }
143 144
144 } // namespace extensions 145 } // namespace extensions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698