Closed
Bug 1363959
Opened 7 years ago
Closed 7 years ago
Consider atomizing the strings for the comparisons while resolving a property
Categories
(Core :: XPConnect, defect, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox56 | --- | fixed |
People
(Reporter: ting, Assigned: ting)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
1.35 KB,
patch
|
bholley
:
review+
|
Details | Diff | Splinter Review |
From the profile of test case attachment 8848224 [details], I see string comparisons: js::StringEqualsAscii() xpc::TryResolvePropertyFromSpecs() [1], mozilla::dom::XrayResolveOwnProperty() [2] strcmp() xpc::XrayTraits::getExpandoObject() [3] I wonder if they worth atomizing, so the string comparison can be a simple pointer check. [1] http://searchfox.org/mozilla-central/rev/d66b9f27d5630a90b2fce4d70d4e9050f43df9b4/js/xpconnect/wrappers/XrayWrapper.cpp#424,448 [2] http://searchfox.org/mozilla-central/rev/d66b9f27d5630a90b2fce4d70d4e9050f43df9b4/dom/bindings/BindingUtils.cpp#1689,1719 [3] http://searchfox.org/mozilla-central/rev/d66b9f27d5630a90b2fce4d70d4e9050f43df9b4/js/xpconnect/wrappers/XrayWrapper.cpp#1152
Assignee | ||
Comment 1•7 years ago
|
||
(In reply to Ting-Yu Chou [:ting] from comment #0) > I wonder if they worth atomizing, so the string comparison can be a simple > pointer check. I meant I wonder if they can be atomized...
Updated•7 years ago
|
Priority: -- → P2
Whiteboard: [qf]
Updated•7 years ago
|
Whiteboard: [qf] → [qf:p1]
Assignee | ||
Comment 2•7 years ago
|
||
(In reply to Ting-Yu Chou [:ting] from comment #0) > strcmp() > xpc::XrayTraits::getExpandoObject() [3] Bug 1363963 ease this one.
Assignee | ||
Comment 3•7 years ago
|
||
Profile shows this doesn't really worth P1 after bug 1363963 landed.
Whiteboard: [qf:p1] → [qf]
Updated•7 years ago
|
Whiteboard: [qf] → [qf:p3]
Assignee | ||
Comment 4•7 years ago
|
||
Attachment #8887822 -
Flags: review?(bobbyholley)
Assignee | ||
Comment 5•7 years ago
|
||
(In reply to Ting-Yu Chou [:ting] from comment #0) > From the profile of test case attachment 8848224 [details], I see string > comparisons: > > js::StringEqualsAscii() > xpc::TryResolvePropertyFromSpecs() [1], > mozilla::dom::XrayResolveOwnProperty() [2] I can still see [2] in VTune, so the patch removes it.
Updated•7 years ago
|
Attachment #8887822 -
Flags: review?(bobbyholley) → review+
Pushed by tchou@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/861e1a6afff7 Compare jsid equality instead of string comparison in XrayResolveOwnProperty(). r=bholley
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → janus926
Status: NEW → ASSIGNED
Comment 7•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/861e1a6afff7
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox56:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
Updated•2 years ago
|
Performance Impact: --- → P3
Whiteboard: [qf:p3]
You need to log in
before you can comment on or make changes to this bug.
Description
•