javascript-indonesias · GulajavaMinistudio · Nov 4, 2018 · Sep 27, 2018 · Oct 18, 2018 · Nov 2, 2018
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -746,7 +746,7 @@ If you rely on the $modelValue validation, you can overwrite the `min`/`max` val
   link: function(scope, element, attrs, ctrl) {
     var maxValidator = ctrl.$validators.max;
 
-    ctrk.$validators.max = function(modelValue, viewValue) {
+    ctrl.$validators.max = function(modelValue, viewValue) {
       return maxValidator(modelValue, modelValue);
     };
   }
@@ -1579,7 +1579,7 @@ If you rely on the $modelValue validation, you can overwrite the `min`/`max` val
   link: function(scope, element, attrs, ctrl) {
     var maxValidator = ctrl.$validators.max;
 
-    ctrk.$validators.max = function(modelValue, viewValue) {
+    ctrl.$validators.max = function(modelValue, viewValue) {
       return maxValidator(modelValue, modelValue);
     };
   }

diff --git a/docs/content/guide/migration.ngdoc b/docs/content/guide/migration.ngdoc
@@ -148,7 +148,7 @@ custom directive, as seen in the following example directive definition object:
   link: function(scope, element, attrs, ctrl) {
     var maxValidator = ctrl.$validators.max;
 
-    ctrk.$validators.max = function(modelValue, viewValue) {
+    ctrl.$validators.max = function(modelValue, viewValue) {
       return maxValidator(modelValue, modelValue);
     };
   }

diff --git a/package.json b/package.json
@@ -83,13 +83,13 @@
     "npm-run": "^4.1.0",
     "open-sans-fontface": "^1.4.0",
     "promises-aplus-tests": "~2.1.0",
-    "protractor": "^5.1.2",
+    "protractor": "^5.4.1",
     "q": "~1.0.0",
     "q-io": "^1.10.9",
     "qq": "^0.3.5",
     "rewire": "~2.1.0",
     "sax": "^1.1.1",
-    "selenium-webdriver": "^2.53.1",
+    "selenium-webdriver": "^4.0.0-alpha.1",
     "semver": "^5.4.1",
     "serve-favicon": "^2.3.0",
     "serve-index": "^1.8.0",

diff --git a/src/AngularPublic.js b/src/AngularPublic.js
@@ -7,7 +7,7 @@
 
   htmlAnchorDirective,
   inputDirective,
-  inputDirective,
+  hiddenInputBrowserCacheDirective,
   formDirective,
   scriptDirective,
   selectDirective,
@@ -221,7 +221,8 @@ function publishExternalAPI(angular) {
             ngModelOptions: ngModelOptionsDirective
         }).
         directive({
-          ngInclude: ngIncludeFillContentDirective
+          ngInclude: ngIncludeFillContentDirective,
+          input: hiddenInputBrowserCacheDirective
         }).
         directive(ngAttributeAliasDirectives).
         directive(ngEventDirectives);

diff --git a/src/ng/directive/input.js b/src/ng/directive/input.js
@@ -2193,6 +2193,48 @@ var inputDirective = ['$browser', '$sniffer', '$filter', '$parse',
 }];
 
 
+var hiddenInputBrowserCacheDirective = function() {
+  var valueProperty = {
+    configurable: true,
+    enumerable: false,
+    get: function() {
+      return this.getAttribute('value') || '';
+    },
+    set: function(val) {
+      this.setAttribute('value', val);
+    }
+  };
+
+  return {
+    restrict: 'E',
+    priority: 200,
+    compile: function(_, attr) {
+      if (lowercase(attr.type) !== 'hidden') {
+        return;
+      }
+
+      return {
+        pre: function(scope, element, attr, ctrls) {
+          var node = element[0];
+
+          // Support: Edge
+          // Moving the DOM around prevents autofillling
+          if (node.parentNode) {
+            node.parentNode.insertBefore(node, node.nextSibling);
+          }
+
+          // Support: FF, IE
+          // Avoiding direct assignment to .value prevents autofillling
+          if (Object.defineProperty) {
+            Object.defineProperty(node, 'value', valueProperty);
+          }
+        }
+      };
+    }
+  };
+};
+
+
 
 var CONSTANT_VALUE_REGEXP = /^(true|false|\d+)$/;
 /**

diff --git a/test/e2e/fixtures/back2dom/index.html b/test/e2e/fixtures/back2dom/index.html
@@ -0,0 +1,13 @@
+<!DOCTYPE html>
+<html ng-app="test">
+  <body ng-class="{hacked: internalFnCalled}">
+    <form>
+      <input id="input1" type="hidden" value="{{value}}" />
+      <input id="input2" type="hidden" ng-value="value" />
+
+      <textarea ng-model="value"></textarea>
+    </form>
+    <script src="angular.js"></script>
+    <script src="script.js"></script>
+  </body>
+</html>
diff --git a/test/e2e/fixtures/back2dom/script.js b/test/e2e/fixtures/back2dom/script.js
@@ -0,0 +1,11 @@
+'use strict';
+
+angular
+  .module('test', [])
+  .run(function($rootScope) {
+    $rootScope.internalFnCalled = false;
+
+    $rootScope.internalFn = function() {
+        $rootScope.internalFnCalled = true;
+    };
+  });
diff --git a/test/e2e/tests/input-hidden.spec.js b/test/e2e/tests/input-hidden.spec.js
@@ -14,4 +14,72 @@ describe('hidden thingy', function() {
     var expectedValue = browser.params.browser === 'safari' ? '{{ 7 * 6 }}' : '';
     expect(element(by.css('input')).getAttribute('value')).toEqual(expectedValue);
   });
+
+  it('should prevent browser autofill on browser.refresh', function() {
+
+    loadFixture('back2dom');
+    expect(element(by.css('#input1')).getAttribute('value')).toEqual('');
+    expect(element(by.css('#input2')).getAttribute('value')).toEqual('');
+
+    element(by.css('textarea')).sendKeys('{{ internalFn() }}');
+
+    expect(element(by.css('#input1')).getAttribute('value')).toEqual('{{ internalFn() }}');
+    expect(element(by.css('#input2')).getAttribute('value')).toEqual('{{ internalFn() }}');
+    expect(element(by.css('body')).getAttribute('class')).toBe('');
+
+    browser.refresh();
+    expect(element(by.css('body')).getAttribute('class')).toBe('');
+  });
+
+  it('should prevent browser autofill on location.reload', function() {
+
+    loadFixture('back2dom');
+    expect(element(by.css('#input1')).getAttribute('value')).toEqual('');
+    expect(element(by.css('#input2')).getAttribute('value')).toEqual('');
+
+    element(by.css('textarea')).sendKeys('{{ internalFn() }}');
+
+    expect(element(by.css('#input1')).getAttribute('value')).toEqual('{{ internalFn() }}');
+    expect(element(by.css('#input2')).getAttribute('value')).toEqual('{{ internalFn() }}');
+    expect(element(by.css('body')).getAttribute('class')).toBe('');
+
+    browser.driver.executeScript('location.reload()');
+    expect(element(by.css('body')).getAttribute('class')).toBe('');
+  });
+
+  it('should prevent browser autofill on history.back', function() {
+
+    loadFixture('back2dom');
+    expect(element(by.css('#input1')).getAttribute('value')).toEqual('');
+    expect(element(by.css('#input2')).getAttribute('value')).toEqual('');
+
+    element(by.css('textarea')).sendKeys('{{ internalFn() }}');
+
+    expect(element(by.css('#input1')).getAttribute('value')).toEqual('{{ internalFn() }}');
+    expect(element(by.css('#input2')).getAttribute('value')).toEqual('{{ internalFn() }}');
+    expect(element(by.css('body')).getAttribute('class')).toBe('');
+
+    loadFixture('sample');
+
+    browser.driver.executeScript('history.back()');
+    expect(element(by.css('body')).getAttribute('class')).toBe('');
+  });
+
+  it('should prevent browser autofill on history.forward', function() {
+
+    loadFixture('sample');
+    loadFixture('back2dom');
+    expect(element(by.css('#input1')).getAttribute('value')).toEqual('');
+    expect(element(by.css('#input2')).getAttribute('value')).toEqual('');
+
+    element(by.css('textarea')).sendKeys('{{ internalFn() }}');
+
+    expect(element(by.css('#input1')).getAttribute('value')).toEqual('{{ internalFn() }}');
+    expect(element(by.css('#input2')).getAttribute('value')).toEqual('{{ internalFn() }}');
+    expect(element(by.css('body')).getAttribute('class')).toBe('');
+
+    browser.driver.executeScript('history.back()');
+    browser.driver.executeScript('history.forward()');
+    expect(element(by.css('body')).getAttribute('class')).toBe('');
+  });
 });