bug 1155829 - inline MaiHyperlink::GetAtkHyperlink r=surkov
--- a/accessible/atk/nsMaiHyperlink.cpp
+++ b/accessible/atk/nsMaiHyperlink.cpp
@@ -89,46 +89,35 @@ mai_atk_hyperlink_get_type(void)
}
return type;
}
MaiHyperlink::MaiHyperlink(Accessible* aHyperLink) :
mHyperlink(aHyperLink),
mMaiAtkHyperlink(nullptr)
{
-}
-
-MaiHyperlink::~MaiHyperlink()
-{
- if (mMaiAtkHyperlink) {
- MAI_ATK_HYPERLINK(mMaiAtkHyperlink)->maiHyperlink = nullptr;
- g_object_unref(mMaiAtkHyperlink);
- }
-}
-
-AtkHyperlink*
-MaiHyperlink::GetAtkHyperlink(void)
-{
- NS_ENSURE_TRUE(mHyperlink, nullptr);
-
- if (mMaiAtkHyperlink)
- return mMaiAtkHyperlink;
-
if (!mHyperlink->IsLink())
- return nullptr;
+ return;
mMaiAtkHyperlink =
reinterpret_cast<AtkHyperlink *>
(g_object_new(mai_atk_hyperlink_get_type(), nullptr));
NS_ASSERTION(mMaiAtkHyperlink, "OUT OF MEMORY");
- NS_ENSURE_TRUE(mMaiAtkHyperlink, nullptr);
+ if (!mMaiAtkHyperlink)
+ return;
MAI_ATK_HYPERLINK(mMaiAtkHyperlink)->maiHyperlink = this;
+}
- return mMaiAtkHyperlink;
+MaiHyperlink::~MaiHyperlink()
+{
+ if (mMaiAtkHyperlink) {
+ MAI_ATK_HYPERLINK(mMaiAtkHyperlink)->maiHyperlink = nullptr;
+ g_object_unref(mMaiAtkHyperlink);
+ }
}
/* static functions for ATK callbacks */
void
classInitCB(AtkHyperlinkClass *aClass)
{
--- a/accessible/atk/nsMaiHyperlink.h
+++ b/accessible/atk/nsMaiHyperlink.h
@@ -22,17 +22,17 @@ namespace a11y {
class MaiHyperlink
{
public:
explicit MaiHyperlink(Accessible* aHyperLink);
~MaiHyperlink();
public:
- AtkHyperlink *GetAtkHyperlink(void);
+ AtkHyperlink* GetAtkHyperlink() const { return mMaiAtkHyperlink; }
Accessible* GetAccHyperlink()
{ return mHyperlink && mHyperlink->IsLink() ? mHyperlink : nullptr; }
protected:
Accessible* mHyperlink;
AtkHyperlink* mMaiAtkHyperlink;
};