Commit 055f555c authored by Hirokazu Honda's avatar Hirokazu Honda Committed by Commit Bot

Revert "ozone: drm: Re-set cursor when GPU process connects"

This reverts commit 843847fc.

Reason for revert: VDA unittest on eve is broken.

Original change's description:
> ozone: drm: Re-set cursor when GPU process connects
> 
> When the GPU process restarts we need to re-send the cursor bitmap since
> all GPU process display state is lost.
> 
> It looks like it may have been broken all the way back to d818104f
> ("[Ozone-Drm] Notify cursor of channel established last"), which fixed a
> different issue but also caused the message that was intended to restore
> the cursor (in CommitBoundsChange) to not get delivered.
> 
> Bug: 908682
> Test: kill $(pgrep -f type=gpu-process), cursor restored
> 
> Change-Id: Ibacab25807995d9402c46f88a7d51b68fe8dfbfc
> Reviewed-on: https://chromium-review.googlesource.com/c/1351903
> Reviewed-by: Daniel Nicoara <dnicoara@chromium.org>
> Commit-Queue: Daniel Nicoara <dnicoara@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#611166}

TBR=spang@chromium.org,dnicoara@chromium.org

Change-Id: I6b69290c0d193190a0bcecc56819b3c59872eff8
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 908682
Reviewed-on: https://chromium-review.googlesource.com/c/1353038Reviewed-by: default avatarHirokazu Honda <hiroh@chromium.org>
Commit-Queue: Hirokazu Honda <hiroh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#611594}
parent 6e73ad4e
......@@ -49,7 +49,6 @@ void DrmCursor::SetDrmCursorProxy(std::unique_ptr<DrmCursorProxy> proxy) {
DCHECK(thread_checker_.CalledOnValidThread());
base::AutoLock lock(lock_);
proxy_ = std::move(proxy);
SendCursorShowLocked();
}
void DrmCursor::ResetDrmCursorProxy() {
......
......@@ -78,8 +78,9 @@ void CursorIPC::Send(IPC::Message* message) {
FROM_HERE, base::BindOnce(send_callback_, message)))
return;
// Drop disconnected updates. The cursor will get set once we connect, via
// SetDrmCursorProxy().
// Drop disconnected updates. DrmWindowHost will call
// CommitBoundsChange() when we connect to initialize the cursor
// location.
delete message;
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment