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

Side by Side Diff: third_party/WebKit/Source/core/html/parser/CSSPreloadScanner.cpp

Issue 2544583003: Ensure CSS scanner does not mark preloads as referenced (Closed)
Patch Set: Created 4 years 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
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/html/parser/CSSPreloadScannerTest.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2008, 2010 Apple Inc. All Rights Reserved. 2 * Copyright (C) 2008, 2010 Apple Inc. All Rights Reserved.
3 * Copyright (C) 2009 Torch Mobile, Inc. http://www.torchmobile.com/ 3 * Copyright (C) 2009 Torch Mobile, Inc. http://www.torchmobile.com/
4 * Copyright (C) 2010 Google Inc. All Rights Reserved. 4 * Copyright (C) 2010 Google Inc. All Rights Reserved.
5 * 5 *
6 * Redistribution and use in source and binary forms, with or without 6 * Redistribution and use in source and binary forms, with or without
7 * modification, are permitted provided that the following conditions 7 * modification, are permitted provided that the following conditions
8 * are met: 8 * are met:
9 * 1. Redistributions of source code must retain the above copyright 9 * 1. Redistributions of source code must retain the above copyright
10 * notice, this list of conditions and the following disclaimer. 10 * notice, this list of conditions and the following disclaimer.
(...skipping 239 matching lines...) Expand 10 before | Expand all | Expand 10 after
250 } 250 }
251 251
252 CSSPreloaderResourceClient::CSSPreloaderResourceClient( 252 CSSPreloaderResourceClient::CSSPreloaderResourceClient(
253 Resource* resource, 253 Resource* resource,
254 HTMLResourcePreloader* preloader) 254 HTMLResourcePreloader* preloader)
255 : m_policy(preloader->document()->settings()->cssExternalScannerPreload() 255 : m_policy(preloader->document()->settings()->cssExternalScannerPreload()
256 ? ScanAndPreload 256 ? ScanAndPreload
257 : ScanOnly), 257 : ScanOnly),
258 m_preloader(preloader), 258 m_preloader(preloader),
259 m_resource(toCSSStyleSheetResource(resource)) { 259 m_resource(toCSSStyleSheetResource(resource)) {
260 m_resource->addClient(this); 260 m_resource->addClient(this, Resource::DontMarkAsReferenced);
261 } 261 }
262 262
263 CSSPreloaderResourceClient::~CSSPreloaderResourceClient() {} 263 CSSPreloaderResourceClient::~CSSPreloaderResourceClient() {}
264 264
265 void CSSPreloaderResourceClient::setCSSStyleSheet( 265 void CSSPreloaderResourceClient::setCSSStyleSheet(
266 const String& href, 266 const String& href,
267 const KURL& baseURL, 267 const KURL& baseURL,
268 const String& charset, 268 const String& charset,
269 const CSSStyleSheetResource*) { 269 const CSSStyleSheetResource*) {
270 clearResource(); 270 clearResource();
(...skipping 54 matching lines...) Expand 10 before | Expand all | Expand 10 after
325 m_resource.clear(); 325 m_resource.clear();
326 } 326 }
327 327
328 DEFINE_TRACE(CSSPreloaderResourceClient) { 328 DEFINE_TRACE(CSSPreloaderResourceClient) {
329 visitor->trace(m_preloader); 329 visitor->trace(m_preloader);
330 visitor->trace(m_resource); 330 visitor->trace(m_resource);
331 StyleSheetResourceClient::trace(visitor); 331 StyleSheetResourceClient::trace(visitor);
332 } 332 }
333 333
334 } // namespace blink 334 } // namespace blink
OLDNEW
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/html/parser/CSSPreloadScannerTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698