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

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

Issue 2642823005: Phase II Step 1: Remove updateImage() reentrancy around decodeError() (Closed)
Patch Set: Rebase Created 3 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 // 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 ImageResourceInfo_h 5 #ifndef ImageResourceInfo_h
6 #define ImageResourceInfo_h 6 #define ImageResourceInfo_h
7 7
8 #include "core/CoreExport.h" 8 #include "core/CoreExport.h"
9 #include "platform/heap/Handle.h" 9 #include "platform/heap/Handle.h"
10 #include "platform/heap/Heap.h" 10 #include "platform/heap/Heap.h"
(...skipping 28 matching lines...) Expand all
39 enum DoesCurrentFrameHaveSingleSecurityOrigin { 39 enum DoesCurrentFrameHaveSingleSecurityOrigin {
40 HasMultipleSecurityOrigin, 40 HasMultipleSecurityOrigin,
41 HasSingleSecurityOrigin 41 HasSingleSecurityOrigin
42 }; 42 };
43 virtual bool isAccessAllowed( 43 virtual bool isAccessAllowed(
44 SecurityOrigin*, 44 SecurityOrigin*,
45 DoesCurrentFrameHaveSingleSecurityOrigin) const = 0; 45 DoesCurrentFrameHaveSingleSecurityOrigin) const = 0;
46 virtual bool hasCacheControlNoStoreHeader() const = 0; 46 virtual bool hasCacheControlNoStoreHeader() const = 0;
47 virtual const ResourceError& resourceError() const = 0; 47 virtual const ResourceError& resourceError() const = 0;
48 48
49 // Like Resource::error(), decodeError() makes corresponding ImageResource
50 // (if any) DecodeError and finishes loading.
51 virtual void decodeError(bool allDataReceived) = 0;
52
53 // TODO(hiroshige): Remove this. 49 // TODO(hiroshige): Remove this.
54 virtual void setIsPlaceholder(bool) = 0; 50 virtual void setIsPlaceholder(bool) = 0;
55 51
56 // TODO(hiroshige): Remove this once MemoryCache becomes further weaker. 52 // TODO(hiroshige): Remove this once MemoryCache becomes further weaker.
57 virtual void setDecodedSize(size_t) = 0; 53 virtual void setDecodedSize(size_t) = 0;
58 54
59 // TODO(hiroshige): Remove these. 55 // TODO(hiroshige): Remove these.
60 virtual void willAddClientOrObserver() = 0; 56 virtual void willAddClientOrObserver() = 0;
61 virtual void didRemoveClientOrObserver() = 0; 57 virtual void didRemoveClientOrObserver() = 0;
62 58
63 // TODO(hiroshige): Remove this. crbug.com/666214 59 // TODO(hiroshige): Remove this. crbug.com/666214
64 virtual void emulateLoadStartedForInspector( 60 virtual void emulateLoadStartedForInspector(
65 ResourceFetcher*, 61 ResourceFetcher*,
66 const KURL&, 62 const KURL&,
67 const AtomicString& initiatorName) = 0; 63 const AtomicString& initiatorName) = 0;
68 64
69 DEFINE_INLINE_VIRTUAL_TRACE() {} 65 DEFINE_INLINE_VIRTUAL_TRACE() {}
70 }; 66 };
71 67
72 } // namespace blink 68 } // namespace blink
73 69
74 #endif 70 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698