fix(enableInstrumentation): sync instrumentation status

panel/components/inspected-app/inspected-app.js

@@ -1,9 +1,9 @@
 'use strict';
 
 angular.module('batarang.inspected-app', []).
-  service('inspectedApp', ['$rootScope', inspectedAppService]);
+  service('inspectedApp', ['$rootScope', '$q', inspectedAppService]);
 
-function inspectedAppService($rootScope) {
+function inspectedAppService($rootScope, $q) {
 
   // TODO: maybe state should live elsewhere
   var scopes = this.scopes = {},
@@ -24,11 +24,23 @@ function inspectedAppService($rootScope) {
   this.enableInstrumentation = function (setting) {
     setting = !!setting;
     chrome.devtools.inspectedWindow.eval(
-      "window.document.cookie = '__ngDebug=" + setting + ";';" +
+      "(function () {" +
-      "window.document.location.reload();"
+        "var prev = document.cookie.indexOf('__ngDebug=true') !== -1;" +
+        "if (prev !== " + setting + ") {" +
+          "window.document.cookie = '__ngDebug=" + setting + ";';" +
+          "window.document.location.reload();" +
+        "}" +
+      "}())"
     );
   };
 
+  this.getInstrumentationStatus = function () {
+    return $q(function(resolve, reject) {
+      chrome.devtools.inspectedWindow.eval(
+          "document.cookie.indexOf('__ngDebug=true') !== -1", resolve);
+    });
+  };
+
   /*
    * sets window.$scope to the scope of the given id
    */

panel/components/tabs/tabs.js

@@ -15,10 +15,9 @@ directive('batTabs', function ($compile, $templateCache, $http, inspectedApp) {
         panes.push(pane);
       };
 
-      $scope.$watch('enabled', function (newVal) {
+      inspectedApp.getInstrumentationStatus().then(function (status) {
-        if (newVal === !!newVal) {
+        $scope.enabled = status;
-          inspectedApp.enableInstrumentation(newVal);
+        $scope.$watch('enabled', inspectedApp.enableInstrumentation);
-        }
       });
     },
     link: function (scope, element, attr) {