WIP - debug Mobile Safari 8

Author: gkalpak

Diff for: .travis.yml

@@ -15,11 +15,11 @@ branches:
 
 env:
   matrix:
-    - JOB=ci-checks
+    # - JOB=ci-checks
     - JOB=unit BROWSER_PROVIDER=saucelabs
-    - JOB=docs-e2e BROWSER_PROVIDER=saucelabs
-    - JOB=e2e TEST_TARGET=jqlite BROWSER_PROVIDER=saucelabs
-    - JOB=e2e TEST_TARGET=jquery BROWSER_PROVIDER=saucelabs
+    # - JOB=docs-e2e BROWSER_PROVIDER=saucelabs
+    # - JOB=e2e TEST_TARGET=jqlite BROWSER_PROVIDER=saucelabs
+    # - JOB=e2e TEST_TARGET=jquery BROWSER_PROVIDER=saucelabs
   global:
     - CXX=g++-4.8  # node 4 likes the G++ v4.8 compiler
     - SAUCE_USERNAME=angular-ci

Diff for: Gruntfile.js

@@ -329,7 +329,8 @@ module.exports = function(grunt) {
   grunt.registerTask('test:jquery-2.1', 'Run the jQuery 2.1 unit tests with Karma', ['tests:jquery-2.1']);
   grunt.registerTask('test:modules', 'Run the Karma module tests with Karma', ['build', 'tests:modules']);
   grunt.registerTask('test:docs', 'Run the doc-page tests with Karma', ['package', 'tests:docs']);
-  grunt.registerTask('test:unit', 'Run unit, jQuery and Karma module tests with Karma', ['test:jqlite', 'test:jquery', 'test:jquery-2.2', 'test:jquery-2.1', 'test:modules']);
+  // grunt.registerTask('test:unit', 'Run unit, jQuery and Karma module tests with Karma', ['test:jqlite', 'test:jquery', 'test:jquery-2.2', 'test:jquery-2.1', 'test:modules']);
+  grunt.registerTask('test:unit', 'Run unit, jQuery and Karma module tests with Karma', ['test:jqlite']);
   grunt.registerTask('test:protractor', 'Run the end to end tests with Protractor and keep a test server running in the background', ['webdriver', 'connect:testserver', 'protractor:normal']);
   grunt.registerTask('test:travis-protractor', 'Run the end to end tests with Protractor for Travis CI builds', ['connect:testserver', 'protractor:travis']);
   grunt.registerTask('test:ci-protractor', 'Run the end to end tests with Protractor for Jenkins CI builds', ['webdriver', 'connect:testserver', 'protractor:jenkins']);

Diff for: scripts/travis/build.sh

@@ -11,12 +11,13 @@ elif [ "$JOB" == "unit" ]; then
   if [ "$BROWSER_PROVIDER" == "browserstack" ]; then
     BROWSERS="BS_Chrome,BS_Safari,BS_Firefox,BS_IE_9,BS_IE_10,BS_IE_11,BS_iOS"
   else
-    BROWSERS="SL_Chrome,SL_Firefox,SL_Safari_8,SL_Safari_9,SL_IE_9,SL_IE_10,SL_IE_11,SL_iOS"
+    # BROWSERS="SL_Chrome,SL_Firefox,SL_Safari_8,SL_Safari_9,SL_IE_9,SL_IE_10,SL_IE_11,SL_iOS"
+    BROWSERS="SL_Chrome,SL_iOS"
   fi
 
-  grunt test:promises-aplus
+  # grunt test:promises-aplus
   grunt test:unit --browsers="$BROWSERS" --reporters=dots
-  grunt tests:docs --browsers="$BROWSERS" --reporters=dots
+  # grunt tests:docs --browsers="$BROWSERS" --reporters=dots
 elif [ "$JOB" == "docs-e2e" ]; then
   grunt test:travis-protractor --specs="docs/app/e2e/**/*.scenario.js"
 elif [ "$JOB" == "e2e" ]; then

Diff for: src/apis.js

@@ -42,6 +42,8 @@ function NgMapShim() {
   this._values = [];
   this._lastKey = NaN;
   this._lastIndex = -1;
+
+  this._map = new window.Map();
 }
 NgMapShim.prototype = {
   _idx: function(key) {
@@ -52,17 +54,41 @@ NgMapShim.prototype = {
     this._lastIndex = this._keys.indexOf(key);
     return this._lastIndex;
   },
+  _reportDiff: function(method, key, value, _value, len, _len) {
+    console.log('-------------------------');
+    console.log('DIFF in `' + method + '`:');
+    console.log('  len1  :', len);
+    console.log('  len2  :', _len);
+    console.log('  value1:', value);
+    console.log('  value2:', _value);
+    console.log('  key   :', key);
+    console.log('  entries2:');
+
+    this._map.forEach(function(e) {
+      console.log('    ', e);
+    });
+  },
   _transformKey: function(key) {
     return isNumberNaN(key) ? nanKey : key;
   },
   get: function(key) {
+    var _value = this._map.get(key);
     key = this._transformKey(key);
     var idx = this._idx(key);
     if (idx !== -1) {
-      return this._values[idx];
+      var value = this._values[idx];
+      var len = this._keys.length;
+      var _len = this._map.size;
+      if (value !== _value) this._reportDiff('GET', key, value, _value, len, _len);
+      return value;
     }
+    var len = this._keys.length;
+    var _len = this._map.size;
+    if (_value !== undefined) this._reportDiff('GET', key, undefined, _value, len, _len);
   },
   set: function(key, value) {
+    this._map.set(key, value);
+
     key = this._transformKey(key);
     var idx = this._idx(key);
     if (idx === -1) {
@@ -71,25 +97,39 @@ NgMapShim.prototype = {
     this._keys[idx] = key;
     this._values[idx] = value;
 
+    var len = this._keys.length;
+    var _len = this._map.size;
+    if (len !== _len) this._reportDiff('SET', key, value, undefined, len, _len);
+
     // Support: IE11
     // Do not `return this` to simulate the partial IE11 implementation
   },
   delete: function(key) {
+    var _value = this._map.delete(key);
     key = this._transformKey(key);
     var idx = this._idx(key);
     if (idx === -1) {
+      var len = this._keys.length;
+      var _len = this._map.size;
+      if (_value) this._reportDiff('DELETE', key, false, _value, len, _len);
       return false;
     }
     this._keys.splice(idx, 1);
     this._values.splice(idx, 1);
     this._lastKey = NaN;
     this._lastIndex = -1;
+
+    var len = this._keys.length;
+    var _len = this._map.size;
+    if (!_value) this._reportDiff('DELETE', key, true, _value, len, _len);
+    if (len !== _len) this._reportDiff('DELETE_', key, true, _value, len, _len);
+
     return true;
   }
 };
 
 var NgMap = isFunction(window.Map) && toString.call(window.Map.prototype) === '[object Map]'
-  ? window.Map
+  ? NgMapShim//window.Map
   : NgMapShim;
 
 var $$MapProvider = [/** @this */function() {