Review fixes

Author: Daniel Herzog

src/ecma-debugger/runtimes.js

@@ -1342,13 +1342,15 @@ cls.EcmascriptDebugger["6.0"].Runtimes = function(service_version)
 
   this.reloadWindow = function(window_id)
   {
-    __selected_window = window_id || __selected_window;
-    if (__selected_window)
+    if (!window_id)
+      window_id = __selected_window;
+
+    if (window_id)
     {
-      if (!__windows_reloaded[__selected_window])
-        __windows_reloaded[__selected_window] = 1;
+      if (!__windows_reloaded[window_id])
+        __windows_reloaded[window_id] = 1;
 
-      var rt_id = this.getRuntimeIdsFromWindow(__selected_window)[0];
+      var rt_id = this.getRuntimeIdsFromWindow(window_id)[0];
       if (window.services['ecmascript-debugger'] &&
           window.services['ecmascript-debugger'].is_enabled &&
           // For background processes we can not use the exec service.

src/network/network_service.js

@@ -224,7 +224,7 @@ cls.NetworkLoggerService = function()
     if (!this._current_context)
       return;
 
-    entry._called_get_body = true;
+    entry.called_get_body = true;
     var contentmode = cls.ResourceUtil.mime_to_content_mode(entry.mime);
     var typecode = {datauri: 3, string: 1}[contentmode] || 1;
     var tag = window.tagManager.set_callback(this, this._handle_get_resource, [entry]);
@@ -590,10 +590,10 @@ cls.NetworkLoggerEntry = function(id, resource_id, document_id, context_starttim
   this.requests_responses = [];
   this.current_responsecode = null;
   this.error_in_current_response = false;
+  this.called_get_body = false;
   this._current_request = null;
   this._current_response = null;
   this._set_is_finished_on_responsefinished = false;
-  this._called_get_body = false;
 };
 
 cls.NetworkLoggerEntryPrototype = function()
@@ -980,7 +980,7 @@ cls.NetworkLoggerEntryPrototype = function()
     // Decide if body should be fetched, for when content-tracking is off or it's a cached request.
     if (
       this.is_finished &&
-      !this._called_get_body &&
+      !this.called_get_body &&
       (!this._current_response || !this._current_response.responsebody) &&
       // When we have a response, but didn't see responsefinished, it means there's really no
       // responsebody. Don't attempt to fetch it.