Bug 1139576 - New_ functions should be static

This commit is contained in:
Alexander Surkov 2015-03-13 20:42:57 -04:00
parent 3d07947507
commit 05d5e530a2

View File

@ -136,7 +136,7 @@ MustBeAccessible(nsIContent* aContent, DocAccessible* aDocument)
//////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////
// Accessible constructors // Accessible constructors
Accessible* static Accessible*
New_HTMLLink(nsIContent* aContent, Accessible* aContext) New_HTMLLink(nsIContent* aContent, Accessible* aContext)
{ {
// Only some roles truly enjoy life as HTMLLinkAccessibles, for details // Only some roles truly enjoy life as HTMLLinkAccessibles, for details
@ -150,28 +150,28 @@ New_HTMLLink(nsIContent* aContent, Accessible* aContext)
return new HTMLLinkAccessible(aContent, aContext->Document()); return new HTMLLinkAccessible(aContent, aContext->Document());
} }
Accessible* New_HyperText(nsIContent* aContent, Accessible* aContext) static Accessible* New_HyperText(nsIContent* aContent, Accessible* aContext)
{ return new HyperTextAccessibleWrap(aContent, aContext->Document()); } { return new HyperTextAccessibleWrap(aContent, aContext->Document()); }
Accessible* New_HTMLFigcaption(nsIContent* aContent, Accessible* aContext) static Accessible* New_HTMLFigcaption(nsIContent* aContent, Accessible* aContext)
{ return new HTMLFigcaptionAccessible(aContent, aContext->Document()); } { return new HTMLFigcaptionAccessible(aContent, aContext->Document()); }
Accessible* New_HTMLFigure(nsIContent* aContent, Accessible* aContext) static Accessible* New_HTMLFigure(nsIContent* aContent, Accessible* aContext)
{ return new HTMLFigureAccessible(aContent, aContext->Document()); } { return new HTMLFigureAccessible(aContent, aContext->Document()); }
Accessible* New_HTMLLegend(nsIContent* aContent, Accessible* aContext) static Accessible* New_HTMLLegend(nsIContent* aContent, Accessible* aContext)
{ return new HTMLLegendAccessible(aContent, aContext->Document()); } { return new HTMLLegendAccessible(aContent, aContext->Document()); }
Accessible* New_HTMLOption(nsIContent* aContent, Accessible* aContext) static Accessible* New_HTMLOption(nsIContent* aContent, Accessible* aContext)
{ return new HTMLSelectOptionAccessible(aContent, aContext->Document()); } { return new HTMLSelectOptionAccessible(aContent, aContext->Document()); }
Accessible* New_HTMLOptgroup(nsIContent* aContent, Accessible* aContext) static Accessible* New_HTMLOptgroup(nsIContent* aContent, Accessible* aContext)
{ return new HTMLSelectOptGroupAccessible(aContent, aContext->Document()); } { return new HTMLSelectOptGroupAccessible(aContent, aContext->Document()); }
Accessible* New_HTMLList(nsIContent* aContent, Accessible* aContext) static Accessible* New_HTMLList(nsIContent* aContent, Accessible* aContext)
{ return new HTMLListAccessible(aContent, aContext->Document()); } { return new HTMLListAccessible(aContent, aContext->Document()); }
Accessible* static Accessible*
New_HTMLListitem(nsIContent* aContent, Accessible* aContext) New_HTMLListitem(nsIContent* aContent, Accessible* aContext)
{ {
// If list item is a child of accessible list then create an accessible for // If list item is a child of accessible list then create an accessible for
@ -183,7 +183,7 @@ New_HTMLListitem(nsIContent* aContent, Accessible* aContext)
return nullptr; return nullptr;
} }
Accessible* static Accessible*
New_HTMLDefinition(nsIContent* aContent, Accessible* aContext) New_HTMLDefinition(nsIContent* aContent, Accessible* aContext)
{ {
if (aContext->IsList()) if (aContext->IsList())
@ -191,16 +191,16 @@ New_HTMLDefinition(nsIContent* aContent, Accessible* aContext)
return nullptr; return nullptr;
} }
Accessible* New_HTMLLabel(nsIContent* aContent, Accessible* aContext) static Accessible* New_HTMLLabel(nsIContent* aContent, Accessible* aContext)
{ return new HTMLLabelAccessible(aContent, aContext->Document()); } { return new HTMLLabelAccessible(aContent, aContext->Document()); }
Accessible* New_HTMLOutput(nsIContent* aContent, Accessible* aContext) static Accessible* New_HTMLOutput(nsIContent* aContent, Accessible* aContext)
{ return new HTMLOutputAccessible(aContent, aContext->Document()); } { return new HTMLOutputAccessible(aContent, aContext->Document()); }
Accessible* New_HTMLProgress(nsIContent* aContent, Accessible* aContext) static Accessible* New_HTMLProgress(nsIContent* aContent, Accessible* aContext)
{ return new HTMLProgressMeterAccessible(aContent, aContext->Document()); } { return new HTMLProgressMeterAccessible(aContent, aContext->Document()); }
Accessible* static Accessible*
New_HTMLTableHeaderCell(nsIContent* aContent, Accessible* aContext) New_HTMLTableHeaderCell(nsIContent* aContent, Accessible* aContext)
{ {
if (aContext->IsTableRow() && aContext->GetContent() == aContent->GetParent()) if (aContext->IsTableRow() && aContext->GetContent() == aContent->GetParent())
@ -208,7 +208,7 @@ New_HTMLTableHeaderCell(nsIContent* aContent, Accessible* aContext)
return nullptr; return nullptr;
} }
Accessible* static Accessible*
New_HTMLTableHeaderCellIfScope(nsIContent* aContent, Accessible* aContext) New_HTMLTableHeaderCellIfScope(nsIContent* aContent, Accessible* aContext)
{ {
if (aContext->IsTableRow() && aContext->GetContent() == aContent->GetParent() && if (aContext->IsTableRow() && aContext->GetContent() == aContent->GetParent() &&