New issue
Advanced search Search tips

Issue 675475 link

Starred by 2 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Task


Show other hotlists

Hotlists containing this issue:
layout-backlog


Sign in to add a comment

Move FullscreenController to the browser process

Project Member Reported by foolip@chromium.org, Dec 18 2016

Issue description

FullscreenController is currently owned by WebViewImpl in the renderer process, and in an out-of-process iframes scenario there will be multiple of them. Despite this, each tries to manage the global state of whether we're in fullscreen or trying to transition.

This is quite confusing. A more straightforward model would be if the renderer always sent the fullscreen request to the browser, and did not itself know about the current state beyond the fullscreen elements on documents.

Context: https://codereview.chromium.org/2573773002/diff/1/third_party/WebKit/Source/web/FullscreenController.cpp
 
Project Member

Comment 1 by sheriffbot@chromium.org, Feb 16 2018

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 2 by e...@chromium.org, Feb 22 2018

Labels: -Type-Bug Type-Task
Status: Available (was: Untriaged)

Sign in to add a comment