Introduce DEFINE_AX_OBJECT_TYPE_CASTS to replace manual toFoo() in accessibility child class

Source/core/accessibility/AXListBoxOption.h

Index: Source/core/accessibility/AXListBoxOption.h
diff --git a/Source/core/accessibility/AXListBoxOption.h b/Source/core/accessibility/AXListBoxOption.h
index 63c634a1d7c3ec93b235ebc51070e466becd9cb0..02e04330ebc45d1448d318a51d5c4fe9f3c4c9ab 100644
--- a/Source/core/accessibility/AXListBoxOption.h
+++ b/Source/core/accessibility/AXListBoxOption.h
@@ -75,20 +75,7 @@ private:
     AXObject* listBoxOptionAXObject(HTMLElement*) const;
 };
 
-inline AXListBoxOption* toAXListBoxOption(AXObject* object)
-{
-    ASSERT_WITH_SECURITY_IMPLICATION(!object || object->isListBoxOption());
-    return static_cast<AXListBoxOption*>(object);
-}
-
-inline const AXListBoxOption* toAXListBoxOption(const AXObject* object)
-{
-    ASSERT_WITH_SECURITY_IMPLICATION(!object || object->isListBoxOption());
-    return static_cast<const AXListBoxOption*>(object);
-}
-
-// This will catch anyone doing an unnecessary cast.
-void toAXListBoxOption(const AXListBoxOption*);
+DEFINE_AX_OBJECT_TYPE_CASTS(AXListBoxOption, isListBoxOption());

[r.kasibhatla, 2013/10/21 12:28:31]

Wouldn't it be better to construct even isFoo from object type, something like 
DEFINE_TYPE_CASTS(thisType, AXObject, object, object->is##AXObject(),
object.is##AXObject())

1. It would really simplify the usage for devs - just define the macro passing
classname.
2. There would be consistency even in naming of isFoo() calls - just like
toFoo()
3. It would make regex matching in style-checker efficient - what to exactly
look out for.

[tkent, 2013/10/21 22:06:11]

We should go ahead with this CL as is, and discuss this separately.
This issue affects whole the code, and we need to discuss it in blink-dev.

IMO, renaming AXObject::isFoo to AXObject::isAXFoo looks redundant. We already
have AX prefix in the class name.

 
 } // namespace WebCore