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

Unified Diff: chrome/browser/media/desktop_media_list_base.cc

Issue 1622733002: Add CombinedDesktopMediaList class (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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
« no previous file with comments | « chrome/browser/media/desktop_media_list_base.h ('k') | chrome/browser/media/desktop_media_list_observer.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/media/desktop_media_list_base.cc
diff --git a/chrome/browser/media/desktop_media_list_base.cc b/chrome/browser/media/desktop_media_list_base.cc
index ce9941b8dbd96e81013e40160b96fa80eb9daff0..b4c9c2f1237dddcfea382e3f61379eb39627f22b 100644
--- a/chrome/browser/media/desktop_media_list_base.cc
+++ b/chrome/browser/media/desktop_media_list_base.cc
@@ -12,9 +12,7 @@
using content::BrowserThread;
DesktopMediaListBase::DesktopMediaListBase(base::TimeDelta update_period)
- : weak_factory_(this) {
- update_period_ = update_period;
-}
+ : update_period_(update_period), weak_factory_(this) {}
DesktopMediaListBase::~DesktopMediaListBase() {}
@@ -66,7 +64,7 @@ void DesktopMediaListBase::UpdateSourcesList(
for (size_t i = 0; i < sources_.size(); ++i) {
if (new_source_set.find(sources_[i].id) == new_source_set.end()) {
sources_.erase(sources_.begin() + i);
- observer_->OnSourceRemoved(i);
+ observer_->OnSourceRemoved(this, i);
--i;
}
}
@@ -82,7 +80,7 @@ void DesktopMediaListBase::UpdateSourcesList(
sources_.insert(sources_.begin() + i, Source());
sources_[i].id = new_sources[i].id;
sources_[i].name = new_sources[i].name;
- observer_->OnSourceAdded(i);
+ observer_->OnSourceAdded(this, i);
}
}
}
@@ -106,12 +104,12 @@ void DesktopMediaListBase::UpdateSourcesList(
sources_.erase(sources_.begin() + old_pos);
sources_.insert(sources_.begin() + pos, temp);
- observer_->OnSourceMoved(old_pos, pos);
+ observer_->OnSourceMoved(this, old_pos, pos);
}
if (sources_[pos].name != new_sources[pos].name) {
sources_[pos].name = new_sources[pos].name;
- observer_->OnSourceNameChanged(pos);
+ observer_->OnSourceNameChanged(this, pos);
}
++pos;
}
@@ -122,7 +120,7 @@ void DesktopMediaListBase::UpdateSourceThumbnail(content::DesktopMediaID id,
for (size_t i = 0; i < sources_.size(); ++i) {
if (sources_[i].id == id) {
sources_[i].thumbnail = image;
- observer_->OnSourceThumbnailChanged(i);
+ observer_->OnSourceThumbnailChanged(this, i);
break;
}
}
« no previous file with comments | « chrome/browser/media/desktop_media_list_base.h ('k') | chrome/browser/media/desktop_media_list_observer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698