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

Side by Side Diff: third_party/WebKit/Source/core/loader/LinkPreloadResourceClients.h

Issue 2343623002: Add a warning whenever link preloads are not used (Closed)
Patch Set: more comments Created 4 years, 3 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 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef LinkPreloadResourceClients_h 5 #ifndef LinkPreloadResourceClients_h
6 #define LinkPreloadResourceClients_h 6 #define LinkPreloadResourceClients_h
7 7
8 #include "core/fetch/CSSStyleSheetResource.h" 8 #include "core/fetch/CSSStyleSheetResource.h"
9 #include "core/fetch/FontResource.h" 9 #include "core/fetch/FontResource.h"
10 #include "core/fetch/ImageResource.h" 10 #include "core/fetch/ImageResource.h"
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
63 DEFINE_INLINE_VIRTUAL_TRACE() 63 DEFINE_INLINE_VIRTUAL_TRACE()
64 { 64 {
65 LinkPreloadResourceClient::trace(visitor); 65 LinkPreloadResourceClient::trace(visitor);
66 ResourceOwner<ScriptResource, ScriptResourceClient>::trace(visitor); 66 ResourceOwner<ScriptResource, ScriptResourceClient>::trace(visitor);
67 } 67 }
68 68
69 private: 69 private:
70 LinkPreloadScriptResourceClient(LinkLoader* loader, ScriptResource* resource ) 70 LinkPreloadScriptResourceClient(LinkLoader* loader, ScriptResource* resource )
71 : LinkPreloadResourceClient(loader) 71 : LinkPreloadResourceClient(loader)
72 { 72 {
73 setResource(resource); 73 setResource(resource, Resource::DontMarkAsReferenced);
74 } 74 }
75 }; 75 };
76 76
77 class LinkPreloadStyleResourceClient: public LinkPreloadResourceClient, public R esourceOwner<CSSStyleSheetResource, StyleSheetResourceClient> { 77 class LinkPreloadStyleResourceClient: public LinkPreloadResourceClient, public R esourceOwner<CSSStyleSheetResource, StyleSheetResourceClient> {
78 USING_GARBAGE_COLLECTED_MIXIN(LinkPreloadStyleResourceClient); 78 USING_GARBAGE_COLLECTED_MIXIN(LinkPreloadStyleResourceClient);
79 public: 79 public:
80 static LinkPreloadStyleResourceClient* create(LinkLoader* loader, CSSStyleSh eetResource* resource) 80 static LinkPreloadStyleResourceClient* create(LinkLoader* loader, CSSStyleSh eetResource* resource)
81 { 81 {
82 return new LinkPreloadStyleResourceClient(loader, resource); 82 return new LinkPreloadStyleResourceClient(loader, resource);
83 } 83 }
(...skipping 12 matching lines...) Expand all
96 DEFINE_INLINE_VIRTUAL_TRACE() 96 DEFINE_INLINE_VIRTUAL_TRACE()
97 { 97 {
98 LinkPreloadResourceClient::trace(visitor); 98 LinkPreloadResourceClient::trace(visitor);
99 ResourceOwner<CSSStyleSheetResource, StyleSheetResourceClient>::trace(vi sitor); 99 ResourceOwner<CSSStyleSheetResource, StyleSheetResourceClient>::trace(vi sitor);
100 } 100 }
101 101
102 private: 102 private:
103 LinkPreloadStyleResourceClient(LinkLoader* loader, CSSStyleSheetResource* re source) 103 LinkPreloadStyleResourceClient(LinkLoader* loader, CSSStyleSheetResource* re source)
104 : LinkPreloadResourceClient(loader) 104 : LinkPreloadResourceClient(loader)
105 { 105 {
106 setResource(resource); 106 setResource(resource, Resource::DontMarkAsReferenced);
107 } 107 }
108 }; 108 };
109 109
110 class LinkPreloadImageResourceClient: public LinkPreloadResourceClient, public R esourceOwner<ImageResource> { 110 class LinkPreloadImageResourceClient: public LinkPreloadResourceClient, public R esourceOwner<ImageResource> {
111 USING_GARBAGE_COLLECTED_MIXIN(LinkPreloadImageResourceClient); 111 USING_GARBAGE_COLLECTED_MIXIN(LinkPreloadImageResourceClient);
112 public: 112 public:
113 static LinkPreloadImageResourceClient* create(LinkLoader* loader, ImageResou rce* resource) 113 static LinkPreloadImageResourceClient* create(LinkLoader* loader, ImageResou rce* resource)
114 { 114 {
115 return new LinkPreloadImageResourceClient(loader, resource); 115 return new LinkPreloadImageResourceClient(loader, resource);
116 } 116 }
(...skipping 12 matching lines...) Expand all
129 DEFINE_INLINE_VIRTUAL_TRACE() 129 DEFINE_INLINE_VIRTUAL_TRACE()
130 { 130 {
131 LinkPreloadResourceClient::trace(visitor); 131 LinkPreloadResourceClient::trace(visitor);
132 ResourceOwner<ImageResource>::trace(visitor); 132 ResourceOwner<ImageResource>::trace(visitor);
133 } 133 }
134 134
135 private: 135 private:
136 LinkPreloadImageResourceClient(LinkLoader* loader, ImageResource* resource) 136 LinkPreloadImageResourceClient(LinkLoader* loader, ImageResource* resource)
137 : LinkPreloadResourceClient(loader) 137 : LinkPreloadResourceClient(loader)
138 { 138 {
139 setResource(resource); 139 setResource(resource, Resource::DontMarkAsReferenced);
140 } 140 }
141 }; 141 };
142 142
143 class LinkPreloadFontResourceClient: public LinkPreloadResourceClient, public Re sourceOwner<FontResource, FontResourceClient> { 143 class LinkPreloadFontResourceClient: public LinkPreloadResourceClient, public Re sourceOwner<FontResource, FontResourceClient> {
144 USING_GARBAGE_COLLECTED_MIXIN(LinkPreloadFontResourceClient); 144 USING_GARBAGE_COLLECTED_MIXIN(LinkPreloadFontResourceClient);
145 public: 145 public:
146 static LinkPreloadFontResourceClient* create(LinkLoader* loader, FontResourc e* resource) 146 static LinkPreloadFontResourceClient* create(LinkLoader* loader, FontResourc e* resource)
147 { 147 {
148 return new LinkPreloadFontResourceClient(loader, resource); 148 return new LinkPreloadFontResourceClient(loader, resource);
149 } 149 }
(...skipping 12 matching lines...) Expand all
162 DEFINE_INLINE_VIRTUAL_TRACE() 162 DEFINE_INLINE_VIRTUAL_TRACE()
163 { 163 {
164 LinkPreloadResourceClient::trace(visitor); 164 LinkPreloadResourceClient::trace(visitor);
165 ResourceOwner<FontResource, FontResourceClient>::trace(visitor); 165 ResourceOwner<FontResource, FontResourceClient>::trace(visitor);
166 } 166 }
167 167
168 private: 168 private:
169 LinkPreloadFontResourceClient(LinkLoader* loader, FontResource* resource) 169 LinkPreloadFontResourceClient(LinkLoader* loader, FontResource* resource)
170 : LinkPreloadResourceClient(loader) 170 : LinkPreloadResourceClient(loader)
171 { 171 {
172 setResource(resource); 172 setResource(resource, Resource::DontMarkAsReferenced);
173 } 173 }
174 }; 174 };
175 175
176 class LinkPreloadRawResourceClient: public LinkPreloadResourceClient, public Res ourceOwner<RawResource, RawResourceClient> { 176 class LinkPreloadRawResourceClient: public LinkPreloadResourceClient, public Res ourceOwner<RawResource, RawResourceClient> {
177 USING_GARBAGE_COLLECTED_MIXIN(LinkPreloadRawResourceClient); 177 USING_GARBAGE_COLLECTED_MIXIN(LinkPreloadRawResourceClient);
178 public: 178 public:
179 static LinkPreloadRawResourceClient* create(LinkLoader* loader, RawResource* resource) 179 static LinkPreloadRawResourceClient* create(LinkLoader* loader, RawResource* resource)
180 { 180 {
181 return new LinkPreloadRawResourceClient(loader, resource); 181 return new LinkPreloadRawResourceClient(loader, resource);
182 } 182 }
(...skipping 12 matching lines...) Expand all
195 DEFINE_INLINE_VIRTUAL_TRACE() 195 DEFINE_INLINE_VIRTUAL_TRACE()
196 { 196 {
197 LinkPreloadResourceClient::trace(visitor); 197 LinkPreloadResourceClient::trace(visitor);
198 ResourceOwner<RawResource, RawResourceClient>::trace(visitor); 198 ResourceOwner<RawResource, RawResourceClient>::trace(visitor);
199 } 199 }
200 200
201 private: 201 private:
202 LinkPreloadRawResourceClient(LinkLoader* loader, RawResource* resource) 202 LinkPreloadRawResourceClient(LinkLoader* loader, RawResource* resource)
203 : LinkPreloadResourceClient(loader) 203 : LinkPreloadResourceClient(loader)
204 { 204 {
205 setResource(resource); 205 setResource(resource, Resource::DontMarkAsReferenced);
206 } 206 }
207 }; 207 };
208 208
209 } // namespace blink 209 } // namespace blink
210 210
211 #endif // LinkPreloadResourceClients_h 211 #endif // LinkPreloadResourceClients_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698