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

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

Issue 1773503002: Shortcut ImageResource::canRender() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 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 53 matching lines...) Expand 10 before | Expand all | Expand 10 after
64 ~ImageResource() override; 64 ~ImageResource() override;
65 65
66 void load(ResourceFetcher*, const ResourceLoaderOptions&) override; 66 void load(ResourceFetcher*, const ResourceLoaderOptions&) override;
67 67
68 blink::Image* image(); // Returns the nullImage() if the image is not availa ble yet. 68 blink::Image* image(); // Returns the nullImage() if the image is not availa ble yet.
69 bool hasImage() const { return m_image.get(); } 69 bool hasImage() const { return m_image.get(); }
70 70
71 static std::pair<blink::Image*, float> brokenImage(float deviceScaleFactor); // Returns an image and the image's resolution scale factor. 71 static std::pair<blink::Image*, float> brokenImage(float deviceScaleFactor); // Returns an image and the image's resolution scale factor.
72 bool willPaintBrokenImage() const; 72 bool willPaintBrokenImage() const;
73 73
74 // Assumes that image rotation or scale doesn't effect the image size being empty or not.
75 bool canRender() { return !errorOccurred() && !imageSize(DoNotRespectImageOr ientation, 1).isEmpty(); }
76
77 bool usesImageContainerSize() const; 74 bool usesImageContainerSize() const;
78 bool imageHasRelativeSize() const; 75 bool imageHasRelativeSize() const;
79 // The device pixel ratio we got from the server for this image, or 1.0. 76 // The device pixel ratio we got from the server for this image, or 1.0.
80 float devicePixelRatioHeaderValue() const { return m_devicePixelRatioHeaderV alue; } 77 float devicePixelRatioHeaderValue() const { return m_devicePixelRatioHeaderV alue; }
81 bool hasDevicePixelRatioHeaderValue() const { return m_hasDevicePixelRatioHe aderValue; } 78 bool hasDevicePixelRatioHeaderValue() const { return m_hasDevicePixelRatioHe aderValue; }
82 79
83 enum SizeType { 80 enum SizeType {
84 IntrinsicSize, // Report the intrinsic size. 81 IntrinsicSize, // Report the intrinsic size.
85 IntrinsicCorrectedToDPR, // Report the intrinsic size corrected to accou nt for image density. 82 IntrinsicCorrectedToDPR, // Report the intrinsic size corrected to accou nt for image density.
86 }; 83 };
(...skipping 70 matching lines...) Expand 10 before | Expand all | Expand 10 after
157 154
158 RefPtr<blink::Image> m_image; 155 RefPtr<blink::Image> m_image;
159 bool m_hasDevicePixelRatioHeaderValue; 156 bool m_hasDevicePixelRatioHeaderValue;
160 }; 157 };
161 158
162 DEFINE_RESOURCE_TYPE_CASTS(Image); 159 DEFINE_RESOURCE_TYPE_CASTS(Image);
163 160
164 } // namespace blink 161 } // namespace blink
165 162
166 #endif 163 #endif
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/css/CSSCrossfadeValue.cpp ('k') | third_party/WebKit/Source/core/style/StyleFetchedImage.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698