Fix for LoginFeedbackTest.Basic broken by recent change.

content/browser/media/capture/aura_window_capture_machine.cc

Index: content/browser/media/capture/aura_window_capture_machine.cc
diff --git a/content/browser/media/capture/aura_window_capture_machine.cc b/content/browser/media/capture/aura_window_capture_machine.cc
index a56208f38aa1fac4fada467d6c43e788d2f207c0..26e33cf70492cf922c8bc76fe2bafd11761f5ddc 100644
--- a/content/browser/media/capture/aura_window_capture_machine.cc
+++ b/content/browser/media/capture/aura_window_capture_machine.cc
@@ -124,8 +124,13 @@ void AuraWindowCaptureMachine::InternalStop(const base::Closure& callback) {
 }
 
 void AuraWindowCaptureMachine::MaybeCaptureForRefresh() {
-  DCHECK_CURRENTLY_ON(BrowserThread::UI);
-  Capture(false);
+  BrowserThread::PostTask(
+      BrowserThread::UI, FROM_HERE,
+      base::Bind(&AuraWindowCaptureMachine::Capture,
+                 // Use of Unretained() is safe here since this task must run
+                 // before InternalStop().
+                 base::Unretained(this),

[mcasas, 2016/04/07 19:12:57]

Actually, since |this| pointer would be used
on UI thread, you could use
weak_factory_.GetWeakPtr()
right?

https://code.google.com/p/chromium/codesearch#chromium/src/content/browser/me...

[miu, 2016/04/07 19:55:31]

tl;dr: It doesn't really matter, but the raw pointer is less overhead.

I had considered this.  However, we can easily reason that it's not needed:

1. Since we know MaybeCaptureForRefresh() must be called on the same thread as
Stop(), and will never be called after Stop(); then we know the tasks posted to
run on the UI thread will be sequenced such that the raw pointer will always be
valid.

2. If instead we post a task using a WeakPtr, the tasks running on the UI thread
will still have the same execution sequence.  The weak pointer could never be
invalidated in the meantime (because InternalStop() always has to run after the
task posted here).  So, using the WeakPtr doesn't add any value, but *does* add
overhead.

+                 false));
 }
 
 void AuraWindowCaptureMachine::SetWindow(aura::Window* window) {