Chromium Code Reviews| Index: third_party/WebKit/Source/core/dom/Fullscreen.cpp |
| diff --git a/third_party/WebKit/Source/core/dom/Fullscreen.cpp b/third_party/WebKit/Source/core/dom/Fullscreen.cpp |
| index 8c97a4c44930dab5518a0139b8bedaaf65f6f351..a32d61069ec02699b9ce721265eba8f5fb61f7cd 100644 |
| --- a/third_party/WebKit/Source/core/dom/Fullscreen.cpp |
| +++ b/third_party/WebKit/Source/core/dom/Fullscreen.cpp |
| @@ -200,11 +200,7 @@ void Fullscreen::documentWasDisposed() |
| void Fullscreen::requestFullscreen(Element& element, RequestType requestType) |
| { |
| - // It is required by isSecureContext() but isn't |
| - // actually used. This could be used later if a warning is shown in the |
| - // developer console. |
| - String errorMessage; |
| - if (document()->isSecureContext(errorMessage)) { |
| + if (document()->ExecutionContext::isSecureContext()) { |
|
estark
2015/12/18 00:00:57
You should remove the "ExecutionContext::" qualifi
|
| UseCounter::count(document(), UseCounter::FullscreenSecureOrigin); |
| } else { |
| UseCounter::count(document(), UseCounter::FullscreenInsecureOrigin); |