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

Side by Side Diff: third_party/WebKit/Source/core/frame/Frame.h

Issue 2009453002: service worker: Don't control a subframe of an insecure context (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: cover plznavigation case too Created 4 years, 6 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org> 2 * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
3 * 1999-2001 Lars Knoll <knoll@kde.org> 3 * 1999-2001 Lars Knoll <knoll@kde.org>
4 * 1999-2001 Antti Koivisto <koivisto@kde.org> 4 * 1999-2001 Antti Koivisto <koivisto@kde.org>
5 * 2000-2001 Simon Hausmann <hausmann@kde.org> 5 * 2000-2001 Simon Hausmann <hausmann@kde.org>
6 * 2000-2001 Dirk Mueller <mueller@kde.org> 6 * 2000-2001 Dirk Mueller <mueller@kde.org>
7 * 2000 Stefan Schimanski <1Stein@gmx.de> 7 * 2000 Stefan Schimanski <1Stein@gmx.de>
8 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved. 8 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved.
9 * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies) 9 * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
10 * Copyright (C) 2008 Eric Seidel <eric@webkit.org> 10 * Copyright (C) 2008 Eric Seidel <eric@webkit.org>
(...skipping 118 matching lines...) Expand 10 before | Expand all | Expand 10 after
129 // In the case of LocalFrames, it means that the frame has sent a didStartLo ading() 129 // In the case of LocalFrames, it means that the frame has sent a didStartLo ading()
130 // callback, but not the matching didStopLoading(). Inside blink, you probab ly 130 // callback, but not the matching didStopLoading(). Inside blink, you probab ly
131 // want Document::loadEventFinished() instead. 131 // want Document::loadEventFinished() instead.
132 void setIsLoading(bool isLoading) { m_isLoading = isLoading; } 132 void setIsLoading(bool isLoading) { m_isLoading = isLoading; }
133 bool isLoading() const { return m_isLoading; } 133 bool isLoading() const { return m_isLoading; }
134 134
135 virtual WindowProxyManager* getWindowProxyManager() const = 0; 135 virtual WindowProxyManager* getWindowProxyManager() const = 0;
136 136
137 virtual void didChangeVisibilityState(); 137 virtual void didChangeVisibilityState();
138 138
139 // Returns whether it's possible for a descendent of this frame to
140 // be a secure context.
141 bool canHaveSecureChild(String* errorMessage) const;
142
139 protected: 143 protected:
140 Frame(FrameClient*, FrameHost*, FrameOwner*); 144 Frame(FrameClient*, FrameHost*, FrameOwner*);
141 145
142 mutable FrameTree m_treeNode; 146 mutable FrameTree m_treeNode;
143 147
144 Member<FrameHost> m_host; 148 Member<FrameHost> m_host;
145 Member<FrameOwner> m_owner; 149 Member<FrameOwner> m_owner;
146 150
147 private: 151 private:
148 bool canNavigateWithoutFramebusting(const Frame&, String& errorReason); 152 bool canNavigateWithoutFramebusting(const Frame&, String& errorReason);
(...skipping 16 matching lines...) Expand all
165 { 169 {
166 return m_treeNode; 170 return m_treeNode;
167 } 171 }
168 172
169 // Allow equality comparisons of Frames by reference or pointer, interchangeably . 173 // Allow equality comparisons of Frames by reference or pointer, interchangeably .
170 DEFINE_COMPARISON_OPERATORS_WITH_REFERENCES_REFCOUNTED(Frame) 174 DEFINE_COMPARISON_OPERATORS_WITH_REFERENCES_REFCOUNTED(Frame)
171 175
172 } // namespace blink 176 } // namespace blink
173 177
174 #endif // Frame_h 178 #endif // Frame_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698