Closed Bug 1491784 Opened 4 years ago Closed 4 years ago
Tab Select accesses selected Item too often
46 bytes, text/x-phabricator-request
|Details | Review|
+++ This bug was initially created as a clone of Bug #1491286 +++ The selectedItem getter is kind of expensive, we should only access it once in _handleTabSelect: https://searchfox.org/mozilla-central/rev/99cbc0aec3e1c0b65ff9052523fb5c181b248f57/browser/base/content/tabbrowser.xml#400,402 See the patch in bug 1491286 for how to fix this.
I will send a patch for this.
I guess I did not get everything right, but the diff is on phabricator now: https://phabricator.services.mozilla.com/D6287
Comment on attachment 9010365 [details] Bug 1491784 - Access selectedItem only once in _handleTabSelect. r=dao Dão Gottwald [::dao] has approved the revision.
Attachment #9010365 - Flags: review+
Attachment 9010365 [details] looks good. You can abandon attachment 9010368 [details]. Gonna land this in a bit. Thanks!
Assignee: nobody → maggus.staab
sorry for the double post. I am new to hg and mixed some commands up, because I am used to git and it seems the commands are not equivalent for some parts.
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/2f51b39c98c0 Access selectedItem only once in _handleTabSelect. r=dao
(In reply to Dão Gottwald [::dao] from comment #6) > You can abandon attachment 9010368 [details] just marked the attachment as obsolete. didn't found a button or similar to close the corresponding diff in phabricator
Yeah, no clue how to do that in phabricator. I guess you can just leave the diff open.
You need to log in before you can comment on or make changes to this bug.