Closed
Bug 731904
Opened 14 years ago
Closed 14 years ago
get rid nsCoreUtils::IsCorrectFrameType
Categories
(Core :: Disability Access APIs, defect)
Core
Disability Access APIs
Tracking
()
RESOLVED
FIXED
mozilla13
People
(Reporter: surkov, Assigned: capella)
References
(Blocks 1 open bug)
Details
(Whiteboard: [good first bug][mentor=surkov.alexander@gmail.com][lang=c++])
Attachments
(1 file, 1 obsolete file)
4.00 KB,
patch
|
surkov
:
review+
tbsaunde
:
feedback+
|
Details | Diff | Splinter Review |
Nowdays nsCoreUtils::IsCorrectFrameType is aFrame->GetType() == aAtom. We don't need util method for this.
Reporter | ||
Comment 1•14 years ago
|
||
(In reply to alexander :surkov from comment #0)
> Nowdays nsCoreUtils::IsCorrectFrameType is aFrame->GetType() == aAtom.
fyi, bug 190735 changed that
Assignee | ||
Comment 2•14 years ago
|
||
Attached changes built clean locally, run clean mochitest-a11y test, mochitest-plain run clean with some timeouts ...
Assignee: nobody → markcapella
Status: NEW → ASSIGNED
Comment 3•14 years ago
|
||
Comment on attachment 602729 [details] [diff] [review]
Patch (v1)
Review of attachment 602729 [details] [diff] [review]:
-----------------------------------------------------------------
::: accessible/src/base/nsAccessible.cpp
@@ +969,5 @@
> *aBoundingFrame = ancestorFrame;
> // If any other frame type, we only need to deal with the primary frame
> // Otherwise, there may be more frames attached to the same content node
> + if (!(ancestorFrame->GetType() == nsGkAtoms::inlineFrame) &&
> + !(ancestorFrame->GetType() == nsGkAtoms::textFrame))
Make this use != instead
Reporter | ||
Comment 4•14 years ago
|
||
Comment on attachment 602729 [details] [diff] [review]
Patch (v1)
r=me with Ms2ger's comment addressed (I'll fix it before landing)
Attachment #602729 -
Flags: review+
Assignee | ||
Comment 5•14 years ago
|
||
Better code patch attached ... build running ...
Attachment #602729 -
Attachment is obsolete: true
Attachment #602732 -
Flags: review?(surkov.alexander)
Reporter | ||
Updated•14 years ago
|
Attachment #602732 -
Flags: review?(surkov.alexander) → review+
Reporter | ||
Comment 6•14 years ago
|
||
Comment 7•14 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla13
Updated•14 years ago
|
Attachment #602732 -
Flags: feedback+
You need to log in
before you can comment on or make changes to this bug.
Description
•