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

Side by Side Diff: chrome/browser/media/media_stream_capture_indicator.cc

Issue 11273060: Change MediaStreamCaptureIndicator back to ref counted (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 1 month 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
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "chrome/browser/media/media_stream_capture_indicator.h" 5 #include "chrome/browser/media/media_stream_capture_indicator.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/i18n/rtl.h" 8 #include "base/i18n/rtl.h"
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "base/utf_string_conversions.h" 10 #include "base/utf_string_conversions.h"
(...skipping 107 matching lines...) Expand 10 before | Expand all | Expand 10 after
118 const MediaStreamCaptureIndicator::CaptureDeviceTab& tab) { 118 const MediaStreamCaptureIndicator::CaptureDeviceTab& tab) {
119 return (render_process_id_ == tab.render_process_id && 119 return (render_process_id_ == tab.render_process_id &&
120 render_view_id_ == tab.render_view_id); 120 render_view_id_ == tab.render_view_id);
121 } 121 }
122 122
123 MediaStreamCaptureIndicator::MediaStreamCaptureIndicator() 123 MediaStreamCaptureIndicator::MediaStreamCaptureIndicator()
124 : status_icon_(NULL), 124 : status_icon_(NULL),
125 mic_image_(NULL), 125 mic_image_(NULL),
126 camera_image_(NULL), 126 camera_image_(NULL),
127 balloon_image_(NULL), 127 balloon_image_(NULL),
128 request_index_(0), 128 request_index_(0) {
129 weak_ptr_factory_(ALLOW_THIS_IN_INITIALIZER_LIST(this)) {
130 } 129 }
131 130
132 MediaStreamCaptureIndicator::~MediaStreamCaptureIndicator() { 131 MediaStreamCaptureIndicator::~MediaStreamCaptureIndicator() {
133 // The user is responsible for cleaning up by closing all the opened devices. 132 // The user is responsible for cleaning up by closing all the opened devices.
134 DCHECK(tabs_.empty()); 133 DCHECK(tabs_.empty());
135 } 134 }
136 135
137 bool MediaStreamCaptureIndicator::IsCommandIdChecked( 136 bool MediaStreamCaptureIndicator::IsCommandIdChecked(
138 int command_id) const { 137 int command_id) const {
139 NOTIMPLEMENTED() << "There are no checked items in the MediaStream menu."; 138 NOTIMPLEMENTED() << "There are no checked items in the MediaStream menu.";
(...skipping 30 matching lines...) Expand all
170 void MediaStreamCaptureIndicator::CaptureDevicesOpened( 169 void MediaStreamCaptureIndicator::CaptureDevicesOpened(
171 int render_process_id, 170 int render_process_id,
172 int render_view_id, 171 int render_view_id,
173 const content::MediaStreamDevices& devices) { 172 const content::MediaStreamDevices& devices) {
174 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO)); 173 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
175 DCHECK(!devices.empty()); 174 DCHECK(!devices.empty());
176 175
177 BrowserThread::PostTask( 176 BrowserThread::PostTask(
178 BrowserThread::UI, FROM_HERE, 177 BrowserThread::UI, FROM_HERE,
179 base::Bind(&MediaStreamCaptureIndicator::DoDevicesOpenedOnUIThread, 178 base::Bind(&MediaStreamCaptureIndicator::DoDevicesOpenedOnUIThread,
180 weak_ptr_factory_.GetWeakPtr(), 179 this, render_process_id, render_view_id, devices));
181 render_process_id, render_view_id, devices));
182 } 180 }
183 181
184 void MediaStreamCaptureIndicator::CaptureDevicesClosed( 182 void MediaStreamCaptureIndicator::CaptureDevicesClosed(
185 int render_process_id, 183 int render_process_id,
186 int render_view_id, 184 int render_view_id,
187 const content::MediaStreamDevices& devices) { 185 const content::MediaStreamDevices& devices) {
188 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO)); 186 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
189 DCHECK(!devices.empty()); 187 DCHECK(!devices.empty());
190 188
191 BrowserThread::PostTask( 189 BrowserThread::PostTask(
192 BrowserThread::UI, FROM_HERE, 190 BrowserThread::UI, FROM_HERE,
193 base::Bind(&MediaStreamCaptureIndicator::DoDevicesClosedOnUIThread, 191 base::Bind(&MediaStreamCaptureIndicator::DoDevicesClosedOnUIThread,
194 weak_ptr_factory_.GetWeakPtr(), 192 this, render_process_id, render_view_id, devices));
195 render_process_id, render_view_id, devices));
196 } 193 }
197 194
198 void MediaStreamCaptureIndicator::DoDevicesOpenedOnUIThread( 195 void MediaStreamCaptureIndicator::DoDevicesOpenedOnUIThread(
199 int render_process_id, 196 int render_process_id,
200 int render_view_id, 197 int render_view_id,
201 const content::MediaStreamDevices& devices) { 198 const content::MediaStreamDevices& devices) {
202 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 199 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
203 200
204 CreateStatusTray(); 201 CreateStatusTray();
205 202
(...skipping 264 matching lines...) Expand 10 before | Expand all | Expand 10 after
470 467
471 void MediaStreamCaptureIndicator::EnsureImageLoadingTracker() { 468 void MediaStreamCaptureIndicator::EnsureImageLoadingTracker() {
472 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 469 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
473 if (tracker_.get()) 470 if (tracker_.get())
474 return; 471 return;
475 472
476 tracker_.reset(new ImageLoadingTracker(this)); 473 tracker_.reset(new ImageLoadingTracker(this));
477 pending_messages_.clear(); 474 pending_messages_.clear();
478 request_index_ = 0; 475 request_index_ = 0;
479 } 476 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698