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

Side by Side Diff: chrome/browser/ui/cocoa/media_picker/desktop_media_picker_bridge.mm

Issue 1622733002: Add CombinedDesktopMediaList class (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed for android build 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 #import "chrome/browser/ui/cocoa/media_picker/desktop_media_picker_bridge.h" 5 #import "chrome/browser/ui/cocoa/media_picker/desktop_media_picker_bridge.h"
6 6
7 DesktopMediaPickerBridge::DesktopMediaPickerBridge( 7 DesktopMediaPickerBridge::DesktopMediaPickerBridge(
8 id<DesktopMediaPickerObserver> observer) 8 id<DesktopMediaPickerObserver> observer)
9 : observer_(observer) { 9 : observer_(observer) {
10 } 10 }
11 11
12 DesktopMediaPickerBridge::~DesktopMediaPickerBridge() { 12 DesktopMediaPickerBridge::~DesktopMediaPickerBridge() {
13 } 13 }
14 14
15 void DesktopMediaPickerBridge::OnSourceAdded(int index) { 15 void DesktopMediaPickerBridge::OnSourceAdded(DesktopMediaList* list,
qiangchen 2016/01/26 01:01:50 According to style guide. Comment the unused param
16 int index) {
16 [observer_ sourceAddedAtIndex:index]; 17 [observer_ sourceAddedAtIndex:index];
17 } 18 }
18 19
19 void DesktopMediaPickerBridge::OnSourceRemoved(int index) { 20 void DesktopMediaPickerBridge::OnSourceRemoved(DesktopMediaList* list,
21 int index) {
20 [observer_ sourceRemovedAtIndex:index]; 22 [observer_ sourceRemovedAtIndex:index];
21 } 23 }
22 24
23 void DesktopMediaPickerBridge::OnSourceMoved(int old_index, int new_index) { 25 void DesktopMediaPickerBridge::OnSourceMoved(DesktopMediaList* list,
26 int old_index,
27 int new_index) {
24 [observer_ sourceMovedFrom:old_index to:new_index]; 28 [observer_ sourceMovedFrom:old_index to:new_index];
25 } 29 }
26 30
27 void DesktopMediaPickerBridge::OnSourceNameChanged(int index) { 31 void DesktopMediaPickerBridge::OnSourceNameChanged(DesktopMediaList* list,
32 int index) {
28 [observer_ sourceNameChangedAtIndex:index]; 33 [observer_ sourceNameChangedAtIndex:index];
29 } 34 }
30 35
31 void DesktopMediaPickerBridge::OnSourceThumbnailChanged(int index) { 36 void DesktopMediaPickerBridge::OnSourceThumbnailChanged(DesktopMediaList* list,
37 int index) {
32 [observer_ sourceThumbnailChangedAtIndex:index]; 38 [observer_ sourceThumbnailChangedAtIndex:index];
33 } 39 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698