Chromium Code Reviews

Issue 477133002: Sync fullscreen element removal with the spec (again) (Closed)

Created:
6 years, 4 months ago by philipj_slow
Modified:
6 years, 4 months ago
Reviewers:
falken
CC:
blink-reviews, blink-reviews-dom_chromium.org, dglazkov+blink, sof, eae+blinkwatch, rwlbuis
Project:
blink
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : rebase #

Unified diffs Side-by-side diffs Stats (+9 lines, -9 lines)
M Source/core/dom/Fullscreen.cpp View 1 chunk +9 lines, -9 lines 0 comments

Messages

Total messages: 5 (0 generated)
philipj_slow
PTAL
6 years, 4 months ago (2014-08-15 14:09:55 UTC) #1
falken
lgtm
6 years, 4 months ago (2014-08-18 02:17:49 UTC) #2
philipj_slow
The CQ bit was checked by philipj@opera.com
6 years, 4 months ago (2014-08-18 08:41:17 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/philipj@opera.com/477133002/20001
6 years, 4 months ago (2014-08-18 08:42:15 UTC) #4
commit-bot: I haz the power
6 years, 4 months ago (2014-08-18 08:43:12 UTC) #5
Message was sent while issue was closed.
Committed patchset #2 (20001) as 180435

Powered by Google App Engine