From 80ef8f37775defa6ed8a35ca57d11d81b93b1e3f Mon Sep 17 00:00:00 2001
From: gorhill <rhill@raymondhill.net>
Date: Sat, 10 Jan 2015 11:32:24 -0500
Subject: [PATCH] code review

---
 src/js/popup.js | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/js/popup.js b/src/js/popup.js
index cd01c129e..c232459ac 100644
--- a/src/js/popup.js
+++ b/src/js/popup.js
@@ -299,7 +299,6 @@ var renderPopup = function() {
 
     // Condition for dynamic filtering toggler:
     // - Advanced user
-    uDom('body').removeClass('dirty');
     uDom('body').toggleClass('advancedUser', popupData.advancedUserEnabled);
 
     uDom('#switch').toggleClass('off', popupData.pageURL === '' || !popupData.netFilteringSwitch);
@@ -544,15 +543,15 @@ var reloadTab = function() {
 //
 // - broadcasting() is not an option given there could be a lot of tabs opened,
 //   and maybe even many frames within these tabs, i.e. unacceptable overhead 
-//   regardless of whether the popup is opened or not).
+//   regardless of whether the popup is opened or not.
 //
 // - Modifying the messaging API is not an option, as this would require
 //   revisiting all platform-specific code to support targeted broadcasting,
 //   which who knows could be not so trivial for some platforms.
 //
 // A well done polling is a better anyways IMO, I prefer that data is pulled 
-// on demand rather than having the main process assumes somebody may need to
-// be expressly notified.
+// on demand rather than forcing the main process to assume a client may need
+// it and thus having to push it all the time unconditionally.
 
 var pollForContentChange = (function() {
     var pollTimer = null;