Commit 9dd72195 authored by Egor Pasko's avatar Egor Pasko Committed by Commit Bot

Remove DCHECK from GetWindowClientInfoOnUI

As discussed in the bug, the condition under DCHECK is legitimate, and
the existing code does no harm in this case.

Bug: 923806
Change-Id: I5c46ac94174ba91184d6a5dad46702899abef95e
Reviewed-on: https://chromium-review.googlesource.com/c/1431613
Commit-Queue: Matt Falkenhagen <falken@chromium.org>
Reviewed-by: default avatarMatt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625438}
parent 5d52425f
......@@ -128,8 +128,6 @@ blink::mojom::ServiceWorkerClientInfoPtr GetWindowClientInfoOnUI(
// for a frame that is actually being navigated and isn't exactly what we are
// expecting.
PageVisibilityState visibility = render_frame_host->GetVisibilityState();
// Service workers do no prerender, this would be an invalid visibility state.
DCHECK_NE(visibility, PageVisibilityState::kPrerender);
bool page_hidden = visibility != PageVisibilityState::kVisible;
return blink::mojom::ServiceWorkerClientInfo::New(
render_frame_host->GetLastCommittedURL(),
......
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