Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: chrome/content/ui/sendReport.js

Issue 29329479: Issue 3222 - Get rid of RequestEntry.typeDescr property (Closed)
Patch Set: Created Oct. 29, 2015, 1:57 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/content/ui/composer.js ('k') | chrome/content/ui/sidebar.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/content/ui/sendReport.js
===================================================================
--- a/chrome/content/ui/sendReport.js
+++ b/chrome/content/ui/sendReport.js
@@ -196,28 +196,28 @@ let requestsDataSource =
this.callback = callback;
this.requestNotifier = new RequestNotifier(wnd, this.onRequestFound, this);
},
onRequestFound: function(frame, node, entry, scanComplete)
{
if (entry)
{
- let key = entry.location + " " + entry.typeDescr + " " + entry.docDomain;
+ let key = entry.location + " " + entry.type + " " + entry.docDomain;
let requestXML;
if (key in this.nodeByKey)
{
requestXML = this.nodeByKey[key];
requestXML.setAttribute("count", parseInt(requestXML.getAttribute("count"), 10) + 1);
}
else
{
requestXML = this.nodeByKey[key] = appendElement(this.requests, "request", {
location: censorURL(entry.location),
- type: entry.typeDescr,
+ type: entry.type,
docDomain: entry.docDomain,
thirdParty: entry.thirdParty,
count: 1
});
}
// Location is meaningless for element hiding hits
if (entry.filter && entry.filter instanceof ElemHideBase)
@@ -922,17 +922,17 @@ let issuesDataSource =
}
let seenFilters = Object.create(null);
for (let request of requestsDataSource.origRequests)
{
if (request.filter)
continue;
- let filter = disabledMatcher.matchesAny(request.location, RegExpFilter.typeMap[request.typeDescr], request.docDomain, request.thirdParty);
+ let filter = disabledMatcher.matchesAny(request.location, RegExpFilter.typeMap[request.type], request.docDomain, request.thirdParty);
if (filter && !(filter.text in seenFilters))
{
this.disabledFilters.push(filter);
seenFilters[filter.text] = true;
}
}
// Find disabled subscriptions with filters matching any of the requests
@@ -947,17 +947,17 @@ let issuesDataSource =
if (filter instanceof BlockingFilter)
disabledMatcher.add(filter);
for (let request of requestsDataSource.origRequests)
{
if (request.filter)
continue;
- let filter = disabledMatcher.matchesAny(request.location, RegExpFilter.typeMap[request.typeDescr], request.docDomain, request.thirdParty);
+ let filter = disabledMatcher.matchesAny(request.location, RegExpFilter.typeMap[request.type], request.docDomain, request.thirdParty);
if (filter && !(subscription.url in seenSubscriptions))
{
this.disabledSubscriptions.push(subscription);
seenSubscriptions[subscription.text] = true;
break;
}
}
}
« no previous file with comments | « chrome/content/ui/composer.js ('k') | chrome/content/ui/sidebar.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld