Add interface for DesktopMediaPickerModel

chrome/browser/media/desktop_media_picker_model.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_MEDIA_DESKTOP_MEDIA_PICKER_MODEL_H_
 #define CHROME_BROWSER_MEDIA_DESKTOP_MEDIA_PICKER_MODEL_H_
 
 #include "base/basictypes.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
@@ skipping 31 matching lines @@
     // Id of the source.
     content::DesktopMediaID id;
 
     // Name of the source that should be shown to the user.
     string16 name;
 
     // The thumbnail for the source.
     gfx::ImageSkia thumbnail;
   };
 
-  // Caller may pass NULL for either of the arguments in case when only some
-  // types of sources the model should be populated with (e.g. it will only
-  // contain windows, if |screen_capturer| is NULL).
-  DesktopMediaPickerModel(scoped_ptr<webrtc::ScreenCapturer> screen_capturer,
-                          scoped_ptr<webrtc::WindowCapturer> window_capturer);
-  virtual ~DesktopMediaPickerModel();
+  virtual ~DesktopMediaPickerModel() {}
 
   // Sets time interval between updates. By default list of sources and their
   // thumbnail are updated once per second. If called after StartUpdating() then
   // it will take effect only after the next update.
-  void SetUpdatePeriod(base::TimeDelta period);
+  virtual void SetUpdatePeriod(base::TimeDelta period) = 0;
 
   // Sets size to which the thumbnails should be scaled. If called after
   // StartUpdating() then some thumbnails may be still scaled to the old size
   // until they are updated.
-  void SetThumbnailSize(const gfx::Size& thumbnail_size);
+  virtual void SetThumbnailSize(const gfx::Size& thumbnail_size) = 0;
 
   // Starts updating the model. The model is initially empty, so OnSourceAdded()
   // notifications will be generated for each existing source as it is
   // enumerated. After the initial enumeration the model will be refreshed based
   // on the update period, and notifications generated only for changes in the
   // model.
-  void StartUpdating(Observer* observer);
+  virtual void StartUpdating(Observer* observer) = 0;
 
-  int source_count() const { return sources_.size(); }
-  const Source& source(int index) const { return sources_.at(index); }
+  virtual int source_count() const = 0;
+  virtual const Source& source(int index) const = 0;
+
+ private:
+  DISALLOW_ASSIGN(DesktopMediaPickerModel);

[Sergey Ulanov, 2013/09/04 22:50:06]

nit: Why not DISALLOW_COPY_AND_ASSIGN? In either case it's not really useful for
abstract interfaces, so you can just remove this line.

[dcaiafa, 2013/09/04 23:00:27]

Done.

+};
+
+class DesktopMediaPickerModelImpl : public DesktopMediaPickerModel {
+ public:
+  // Caller may pass NULL for either of the arguments in case when only some
+  // types of sources the model should be populated with (e.g. it will only
+  // contain windows, if |screen_capturer| is NULL).
+  DesktopMediaPickerModelImpl(
+      scoped_ptr<webrtc::ScreenCapturer> screen_capturer,
+      scoped_ptr<webrtc::WindowCapturer> window_capturer);
+  virtual ~DesktopMediaPickerModelImpl();
+
+  // DesktopMediaPickerModelInterface:
+  virtual void SetUpdatePeriod(base::TimeDelta period) OVERRIDE;
+  virtual void SetThumbnailSize(const gfx::Size& thumbnail_size) OVERRIDE;
+  virtual void StartUpdating(Observer* observer) OVERRIDE;
+  virtual int source_count() const OVERRIDE;
+  virtual const Source& source(int index) const OVERRIDE;
 
  private:
   class Worker;
   friend class Worker;
 
   // Struct used to represent sources list the model gets from the Worker.
   struct SourceDescription {
     SourceDescription(content::DesktopMediaID id, const string16& name);
 
     content::DesktopMediaID id;
@@ skipping 32 matching lines @@
   scoped_refptr<base::SequencedTaskRunner> capture_task_runner_;
 
   // An object that does all the work of getting list of sources on a background
   // thread (see |capture_task_runner_|). Destroyed on |capture_task_runner_|
   // after the model is destroyed.
   scoped_ptr<Worker> worker_;
 
   // Current list of sources.
   std::vector<Source> sources_;
 
-  base::WeakPtrFactory<DesktopMediaPickerModel> weak_factory_;
+  base::WeakPtrFactory<DesktopMediaPickerModelImpl> weak_factory_;
 
-  DISALLOW_COPY_AND_ASSIGN(DesktopMediaPickerModel);
+  DISALLOW_COPY_AND_ASSIGN(DesktopMediaPickerModelImpl);
 };
 
 #endif  // CHROME_BROWSER_MEDIA_DESKTOP_MEDIA_PICKER_MODEL_H_