Closed Bug 364617 Opened 18 years ago Closed 18 years ago

[FIX]Remove aFrame arg from most computed style methods

Categories

(Core :: DOM: CSS Object Model, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9alpha1

People

(Reporter: bzbarsky, Assigned: bzbarsky)

References

Details

Attachments

(1 file)

There's no need to pass aFrame around; we should just store the frame in a member and be done with it.
Attached patch FixSplinter Review
This is pretty mechanical -- just use mFrame throughout, pretty much.  The only interesting part is in GetStyleData() and GetPropertyCSSValue(), and maybe in GetContainingBlock().
Attachment #249370 - Flags: superreview?(roc)
Attachment #249370 - Flags: review?(roc)
Attachment #249370 - Flags: superreview?(roc)
Attachment #249370 - Flags: superreview+
Attachment #249370 - Flags: review?(roc)
Attachment #249370 - Flags: review+
Checked in.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Blocks: 364821
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: