From b49b0864d3d288daf8c09ce1768adb12a2e16c8c Mon Sep 17 00:00:00 2001
From: gorhill <rhill@raymondhill.net>
Date: Sun, 28 Dec 2014 12:04:48 -0500
Subject: [PATCH] code review: #433

---
 src/js/pagestore.js |  8 ++++----
 src/js/ublock.js    | 14 ++++++--------
 2 files changed, 10 insertions(+), 12 deletions(-)

diff --git a/src/js/pagestore.js b/src/js/pagestore.js
index 51d67d0f1..acc158a23 100644
--- a/src/js/pagestore.js
+++ b/src/js/pagestore.js
@@ -296,7 +296,7 @@ PageStore.factory = function(tabId, pageURL) {
 
 /******************************************************************************/
 
-PageStore.prototype.bitFromRequestType = {
+PageStore.prototype.bitFromResult = {
       '':  1,
     'sb':  2,
     'sa':  4,
@@ -320,8 +320,8 @@ PageStore.prototype.init = function(tabId, pageURL) {
 
     // This is part of the filtering evaluation context
     this.requestURL = this.requestHostname = this.requestType = '';
-    this.requestHostnames = {};
 
+    this.requestHostnames = {};
     this.frames = {};
     this.netFiltering = true;
     this.netFilteringReadTime = 0;
@@ -452,9 +452,9 @@ PageStore.prototype.filterRequest = function(context) {
 
     var requestHostname = context.requestHostname;
     if ( this.requestHostnames.hasOwnProperty(requestHostname) ) {
-        this.requestHostnames[requestHostname] |= this.bitFromRequestType[result.slice(0, 2)];
+        this.requestHostnames[requestHostname] |= this.bitFromResult[result.slice(0, 2)];
     } else {
-        this.requestHostnames[requestHostname] = this.bitFromRequestType[result.slice(0, 2)];
+        this.requestHostnames[requestHostname] = this.bitFromResult[result.slice(0, 2)];
     }
 
     return result;
diff --git a/src/js/ublock.js b/src/js/ublock.js
index 82bbe9592..d1d4461fb 100644
--- a/src/js/ublock.js
+++ b/src/js/ublock.js
@@ -371,20 +371,18 @@ var matchWhitelistDirective = function(url, hostname, directive) {
         if ( df.mustBlockOrAllow() ) {
             return df.toFilterString();
         }
-    }
-
-    if ( requestType === 'sub_frame' ) {
+    } else if ( requestType === 'sub_frame' ) {
         if ( this.isFirstParty(rootHostname, requestHostname) === false ) {
             df.evaluateCellZY(rootHostname, requestHostname, '3p-frame');
             if ( df.mustBlockOrAllow() ) {
                 return df.toFilterString();
             }
         }
-    }
-
-    df.evaluateCellZY(rootHostname, requestHostname, requestType);
-    if ( df.mustBlockOrAllow() ) {
-        return df.toFilterString();
+    } else {
+        df.evaluateCellZY(rootHostname, requestHostname, requestType);
+        if ( df.mustBlockOrAllow() ) {
+            return df.toFilterString();
+        }
     }
 
     // 5. Static filtering never override dynamic filtering