media/capture: Remove constant argument in WebContentsTracker.

In the constructor:
~~~
  explicit WebContentsTracker(bool track_fullscreen_rwhv);
~~~
|track_fullscreen_rwhv| is always false. So it must be removed, with its
implementation.

Bug: None.
Change-Id: Icccac2143d6150cd00d040853cb2c610aba4aa0e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1841340Reviewed-by: Yuri Wiitala <miu@chromium.org>
Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#703652}

content/browser/media/capture/web_contents_audio_input_stream.cc

@@ -385,7 +385,7 @@ WebContentsAudioInputStream* WebContentsAudioInputStream::Create(
 
   return new WebContentsAudioInputStream(
       media_id.render_process_id, media_id.main_render_frame_id,
-      audio_mirroring_manager, new WebContentsTracker(false),
+      audio_mirroring_manager, new WebContentsTracker(),
       new media::VirtualAudioInputStream(
           params, worker_task_runner,
           media::VirtualAudioInputStream::AfterCloseCallback()),

content/browser/media/capture/web_contents_audio_input_stream_unittest.cc

@@ -77,8 +77,7 @@ class MockAudioMirroringManager : public AudioMirroringManager {
 
 class MockWebContentsTracker : public WebContentsTracker {
  public:
-  MockWebContentsTracker() : WebContentsTracker(false) {}
-
+  MockWebContentsTracker() = default;
   MOCK_METHOD3(Start,
                void(int render_process_id, int render_frame_id,
                     const ChangeCallback& callback));

content/browser/media/capture/web_contents_tracker.cc

@@ -15,9 +15,7 @@
 
 namespace content {
 
-WebContentsTracker::WebContentsTracker(bool track_fullscreen_rwhv)
-    : track_fullscreen_rwhv_(track_fullscreen_rwhv),
-      last_target_view_(nullptr) {}
+WebContentsTracker::WebContentsTracker() : last_target_view_(nullptr) {}
 
 WebContentsTracker::~WebContentsTracker() {
   // Likely unintentional BUG if Stop() was not called before this point.
@@ -64,11 +62,6 @@ RenderWidgetHostView* WebContentsTracker::GetTargetView() const {
   if (!wc)
     return nullptr;
 
-  if (track_fullscreen_rwhv_) {
-    if (auto* view = wc->GetFullscreenRenderWidgetHostView())
-      return view;
-  }
-
   if (auto* view = wc->GetRenderWidgetHostView()) {
     // Make sure the RWHV is still associated with a RWH before considering the
     // view "alive." This is because a null RWH indicates the RWHV has had its

content/browser/media/capture/web_contents_tracker.h

@@ -33,11 +33,7 @@ class CONTENT_EXPORT WebContentsTracker
     : public base::RefCountedThreadSafe<WebContentsTracker>,
       public WebContentsObserver {
  public:
-  // If |track_fullscreen_rwhv| is true, the ChangeCallback will be run when a
-  // WebContents shows/destroys a fullscreen RenderWidgetHostView. If false,
-  // fullscreen events are ignored. Normally, specify true for video capture and
-  // false for audio capture.
-  explicit WebContentsTracker(bool track_fullscreen_rwhv);
+  WebContentsTracker();
 
   // Callback to indicate a new RenderWidgetHostView should be targeted for
   // capture. This is also invoked with false to indicate tracking will not
@@ -116,10 +112,6 @@ class CONTENT_EXPORT WebContentsTracker
   void DidShowFullscreenWidget() final;
   void DidDestroyFullscreenWidget() final;
 
-  // If true, the client is interested in the showing/destruction of fullscreen
-  // RenderWidgetHostViews.
-  const bool track_fullscreen_rwhv_;
-
   // Pointer to the RenderWidgetHostView provided in the last run of
   // |callback_|. This is used to eliminate duplicate callback runs.
   RenderWidgetHostView* last_target_view_;