Issue 477133002: Sync fullscreen element removal with the spec (again) - Code Review

Chromium Code Reviews

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

(121)

My Issues | Starred Open | Closed | All

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

Base URL:
svn://svn.chromium.org/blink/trunk

Project:
blink

Visibility:
Public.

More Reviews

Description

Sync fullscreen element removal with the spec (again) http://fullscreen.spec.whatwg.org/#model https://www.w3.org/Bugs/Public/show_bug.cgi?id=26568#c12 https://www.w3.org/Bugs/Public/show_bug.cgi?id=26568#c16 BUG=383813 NOTRY=true Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=180435

Patch Set 1 #

Patch Set 2 : rebase #

Created: 6 years, 4 months ago

Download [raw] [tar.bz2]

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

Messages

Total messages: 5 (0 generated)

Expand Messages | Collapse Messages

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

Expand Messages | Collapse Messages

Powered by Google App Engine

This is Rietveld 408576698