From b583a4e7932d19f6a39f34bc3b2295abbd69ae8f Mon Sep 17 00:00:00 2001
From: Valera Rozuvan <valera@edx.org>
Date: Wed, 14 Aug 2013 11:56:13 +0300
Subject: [PATCH] Optimizing code. Caching an element selector.

---
 .../js/src/video/08_video_speed_control.js    | 48 ++++++++++---------
 1 file changed, 25 insertions(+), 23 deletions(-)

diff --git a/common/lib/xmodule/xmodule/js/src/video/08_video_speed_control.js b/common/lib/xmodule/xmodule/js/src/video/08_video_speed_control.js
index 87d0b31b10..27e4888d0d 100644
--- a/common/lib/xmodule/xmodule/js/src/video/08_video_speed_control.js
+++ b/common/lib/xmodule/xmodule/js/src/video/08_video_speed_control.js
@@ -77,6 +77,8 @@ function () {
      * @returns {undefined}
      */
     function _bindHandlers(state) {
+        var speedLinks;
+
         state.videoSpeedControl.videoSpeedsEl.find('a')
             .on('click', state.videoSpeedControl.changeVideoSpeed);
 
@@ -146,29 +148,29 @@ function () {
             // ******************************
             // Attach 'focus', and 'blur' events to elements which represent
             // individual speed entries.
-            state.videoSpeedControl.videoSpeedsEl.find('a.speed_link:last')
-                .on('blur', function () {
-                    // If we have reached the last speed entry, and the focus
-                    // changes to the next element, we need to hide the speeds
-                    // control drop-down.
-                    state.videoSpeedControl.el.removeClass('open');
-                });
-            state.videoSpeedControl.videoSpeedsEl.find('a.speed_link:first')
-                .on('blur', function () {
-                    // This flag will indicate that the focus to the next
-                    // element that will receive it is comming from the first
-                    // speed entry.
-                    //
-                    // This flag will be used to correctly handle scenario of
-                    // tabbing backwards.
-                    state.firstSpeedBlur = true;
-                });
-            state.videoSpeedControl.videoSpeedsEl.find('a.speed_link')
-                .on('focus', function () {
-                    // Clear the flag which is only set when we are un-focusing
-                    // (the blur event) from the first speed entry.
-                    state.firstSpeedBlur = false;
-                });
+            speedLinks = state.videoSpeedControl.videoSpeedsEl
+                .find('a.speed_link');
+
+            speedLinks.last().on('blur', function () {
+                // If we have reached the last speed entry, and the focus
+                // changes to the next element, we need to hide the speeds
+                // control drop-down.
+                state.videoSpeedControl.el.removeClass('open');
+            });
+            speedLinks.first().on('blur', function () {
+                // This flag will indicate that the focus to the next
+                // element that will receive it is comming from the first
+                // speed entry.
+                //
+                // This flag will be used to correctly handle scenario of
+                // tabbing backwards.
+                state.firstSpeedBlur = true;
+            });
+            speedLinks.on('focus', function () {
+                // Clear the flag which is only set when we are un-focusing
+                // (the blur event) from the first speed entry.
+                state.firstSpeedBlur = false;
+            });
         }
     }