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

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

Issue 1571233003: Fix errors caused by unsafe conversions to/from size_t (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: improved ALLOW_NUMERIC_ARG_TYPES_PROMOTABLE_TO Created 4 years, 11 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 72 matching lines...) Expand 10 before | Expand all | Expand 10 after
83 83
84 bool isAccessAllowed(SecurityOrigin*); 84 bool isAccessAllowed(SecurityOrigin*);
85 85
86 void updateImageAnimationPolicy(); 86 void updateImageAnimationPolicy();
87 87
88 void didAddClient(ResourceClient*) override; 88 void didAddClient(ResourceClient*) override;
89 void didRemoveClient(ResourceClient*) override; 89 void didRemoveClient(ResourceClient*) override;
90 90
91 void allClientsRemoved() override; 91 void allClientsRemoved() override;
92 92
93 void appendData(const char*, unsigned) override; 93 void appendData(const char*, size_t) override;
94 void error(Resource::Status) override; 94 void error(Resource::Status) override;
95 void responseReceived(const ResourceResponse&, PassOwnPtr<WebDataConsumerHan dle>) override; 95 void responseReceived(const ResourceResponse&, PassOwnPtr<WebDataConsumerHan dle>) override;
96 void finishOnePart() override; 96 void finishOnePart() override;
97 97
98 // For compatibility, images keep loading even if there are HTTP errors. 98 // For compatibility, images keep loading even if there are HTTP errors.
99 bool shouldIgnoreHTTPStatusCodeErrors() const override { return true; } 99 bool shouldIgnoreHTTPStatusCodeErrors() const override { return true; }
100 100
101 bool isImage() const override { return true; } 101 bool isImage() const override { return true; }
102 bool stillNeedsLoad() const override { return !errorOccurred() && status() = = Unknown && !isLoading(); } 102 bool stillNeedsLoad() const override { return !errorOccurred() && status() = = Unknown && !isLoading(); }
103 103
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
141 141
142 RefPtr<blink::Image> m_image; 142 RefPtr<blink::Image> m_image;
143 bool m_hasDevicePixelRatioHeaderValue; 143 bool m_hasDevicePixelRatioHeaderValue;
144 }; 144 };
145 145
146 DEFINE_RESOURCE_TYPE_CASTS(Image); 146 DEFINE_RESOURCE_TYPE_CASTS(Image);
147 147
148 } 148 }
149 149
150 #endif 150 #endif
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/css/FontFace.cpp ('k') | third_party/WebKit/Source/core/fetch/ImageResource.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698