New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 116044 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2012
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Feature

Restricted
  • Only users with EditIssue permission may comment.



Sign in to add a comment

Weird behavior when switching from Chrome fullscreen to NaCl Fullscreen

Reported by jairbubb...@gmail.com, Feb 28 2012

Issue description

Windows 7 64 bits / pepper 18 / chrome 19.0.1054.0 

You can easily reproduced this issue with the sample fullscreen_tumbler.
1 - Go to chrome fullscreen by pressing F11, chrome says "You have gone fullscreen"
2 - Press Enter to go to NaCl Fullscreen, chromes says "localhost is now fullscreen", it seems ok but it's not: the nacl-module is not really fullscreen, it's just zoomed in the page
3 - you're now stuck in a weird state were you can't go back to NaCl Fullscreen anymore (after pressing esc) but the nacl-module remains zoomed.

What is the intended behavior? Should we allow switching to NaCl Fullscreen mode while in Chrome Fullscreen mode.

Please note that the only reason I'm using NaCl Fullscreen is that I need mouse lock. I think mixing both fulscreen modes is very tricky for the user (and the developper ^^).

Thx,
Julien




 

Comment 1 by scheib@chromium.org, Feb 28 2012

Cc: yzshen@chromium.org
Labels: Feature-FullScreen Hotlist-ImportantForGames

Comment 2 by scheib@chromium.org, Feb 29 2012

Cc: -yzshen@chromium.org
Labels: Feature-NaCl-SDK
Status: Available
Verified in Linux 18.0.1025.39, NaCl only issue, not appearing when running JavaScript fullscreen e.g. from http://media.tojicode.com/q3bsp/

This appears to be a logic issue with NaCl and how the element size is modified. The browser / tab full screen state changing works as expected. The NaCl element fails to shrink back to it's original size. (It does shrink when entering/exiting full screen without browser full screen).
Cc: dmichael@chromium.org bbudge@chromium.org
Labels: -Pri-2 Pri-1 Feature-Plugins-Pepper Mstone-19
Adding a couple more people who may have some ideas here.
Owner: brettw@chromium.org
Assigning to Brett to triage since this is currently unowned P1 M19

Comment 5 by laforge@google.com, Mar 7 2012

Status: Assigned
Available + Owner == Default Assgined
Owner: bbudge@chromium.org

Comment 7 Deleted

Verified on Windows (19.0.1062.0 canary).

Comment 9 Deleted

In the debugger, it looks like the plugin logic that decides if the plugin's transition to fullscreen is complete isn't working when Chrome is in fullscreen.
Darin and I have verified that this isn't a NaCl or Pepper problem, but rather the way Chrome goes into fullscreen mode. We have a simple HTML / JS test that reproduces this. Working on a fix.
Status: Started
http://codereview.chromium.org/9673001/
Status: Fixed
Tested this on Win7/64 bit - 19.0.1077.2 (Official Build 128265)
Steps followed:
1. Opened the Full screen tumble example and hit F11 to enter browser full screen
2. Hit Enter --> Entered Nacl Full screen
3. Hit Esc --> Doesn't get the tumbler out of zoomed in state, but comes out the Nacl FS as per the info-bar shown --> Filed a different bug http://code.google.com/p/chromium/issues/detail?id=119630
4. Hitting Enter gets the tumbler out if zoomed in state.
5. Hitting Enter again takes the tumbler back in to Nacl Full screen.
Project Member

Comment 16 by bugdroid1@chromium.org, Oct 13 2012

Labels: Restrict-AddIssueComment-Commit
This issue has been closed for some time. No one will pay attention to new comments.
If you are seeing this bug or have new data, please click New Issue to start a new bug.
Project Member

Comment 17 by bugdroid1@chromium.org, Mar 10 2013

Labels: -Area-Internals -Feature-NaCl -Feature-FullScreen -Feature-NaCl-SDK -Feature-Plugins-Pepper -Mstone-19 Cr-Platform-NaCl M-19 Cr-Internals Cr-Content-Plugins-Pepper Cr-Platform-NaCl-SDK Cr-UI-Browser-FullScreen
Project Member

Comment 18 by bugdroid1@chromium.org, Mar 14 2013

Labels: -Restrict-AddIssueComment-Commit Restrict-AddIssueComment-EditIssue
Project Member

Comment 19 by bugdroid1@chromium.org, Apr 6 2013

Labels: Cr-Blink
Project Member

Comment 20 by bugdroid1@chromium.org, Apr 6 2013

Labels: Cr-Internals-Plugins
Project Member

Comment 21 by bugdroid1@chromium.org, Apr 6 2013

Labels: -Cr-Content-Plugins-Pepper Cr-Internals-Plugins-Pepper

Sign in to add a comment