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

Side by Side Diff: third_party/WebKit/Source/core/style/StyleFetchedImage.h

Issue 2941533002: Break StyleImage dependency on LayoutObject (Closed)
Patch Set: Address reviewer comments Created 3 years, 6 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) 2000 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 2000 Lars Knoll (knoll@kde.org)
3 * (C) 2000 Antti Koivisto (koivisto@kde.org) 3 * (C) 2000 Antti Koivisto (koivisto@kde.org)
4 * (C) 2000 Dirk Mueller (mueller@kde.org) 4 * (C) 2000 Dirk Mueller (mueller@kde.org)
5 * Copyright (C) 2003, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved. 5 * Copyright (C) 2003, 2005, 2006, 2007, 2008 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 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
50 CSSValue* ComputedCSSValue() const override; 50 CSSValue* ComputedCSSValue() const override;
51 51
52 bool CanRender() const override; 52 bool CanRender() const override;
53 bool IsLoaded() const override; 53 bool IsLoaded() const override;
54 bool ErrorOccurred() const override; 54 bool ErrorOccurred() const override;
55 LayoutSize ImageSize(const Document&, 55 LayoutSize ImageSize(const Document&,
56 float multiplier, 56 float multiplier,
57 const LayoutSize& default_object_size) const override; 57 const LayoutSize& default_object_size) const override;
58 bool ImageHasRelativeSize() const override; 58 bool ImageHasRelativeSize() const override;
59 bool UsesImageContainerSize() const override; 59 bool UsesImageContainerSize() const override;
60 void AddClient(LayoutObject*) override; 60 void AddClient(ImageResourceObserver*) override;
61 void RemoveClient(LayoutObject*) override; 61 void RemoveClient(ImageResourceObserver*) override;
62 void ImageNotifyFinished(ImageResourceContent*) override; 62 void ImageNotifyFinished(ImageResourceContent*) override;
63 String DebugName() const override { return "StyleFetchedImage"; } 63 String DebugName() const override { return "StyleFetchedImage"; }
64 PassRefPtr<Image> GetImage(const LayoutObject&, 64 PassRefPtr<Image> GetImage(const ImageResourceObserver&,
65 const Document&,
66 const ComputedStyle&,
65 const IntSize&) const override; 67 const IntSize&) const override;
66 bool KnownToBeOpaque(const Document&, const ComputedStyle&) const override; 68 bool KnownToBeOpaque(const Document&, const ComputedStyle&) const override;
67 ImageResourceContent* CachedImage() const override; 69 ImageResourceContent* CachedImage() const override;
68 70
69 DECLARE_VIRTUAL_TRACE(); 71 DECLARE_VIRTUAL_TRACE();
70 72
71 private: 73 private:
72 StyleFetchedImage(ImageResourceContent*, const Document&, const KURL&); 74 StyleFetchedImage(ImageResourceContent*, const Document&, const KURL&);
73 75
74 void Dispose(); 76 void Dispose();
75 77
76 Member<ImageResourceContent> image_; 78 Member<ImageResourceContent> image_;
77 Member<const Document> document_; 79 Member<const Document> document_;
78 const KURL url_; 80 const KURL url_;
79 }; 81 };
80 82
81 DEFINE_STYLE_IMAGE_TYPE_CASTS(StyleFetchedImage, IsImageResource()); 83 DEFINE_STYLE_IMAGE_TYPE_CASTS(StyleFetchedImage, IsImageResource());
82 84
83 } // namespace blink 85 } // namespace blink
84 #endif 86 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698