Fixing flakiness of TextureLayerChangeInvisibleMailboxTest

cc/layers/texture_layer.cc

Index: cc/layers/texture_layer.cc
diff --git a/cc/layers/texture_layer.cc b/cc/layers/texture_layer.cc
index 802fb8d6164526a248b550ecd605c055015d8618..465b381815f4dbaccfa11c817e54bf772dc9e83c 100644
--- a/cc/layers/texture_layer.cc
+++ b/cc/layers/texture_layer.cc
@@ -299,8 +299,12 @@ void TextureLayer::TextureMailboxHolder::ReturnAndReleaseOnImplThread(
     bool is_lost,
     BlockingTaskRunner* main_thread_task_runner) {
   Return(sync_token, is_lost);
-  main_thread_task_runner->PostTask(
-      FROM_HERE, base::Bind(&TextureMailboxHolder::InternalRelease, this));
+  if (main_thread_task_runner->BelongsToCurrentThread()) {

[danakj, 2017/04/04 21:59:37]

You're right this orders it right, but I usually avoid this pattern of

if (on thread) { A } else { post to thread(A) }

The reason is that things that are expecting to be on a fresh call stack
suddenly are only sometimes on a fresh callstack which can lead to unexpected
re-entrancy or other bad side effects (like for instance a recent bug where
during paint we ended up destroying the compositor, which would then crash when
it tried to continue painting but it's destroyed).

So I'd prefer that we keep posting here, and just drop the ack if the frame sink
is released. wdyt?

[Saman Sami, 2017/04/04 22:12:44]

I see. Yes, dropping the ack should work. I sent out a patch doing that. Let's
see how the bots feel.

+    InternalRelease();
+  } else {
+    main_thread_task_runner->PostTask(
+        FROM_HERE, base::Bind(&TextureMailboxHolder::InternalRelease, this));
+  }
 }
 
 }  // namespace cc