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

Side by Side Diff: third_party/WebKit/Source/core/html/HTMLImageElement.h

Issue 2864253003: Split ImageLoader::SetImage() and set flags consistently in tests (Closed)
Patch Set: Refine Created 3 years, 7 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) 1999 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 * (C) 1999 Antti Koivisto (koivisto@kde.org) 3 * (C) 1999 Antti Koivisto (koivisto@kde.org)
4 * Copyright (C) 2004, 2008, 2010 Apple Inc. All rights reserved. 4 * Copyright (C) 2004, 2008, 2010 Apple Inc. All rights reserved.
5 * Copyright (C) 2010 Google Inc. All rights reserved. 5 * Copyright (C) 2010 Google 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 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
76 bool IsServerMap() const; 76 bool IsServerMap() const;
77 77
78 String AltText() const final; 78 String AltText() const final;
79 79
80 ImageResourceContent* CachedImage() const { 80 ImageResourceContent* CachedImage() const {
81 return GetImageLoader().GetImage(); 81 return GetImageLoader().GetImage();
82 } 82 }
83 ImageResource* CachedImageResourceForImageDocument() const { 83 ImageResource* CachedImageResourceForImageDocument() const {
84 return GetImageLoader().ImageResourceForImageDocument(); 84 return GetImageLoader().ImageResourceForImageDocument();
85 } 85 }
86 void SetImageResource(ImageResourceContent* i) { 86 void SetImageForTest(ImageResourceContent* i) {
fs 2017/05/09 08:29:55 Maybe rename |i| to something more appropriate her
hiroshige 2017/05/09 23:07:57 Done.
87 GetImageLoader().SetImage(i); 87 GetImageLoader().SetImageForTest(i);
88 } 88 }
89 89
90 void SetLoadingImageDocument() { GetImageLoader().SetLoadingImageDocument(); } 90 void SetLoadingImageDocument() { GetImageLoader().SetLoadingImageDocument(); }
91 91
92 void setHeight(unsigned); 92 void setHeight(unsigned);
93 93
94 KURL Src() const; 94 KURL Src() const;
95 void SetSrc(const String&); 95 void SetSrc(const String&);
96 96
97 void setWidth(unsigned); 97 void setWidth(unsigned);
(...skipping 104 matching lines...) Expand 10 before | Expand all | Expand 10 after
202 unsigned form_was_set_by_parser_ : 1; 202 unsigned form_was_set_by_parser_ : 1;
203 unsigned element_created_by_parser_ : 1; 203 unsigned element_created_by_parser_ : 1;
204 unsigned is_fallback_image_ : 1; 204 unsigned is_fallback_image_ : 1;
205 205
206 ReferrerPolicy referrer_policy_; 206 ReferrerPolicy referrer_policy_;
207 }; 207 };
208 208
209 } // namespace blink 209 } // namespace blink
210 210
211 #endif // HTMLImageElement_h 211 #endif // HTMLImageElement_h
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/frame/ImageBitmapTest.cpp ('k') | third_party/WebKit/Source/core/html/HTMLObjectElement.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698