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

Unified Diff: Source/modules/serviceworkers/ServiceWorkerWindowClient.cpp

Issue 1040993003: ServiceWorker: Support non-window clients in Clients.matchAll (2/2 blink) (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 9 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
Index: Source/modules/serviceworkers/ServiceWorkerWindowClient.cpp
diff --git a/Source/modules/serviceworkers/ServiceWorkerWindowClient.cpp b/Source/modules/serviceworkers/ServiceWorkerWindowClient.cpp
index bc0ee55a6bdc37b886b990532850ae3ba1c57edf..c0963b7f214b71cef581659ef386bd94a3dec2a4 100644
--- a/Source/modules/serviceworkers/ServiceWorkerWindowClient.cpp
+++ b/Source/modules/serviceworkers/ServiceWorkerWindowClient.cpp
@@ -36,7 +36,6 @@ ServiceWorkerWindowClient::ServiceWorkerWindowClient(const WebServiceWorkerClien
: ServiceWorkerClient(info)
, m_pageVisibilityState(info.pageVisibilityState)
, m_isFocused(info.isFocused)
- , m_frameType(info.frameType)
{
}
@@ -49,23 +48,6 @@ String ServiceWorkerWindowClient::visibilityState() const
return pageVisibilityStateString(static_cast<PageVisibilityState>(m_pageVisibilityState));
}
-String ServiceWorkerWindowClient::frameType() const
-{
- switch (m_frameType) {
- case WebURLRequest::FrameTypeAuxiliary:
- return "auxiliary";
- case WebURLRequest::FrameTypeNested:
- return "nested";
- case WebURLRequest::FrameTypeNone:
- return "none";
- case WebURLRequest::FrameTypeTopLevel:
- return "top-level";
- }
-
- ASSERT_NOT_REACHED();
- return String();
-}
-
ScriptPromise ServiceWorkerWindowClient::focus(ScriptState* scriptState)
{
RefPtrWillBeRawPtr<ScriptPromiseResolver> resolver = ScriptPromiseResolver::create(scriptState);

Powered by Google App Engine
This is Rietveld 408576698