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 34266 9f1580781aa3eaefdfcfd5ddfa919f4aa69440a7
parent 34265 ab66eaf0f57523abdc3457e0be73fc9d8734d255
child 34267 1c11a287b63969bba542082cf328b9301a757e5d
push id389
push userclokep@gmail.com
push dateMon, 18 Mar 2019 19:01:53 +0000
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>