fix(checkbox): click handling when ngCheckbox is set (#10468) (#10472)

Author: busiq

src/components/checkbox/checkbox.js

@@ -184,7 +184,7 @@ function MdCheckboxDirective(inputDirective, $mdAria, $mdConstant, $mdTheming, $
 
         scope.$apply(function() {
           // Toggle the checkbox value...
-          var viewValue = attr.ngChecked ? attr.checked : !ngModelCtrl.$viewValue;
+          var viewValue = attr.ngChecked && attr.ngClick ? attr.checked : !ngModelCtrl.$viewValue;
 
           ngModelCtrl.$setViewValue(viewValue, ev && ev.type);
           ngModelCtrl.$render();

src/components/checkbox/checkbox.spec.js

@@ -272,6 +272,14 @@ describe('mdCheckbox', function() {
       expect(checkbox.hasClass(CHECKED_CSS)).toBe(false);
     });
 
+    it('properly handles click event when ng-checked is set', function() {
+      pageScope.checked = false;
+      var checkbox = compileAndLink('<md-checkbox ng-checked="checked"></md-checkbox>');
+
+      checkbox.triggerHandler('click');
+      expect(isChecked(checkbox)).toBe(true);
+    });
+
     it('should mark the checkbox as selected on load with ng-checked', function() {
       pageScope.isChecked = function() { return true; };