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

Side by Side Diff: third_party/WebKit/Source/core/fetch/CSSStyleSheetResource.h

Issue 1667843003: Make Resource RefCountedWillBeGarbageCollectedFinalized, attempt #2 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase + address review comments Created 4 years, 10 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 Lars Knoll (knoll@mpi-hd.mpg.de) 2 Copyright (C) 1998 Lars Knoll (knoll@mpi-hd.mpg.de)
3 Copyright (C) 2001 Dirk Mueller <mueller@kde.org> 3 Copyright (C) 2001 Dirk Mueller <mueller@kde.org>
4 Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com) 4 Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com)
5 Copyright (C) 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved. 5 Copyright (C) 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
6 6
7 This library is free software; you can redistribute it and/or 7 This library is free software; you can redistribute it and/or
8 modify it under the terms of the GNU Library General Public 8 modify it under the terms of the GNU Library General Public
9 License as published by the Free Software Foundation; either 9 License as published by the Free Software Foundation; either
10 version 2 of the License, or (at your option) any later version. 10 version 2 of the License, or (at your option) any later version.
11 11
12 This library is distributed in the hope that it will be useful, 12 This library is distributed in the hope that it will be useful,
13 but WITHOUT ANY WARRANTY; without even the implied warranty of 13 but WITHOUT ANY WARRANTY; without even the implied warranty of
14 MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU 14 MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
15 Library General Public License for more details. 15 Library General Public License for more details.
16 16
17 You should have received a copy of the GNU Library General Public License 17 You should have received a copy of the GNU Library General Public License
18 along with this library; see the file COPYING.LIB. If not, write to 18 along with this library; see the file COPYING.LIB. If not, write to
19 the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, 19 the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
20 Boston, MA 02110-1301, USA. 20 Boston, MA 02110-1301, USA.
21 21
22 This class provides all functionality needed for loading images, style sheet s and html 22 This class provides all functionality needed for loading images, style sheet s and html
23 pages from the web. It has a memory cache for these objects. 23 pages from the web. It has a memory cache for these objects.
24 */ 24 */
25 25
26 #ifndef CSSStyleSheetResource_h 26 #ifndef CSSStyleSheetResource_h
27 #define CSSStyleSheetResource_h 27 #define CSSStyleSheetResource_h
28 28
29 #include "core/CoreExport.h" 29 #include "core/CoreExport.h"
30 #include "core/fetch/ResourcePtr.h"
31 #include "core/fetch/StyleSheetResource.h" 30 #include "core/fetch/StyleSheetResource.h"
32 #include "platform/heap/Handle.h" 31 #include "platform/heap/Handle.h"
33 32
34 namespace blink { 33 namespace blink {
35 34
36 class CSSParserContext; 35 class CSSParserContext;
37 class FetchRequest; 36 class FetchRequest;
38 class ResourceClient; 37 class ResourceClient;
39 class ResourceFetcher; 38 class ResourceFetcher;
40 class StyleSheetContents; 39 class StyleSheetContents;
41 40
42 class CORE_EXPORT CSSStyleSheetResource final : public StyleSheetResource { 41 class CORE_EXPORT CSSStyleSheetResource final : public StyleSheetResource {
43 public: 42 public:
44 enum class MIMETypeCheck { Strict, Lax }; 43 enum class MIMETypeCheck { Strict, Lax };
45 44
46 static ResourcePtr<CSSStyleSheetResource> fetch(FetchRequest&, ResourceFetch er*); 45 static PassRefPtrWillBeRawPtr<CSSStyleSheetResource> fetch(FetchRequest&, Re sourceFetcher*);
47 static ResourcePtr<CSSStyleSheetResource> createForTest(const ResourceReques t&, const String& charset); 46 static PassRefPtrWillBeRawPtr<CSSStyleSheetResource> createForTest(const Res ourceRequest&, const String& charset);
48 47
49 ~CSSStyleSheetResource() override; 48 ~CSSStyleSheetResource() override;
50 DECLARE_VIRTUAL_TRACE(); 49 DECLARE_VIRTUAL_TRACE();
51 50
52 const String sheetText(MIMETypeCheck = MIMETypeCheck::Strict) const; 51 const String sheetText(MIMETypeCheck = MIMETypeCheck::Strict) const;
53 52
54 const AtomicString mimeType() const; 53 const AtomicString mimeType() const;
55 54
56 void didAddClient(ResourceClient*) override; 55 void didAddClient(ResourceClient*) override;
57 56
58 PassRefPtrWillBeRawPtr<StyleSheetContents> restoreParsedStyleSheet(const CSS ParserContext&); 57 PassRefPtrWillBeRawPtr<StyleSheetContents> restoreParsedStyleSheet(const CSS ParserContext&);
59 void saveParsedStyleSheet(PassRefPtrWillBeRawPtr<StyleSheetContents>); 58 void saveParsedStyleSheet(PassRefPtrWillBeRawPtr<StyleSheetContents>);
60 59
61 protected: 60 protected:
62 bool isSafeToUnlock() const override; 61 bool isSafeToUnlock() const override;
63 void destroyDecodedDataIfPossible() override; 62 void destroyDecodedDataIfPossible() override;
64 void destroyDecodedDataForFailedRevalidation() override { destroyDecodedData IfPossible(); } 63 void destroyDecodedDataForFailedRevalidation() override { destroyDecodedData IfPossible(); }
65 64
66 private: 65 private:
67 class CSSStyleSheetResourceFactory : public ResourceFactory { 66 class CSSStyleSheetResourceFactory : public ResourceFactory {
68 public: 67 public:
69 CSSStyleSheetResourceFactory() 68 CSSStyleSheetResourceFactory()
70 : ResourceFactory(Resource::CSSStyleSheet) { } 69 : ResourceFactory(Resource::CSSStyleSheet) { }
71 70
72 Resource* create(const ResourceRequest& request, const String& charset) const override 71 PassRefPtrWillBeRawPtr<Resource> create(const ResourceRequest& request, const String& charset) const override
73 { 72 {
74 return new CSSStyleSheetResource(request, charset); 73 return adoptRefWillBeNoop(new CSSStyleSheetResource(request, charset ));
75 } 74 }
76 }; 75 };
77 CSSStyleSheetResource(const ResourceRequest&, const String& charset); 76 CSSStyleSheetResource(const ResourceRequest&, const String& charset);
78 77
79 bool canUseSheet(MIMETypeCheck) const; 78 bool canUseSheet(MIMETypeCheck) const;
80 void dispose() override; 79 void removedFromMemoryCache() override;
81 void checkNotify() override; 80 void checkNotify() override;
82 81
83 String m_decodedSheetText; 82 String m_decodedSheetText;
84 83
85 RefPtrWillBeMember<StyleSheetContents> m_parsedStyleSheetCache; 84 RefPtrWillBeMember<StyleSheetContents> m_parsedStyleSheetCache;
86 }; 85 };
87 86
88 DEFINE_RESOURCE_TYPE_CASTS(CSSStyleSheet); 87 DEFINE_RESOURCE_TYPE_CASTS(CSSStyleSheet);
89 88
90 } // namespace blink 89 } // namespace blink
91 90
92 #endif 91 #endif
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/dom/ScriptRunner.h ('k') | third_party/WebKit/Source/core/fetch/CSSStyleSheetResource.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698