From 16833d0fb6585117e9978d1accc3ade83e22e797 Mon Sep 17 00:00:00 2001
From: Peter Bacon Darwin <pete@bacondarwin.com>
Date: Sat, 15 Nov 2014 22:55:55 +0000
Subject: [PATCH] fix(select): ensure the label attribute is updated in
 Internet Explorer

Only changing the `<option>` text value is not enough to trigger a render
change in IE. We need to explicit update the `label` property too.

Closes #9621
Closes #10042
---
 src/ng/directive/select.js      |  2 ++
 test/ng/directive/selectSpec.js | 17 +++++++++++++++++
 2 files changed, 19 insertions(+)

diff --git a/src/ng/directive/select.js b/src/ng/directive/select.js
index 2c4754fc6..ded62426e 100644
--- a/src/ng/directive/select.js
+++ b/src/ng/directive/select.js
@@ -551,6 +551,7 @@ var selectDirective = ['$compile', '$parse', function($compile,   $parse) {
                 lastElement = existingOption.element;
                 if (existingOption.label !== option.label) {
                   lastElement.text(existingOption.label = option.label);
+                  lastElement.prop('label', existingOption.label);
                 }
                 if (existingOption.id !== option.id) {
                   lastElement.val(existingOption.id = option.id);
@@ -580,6 +581,7 @@ var selectDirective = ['$compile', '$parse', function($compile,   $parse) {
                       .val(option.id)
                       .prop('selected', option.selected)
                       .attr('selected', option.selected)
+                      .prop('label', option.label)
                       .text(option.label);
                 }
 
diff --git a/test/ng/directive/selectSpec.js b/test/ng/directive/selectSpec.js
index 90eb9509f..8c5b788ee 100644
--- a/test/ng/directive/selectSpec.js
+++ b/test/ng/directive/selectSpec.js
@@ -847,6 +847,23 @@ describe('select', function() {
       expect(scope.selected).toBe(scope.values[0]);
     });
 
+    // bug fix #9621
+    it('should update the label property', function() {
+      // ng-options="value.name for value in values"
+      // ng-model="selected"
+      createSingleSelect();
+
+      scope.$apply(function() {
+        scope.values = [{name: 'A'}, {name: 'B'}, {name: 'C'}];
+        scope.selected = scope.values[0];
+      });
+
+      var options = element.find('option');
+      expect(options.eq(0).prop('label')).toEqual('A');
+      expect(options.eq(1).prop('label')).toEqual('B');
+      expect(options.eq(2).prop('label')).toEqual('C');
+    });
+
     describe('binding', function() {
 
       it('should bind to scope value', function() {