Closed
Bug 526953
Opened 16 years ago
Closed 16 years ago
rename nsPluginInstanceOwner's "mOwner" member variable to something less confusing
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
People
(Reporter: jaas, Assigned: jaas)
Details
Attachments
(1 file)
|
25.90 KB,
patch
|
jst
:
review+
jst
:
superreview+
|
Details | Diff | Splinter Review |
nsPluginInstanceOwner has a member variable named "mOwner" which is surprisingly confusing in the context of nsObjectFrame.cpp. In that context one thinks of the "owner" as the nsPluginInstanceOwner object, but in fact it is a reference to the nsObjectFrame.
Attachment #410717 -
Flags: review?(roc)
Attachment #410717 -
Flags: review?(roc) → review?(jst)
|
||
Updated•16 years ago
|
Attachment #410717 -
Flags: superreview+
Attachment #410717 -
Flags: review?(jst)
Attachment #410717 -
Flags: review+
|
|
||
Comment 1•16 years ago
|
||
Comment on attachment 410717 [details] [diff] [review]
fix v1.0
Good cleanup! r+sr=jst
pushed to mozilla-central
http://hg.mozilla.org/mozilla-central/rev/e980db595f62
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•