Bug 1521417 - Use getCellAt return values instead of output parameters; r=philipp
authorMakeMyDay <makemyday@gmx-topmail.de>
Sun, 20 Jan 2019 21:43:54 +0100
changeset 33412 9f1580781aa3
parent 33411 ab66eaf0f575
child 33413 1c11a287b639
push id2368
push userclokep@gmail.com
push dateMon, 28 Jan 2019 21:12:50 +0000
treeherdercomm-beta@56d23c07d815 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersphilipp
bugs1521417
Bug 1521417 - Use getCellAt return values instead of output parameters; r=philipp
calendar/base/content/widgets/calendar-list-tree.xml
--- a/calendar/base/content/widgets/calendar-list-tree.xml
+++ b/calendar/base/content/widgets/calendar-list-tree.xml
@@ -641,21 +641,20 @@
         <parameter name="event"/>
         <parameter name="aCol"/>
         <parameter name="aRow"/>
         <body><![CDATA[
             if (event.clientX && event.clientY) {
                 // If we have a client point, get the row directly from the client
                 // point.
                 aRow = aRow || {};
-                this.treebox.getCellAt(event.clientX,
-                                       event.clientY,
-                                       aRow,
-                                       aCol || {},
-                                       {});
+                aCol = aCol || {};
+                let { col, row } = this.treebox.getCellAt(event.clientX, event.clientY);
+                aCol.value = col;
+                aRow.value = row;
             } else if (document.popupNode && document.popupNode.contextCalendar) {
                 // Otherwise, we can try to get the context calendar from the popupNode.
                 return document.popupNode.contextCalendar;
             }
             return aRow && aRow.value > -1 && this.mCalendarList[aRow.value];
         ]]></body>
       </method>