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

Unified Diff: sky/engine/core/frame/LocalDOMWindow.cpp

Issue 872143004: Remove more properties from the global Window. (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 11 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 | « sky/engine/core/frame/LocalDOMWindow.h ('k') | sky/engine/core/frame/Window.idl » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sky/engine/core/frame/LocalDOMWindow.cpp
diff --git a/sky/engine/core/frame/LocalDOMWindow.cpp b/sky/engine/core/frame/LocalDOMWindow.cpp
index a8b4461b7d3849c80a63f57a681b4b6f1eb14b1c..a19e57c13b086b3043de6c5382968746ee5aea0b 100644
--- a/sky/engine/core/frame/LocalDOMWindow.cpp
+++ b/sky/engine/core/frame/LocalDOMWindow.cpp
@@ -41,7 +41,6 @@
#include "sky/engine/core/css/DOMWindowCSS.h"
#include "sky/engine/core/css/MediaQueryList.h"
#include "sky/engine/core/css/MediaQueryMatcher.h"
-#include "sky/engine/core/css/StyleMedia.h"
#include "sky/engine/core/css/resolver/StyleResolver.h"
#include "sky/engine/core/dom/Document.h"
#include "sky/engine/core/dom/Element.h"
@@ -382,7 +381,6 @@ void LocalDOMWindow::resetDOMWindowProperties()
m_screen = nullptr;
m_console = nullptr;
m_location = nullptr;
- m_media = nullptr;
#if ENABLE(ASSERT)
m_hasBeenReset = true;
#endif
@@ -449,16 +447,6 @@ void LocalDOMWindow::focus(ExecutionContext* context)
m_frame->eventHandler().focusDocumentView();
}
-bool LocalDOMWindow::find(const String& string, bool caseSensitive, bool backwards, bool wrap, bool /*wholeWord*/, bool /*searchInFrames*/, bool /*showDialog*/) const
-{
- // |m_frame| can be destructed during |Editor::findString()| via
- // |Document::updateLayou()|, e.g. event handler removes a frame.
- RefPtr<LocalFrame> protectFrame(m_frame);
-
- // FIXME (13016): Support wholeWord, searchInFrames and showDialog
- return m_frame->editor().findString(string, !backwards, caseSensitive, wrap, false);
-}
-
int LocalDOMWindow::outerHeight() const
{
if (!m_frame)
@@ -544,13 +532,6 @@ Document* LocalDOMWindow::document() const
return m_document.get();
}
-StyleMedia& LocalDOMWindow::styleMedia() const
-{
- if (!m_media)
- m_media = StyleMedia::create(m_frame);
- return *m_media;
-}
-
PassRefPtr<CSSStyleDeclaration> LocalDOMWindow::getComputedStyle(Element* elt) const
{
if (!elt)
« no previous file with comments | « sky/engine/core/frame/LocalDOMWindow.h ('k') | sky/engine/core/frame/Window.idl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698