Tab Capture: Set size before changing target RWHV

content/browser/media/capture/web_contents_tracker.cc

 // Copyright (c) 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.
 
 #include "content/browser/media/capture/web_contents_tracker.h"
 
 #include "base/threading/thread_task_runner_handle.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/render_frame_host.h"
 #include "content/public/browser/render_widget_host_view.h"
@@ skipping 89 matching lines @@
   }
 
   task_runner_->PostTask(FROM_HERE,
                          base::Bind(&WebContentsTracker::MaybeDoCallback, this,
                                     is_still_tracking()));
 }
 
 void WebContentsTracker::MaybeDoCallback(bool was_still_tracking) {
   DCHECK(task_runner_->BelongsToCurrentThread());
 
+  // Notify of a size change just before notifying of a new target. This allows
+  // the downstream implementation to capture the first frame from the new
+  // target at the correct resolution. http://crbug.com/704277
+  if (was_still_tracking)
+    MaybeDoResizeCallback();
   if (!callback_.is_null())
     callback_.Run(was_still_tracking);
-  if (was_still_tracking)
-    MaybeDoResizeCallback();
 }
 
 void WebContentsTracker::MaybeDoResizeCallback() {
   DCHECK(task_runner_->BelongsToCurrentThread());
 
   if (!resize_callback_.is_null())
     resize_callback_.Run();
 }
 
 void WebContentsTracker::StartObservingWebContents(int render_process_id,
@@ skipping 52 matching lines @@
   DVLOG(1) << "DidShowFullscreenWidget()";
   OnPossibleTargetChange(false);
 }
 
 void WebContentsTracker::DidDestroyFullscreenWidget() {
   DVLOG(1) << "DidDestroyFullscreenWidget()";
   OnPossibleTargetChange(false);
 }
 
 }  // namespace content