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

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

Issue 1843533005: Call imageNotifyFinished() just before ResourceClient::notifyFinished() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase, Reflect yhirano's comment Created 4 years, 8 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, 2009, 2010, 2011 Apple Inc. All rights reserved. 5 Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 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.
(...skipping 273 matching lines...) Expand 10 before | Expand all | Expand 10 after
284 } 284 }
285 285
286 ResourceRequest m_request; 286 ResourceRequest m_request;
287 ResourceResponse m_redirectResponse; 287 ResourceResponse m_redirectResponse;
288 }; 288 };
289 const Vector<RedirectPair>& redirectChain() const { return m_redirectChain; } 289 const Vector<RedirectPair>& redirectChain() const { return m_redirectChain; }
290 290
291 virtual bool isSafeToUnlock() const { return false; } 291 virtual bool isSafeToUnlock() const { return false; }
292 virtual void destroyDecodedDataIfPossible() { } 292 virtual void destroyDecodedDataIfPossible() { }
293 293
294 void markClientsFinished(); 294 virtual void markClientsAndObserversFinished();
295 295
296 // Returns the memory dump name used for tracing. See Resource::onMemoryDump . 296 // Returns the memory dump name used for tracing. See Resource::onMemoryDump .
297 String getMemoryDumpName() const; 297 String getMemoryDumpName() const;
298 298
299 ResourceRequest m_resourceRequest; 299 ResourceRequest m_resourceRequest;
300 ResourceRequest m_revalidatingRequest; 300 ResourceRequest m_revalidatingRequest;
301 AtomicString m_accept; 301 AtomicString m_accept;
302 Member<ResourceLoader> m_loader; 302 Member<ResourceLoader> m_loader;
303 ResourceLoaderOptions m_options; 303 ResourceLoaderOptions m_options;
304 304
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
376 Resource::Type m_type; 376 Resource::Type m_type;
377 }; 377 };
378 378
379 #define DEFINE_RESOURCE_TYPE_CASTS(typeName) \ 379 #define DEFINE_RESOURCE_TYPE_CASTS(typeName) \
380 DEFINE_TYPE_CASTS(typeName##Resource, Resource, resource, resource->getType( ) == Resource::typeName, resource.getType() == Resource::typeName); \ 380 DEFINE_TYPE_CASTS(typeName##Resource, Resource, resource, resource->getType( ) == Resource::typeName, resource.getType() == Resource::typeName); \
381 inline typeName##Resource* to##typeName##Resource(const RawPtr<Resource>& pt r) { return to##typeName##Resource(ptr.get()); } 381 inline typeName##Resource* to##typeName##Resource(const RawPtr<Resource>& pt r) { return to##typeName##Resource(ptr.get()); }
382 382
383 } // namespace blink 383 } // namespace blink
384 384
385 #endif 385 #endif
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/fetch/ImageResourceObserver.h ('k') | third_party/WebKit/Source/core/fetch/Resource.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698