Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(476)

Issue 149101: Add an example of cancelling fullscreen mode to the fullscreen. (Closed)

Created:
11 years, 6 months ago by Eric U.
Modified:
9 years, 7 months ago
Reviewers:
vangelis
CC:
o3d-review_googlegroups.com
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 1

Patch Set 2 : Update patch to include an onunload in the unlikely event that it's necessary #

Unified diffs Side-by-side diffs Delta from patch set Stats (+32 lines, -0 lines) Patch
M samples/fullscreen.html View 1 4 chunks +32 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Eric U.
11 years, 6 months ago (2009-06-26 21:21:36 UTC) #1
vangelis
Minor point, otherwise LGTM http://codereview.chromium.org/149101/diff/1/2 File samples/fullscreen.html (right): http://codereview.chromium.org/149101/diff/1/2#newcode330 Line 330: clearTimeout(timeoutId); do we need ...
11 years, 6 months ago (2009-06-26 22:40:33 UTC) #2
Eric U.
11 years, 5 months ago (2009-06-30 01:03:07 UTC) #3
vangelis
11 years, 5 months ago (2009-06-30 01:51:47 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld 408576698