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

Unified Diff: Source/core/dom/Document.cpp

Issue 13851023: Remove ChromeClient cruft (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Patch for landing Created 7 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/WebKit/chromium/src/WebPopupMenuImpl.cpp ('k') | Source/core/html/HTMLMediaElement.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/dom/Document.cpp
diff --git a/Source/core/dom/Document.cpp b/Source/core/dom/Document.cpp
index 4dc71f03aa8227e9d1caf39901b2d4eeb2da23f2..a1d4d92f299507049b8ce13db2dcac3a1ef5f620 100644
--- a/Source/core/dom/Document.cpp
+++ b/Source/core/dom/Document.cpp
@@ -4724,17 +4724,6 @@ void Document::requestFullScreenForElement(Element* element, unsigned short flag
if (!page() || !page()->settings()->fullScreenEnabled())
break;
- if (!page()->chrome()->client()->supportsFullScreenForElement(element, flags & Element::ALLOW_KEYBOARD_INPUT)) {
- // The new full screen API does not accept a "flags" parameter, so fall back to disallowing
- // keyboard input if the chrome client refuses to allow keyboard input.
- if (!inLegacyMozillaMode && flags & Element::ALLOW_KEYBOARD_INPUT) {
- flags &= ~Element::ALLOW_KEYBOARD_INPUT;
- if (!page()->chrome()->client()->supportsFullScreenForElement(element, false))
- break;
- } else
- break;
- }
-
// 2. Let doc be element's node document. (i.e. "this")
Document* currentDoc = this;
@@ -5002,18 +4991,11 @@ void Document::setFullScreenRenderer(RenderFullScreen* renderer)
ASSERT(!m_fullScreenRenderer);
m_fullScreenRenderer = renderer;
-
- // This notification can come in after the page has been destroyed.
- if (page())
- page()->chrome()->client()->fullScreenRendererChanged(m_fullScreenRenderer);
}
void Document::fullScreenRendererDestroyed()
{
m_fullScreenRenderer = 0;
-
- if (page())
- page()->chrome()->client()->fullScreenRendererChanged(0);
}
void Document::setFullScreenRendererSize(const IntSize& size)
@@ -5699,7 +5681,7 @@ PassRefPtr<FontLoader> Document::fontloader()
void Document::didAssociateFormControl(Element* element)
{
- if (!frame() || !frame()->page() || !frame()->page()->chrome()->client()->shouldNotifyOnFormChanges())
+ if (!frame() || !frame()->page())
return;
m_associatedFormControls.add(element);
if (!m_didAssociateFormControlsTimer.isActive())
« no previous file with comments | « Source/WebKit/chromium/src/WebPopupMenuImpl.cpp ('k') | Source/core/html/HTMLMediaElement.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698