OLD | NEW |
1 /* | 1 /* |
2 * Copyright (C) 2004, 2005, 2008, 2009 Nikolas Zimmermann <zimmermann@kde.org> | 2 * Copyright (C) 2004, 2005, 2008, 2009 Nikolas Zimmermann <zimmermann@kde.org> |
3 * Copyright (C) 2004, 2005 Rob Buis <buis@kde.org> | 3 * Copyright (C) 2004, 2005 Rob Buis <buis@kde.org> |
4 * | 4 * |
5 * This library is free software; you can redistribute it and/or | 5 * This library is free software; you can redistribute it and/or |
6 * modify it under the terms of the GNU Library General Public | 6 * modify it under the terms of the GNU Library General Public |
7 * License as published by the Free Software Foundation; either | 7 * License as published by the Free Software Foundation; either |
8 * version 2 of the License, or (at your option) any later version. | 8 * version 2 of the License, or (at your option) any later version. |
9 * | 9 * |
10 * This library is distributed in the hope that it will be useful, | 10 * This library is distributed in the hope that it will be useful, |
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
51 } | 51 } |
52 | 52 |
53 const String& hrefString() const { return m_href->currentValue()->value(); } | 53 const String& hrefString() const { return m_href->currentValue()->value(); } |
54 | 54 |
55 // JS API | 55 // JS API |
56 SVGAnimatedString* href() const { return m_href.get(); } | 56 SVGAnimatedString* href() const { return m_href.get(); } |
57 | 57 |
58 virtual void trace(Visitor*); | 58 virtual void trace(Visitor*); |
59 | 59 |
60 protected: | 60 protected: |
61 explicit SVGURIReference(SVGElement*); | 61 SVGURIReference(); |
| 62 |
| 63 // FIXME: Oilpan: the construction of this mixin requires heap allocation, |
| 64 // which cannot be safely done with Oilpan until the object that include |
| 65 // this mixin have had its vptr initialized -- so as to be able to |
| 66 // accurately trace the entire object should a GC strike while constructing |
| 67 // the mixin. |
| 68 // |
| 69 // Try to come up with a more natural alternative and solution that doesn't |
| 70 // require hoisting the constructor code for mixins into a separate method |
| 71 // like initialize(). It makes construction of these heap-allocation mixins |
| 72 // safe in the meantime, however. |
| 73 // |
| 74 void initialize(SVGElement*); |
62 | 75 |
63 private: | 76 private: |
64 RefPtrWillBeMember<SVGAnimatedString> m_href; | 77 RefPtrWillBeMember<SVGAnimatedString> m_href; |
65 }; | 78 }; |
66 | 79 |
67 } // namespace blink | 80 } // namespace blink |
68 | 81 |
69 #endif // SVGURIReference_h | 82 #endif // SVGURIReference_h |
OLD | NEW |