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

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

Issue 2389643002: Reflow comments in core/fetch (Closed)
Patch Set: yoavs comments Created 4 years, 2 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 Apple Inc. All rights reserved. 5 Copyright (C) 2004, 2005, 2006, 2007 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 19 matching lines...) Expand all
30 30
31 namespace blink { 31 namespace blink {
32 32
33 class ImageResource; 33 class ImageResource;
34 class IntRect; 34 class IntRect;
35 35
36 class CORE_EXPORT ImageResourceObserver { 36 class CORE_EXPORT ImageResourceObserver {
37 public: 37 public:
38 virtual ~ImageResourceObserver() {} 38 virtual ~ImageResourceObserver() {}
39 39
40 // Called whenever a frame of an image changes, either because we got more dat a from the network or 40 // Called whenever a frame of an image changes, either because we got more
41 // because we are animating. If not null, the IntRect is the changed rect of t he image. 41 // data from the network or because we are animating. If not null, the IntRect
42 // is the changed rect of the image.
42 virtual void imageChanged(ImageResource*, const IntRect* = 0) {} 43 virtual void imageChanged(ImageResource*, const IntRect* = 0) {}
43 44
44 // Called just before ResourceClient::notifyFinished() would be called. 45 // Called just before ResourceClient::notifyFinished() would be called. This
45 // This is to avoid an ImageResourceObserver from being 46 // is to avoid an ImageResourceObserver from being also a ResourceClient just
46 // also a ResourceClient just to be notified for load finish. 47 // to be notified for load finish.
47 virtual void imageNotifyFinished(ImageResource*) {} 48 virtual void imageNotifyFinished(ImageResource*) {}
48 49
49 // Called to find out if this client wants to actually display the image. Used to tell when we 50 // Called to find out if this client wants to actually display the image. Used
50 // can halt animation. Content nodes that hold image refs for example would no t render the image, 51 // to tell when we can halt animation. Content nodes that hold image refs for
51 // but LayoutImages would (assuming they have visibility: visible and their la yout tree isn't hidden 52 // example would not render the image, but LayoutImages would (assuming they
52 // e.g., in the b/f cache or in a background tab). 53 // have visibility: visible and their layout tree isn't hidden e.g., in the
54 // b/f cache or in a background tab).
53 virtual bool willRenderImage() { return false; } 55 virtual bool willRenderImage() { return false; }
54 56
55 // Called to get imageAnimation policy from settings 57 // Called to get imageAnimation policy from settings
56 virtual bool getImageAnimationPolicy(ImageAnimationPolicy&) { return false; } 58 virtual bool getImageAnimationPolicy(ImageAnimationPolicy&) { return false; }
57 59
58 virtual ResourcePriority computeResourcePriority() const { 60 virtual ResourcePriority computeResourcePriority() const {
59 return ResourcePriority(); 61 return ResourcePriority();
60 } 62 }
61 63
62 // Name for debugging, e.g. shown in memory-infra. 64 // Name for debugging, e.g. shown in memory-infra.
63 virtual String debugName() const = 0; 65 virtual String debugName() const = 0;
64 66
65 static bool isExpectedType(ImageResourceObserver*) { return true; } 67 static bool isExpectedType(ImageResourceObserver*) { return true; }
66 }; 68 };
67 69
68 } // namespace blink 70 } // namespace blink
69 71
70 #endif 72 #endif
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/fetch/ImageResource.cpp ('k') | third_party/WebKit/Source/core/fetch/ImageResourceTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698