Chromium Code Reviews| Index: Source/core/dom/Fullscreen.cpp |
| diff --git a/Source/core/dom/Fullscreen.cpp b/Source/core/dom/Fullscreen.cpp |
| index 552abaaa706744252e467126c67bdaaa4c3dd2ef..ec727d37a124b17d6646910862dab0a6d44e9453 100644 |
| --- a/Source/core/dom/Fullscreen.cpp |
| +++ b/Source/core/dom/Fullscreen.cpp |
| @@ -204,6 +204,18 @@ void Fullscreen::documentWasDisposed() |
| void Fullscreen::requestFullscreen(Element& element, RequestType requestType) |
| { |
| + // It is required by isPrivilegedContext() but isn't |
| + // actually used. This could be used later if a warning is shown in the |
| + // developer console. |
| + String errorMessage; |
| + if (document()->isPrivilegedContext(errorMessage)) { |
| + UseCounter::count(document(), UseCounter::FullscreenSecureOrigin); |
| + } else { |
| + UseCounter::countDeprecation(document(), UseCounter::FullscreenInsecureOrigin); |
| + } |
|
mlamouri (slow - plz ping)
2015/04/30 05:23:53
No need for { and }
jww
2015/04/30 17:28:44
Done.
|
| + // logPowerfulFeatureInsecureOriginWarning(document(), "fullscreen"); |
| + //} |
|
mlamouri (slow - plz ping)
2015/04/30 05:23:53
Remove?
jww
2015/04/30 17:28:45
Yikes, sorry! Done.
|
| + |
| // Ignore this request if the document is not in a live frame. |
| if (!document()->isActive()) |
| return; |