Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(682)

Unified Diff: third_party/WebKit/Source/devtools/front_end/network/FilterSuggestionBuilder.js

Issue 2562193002: Add filtering by priority in the Network filter area. (Closed)
Patch Set: Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/devtools/front_end/network/FilterSuggestionBuilder.js
diff --git a/third_party/WebKit/Source/devtools/front_end/network/FilterSuggestionBuilder.js b/third_party/WebKit/Source/devtools/front_end/network/FilterSuggestionBuilder.js
index a3ea2f88fbe51b00dfcfab29793ec9a8047c696e..ed72af990827c9a59804832854dcc2d303ce087e 100644
--- a/third_party/WebKit/Source/devtools/front_end/network/FilterSuggestionBuilder.js
+++ b/third_party/WebKit/Source/devtools/front_end/network/FilterSuggestionBuilder.js
@@ -80,10 +80,19 @@ Network.FilterSuggestionBuilder = class {
*/
_values(key) {
var result = this._valueLists[key];
+ var priorityMap;
+
if (!result)
return [];
- result.sort();
+ if (key === 'priority') {
allada 2016/12/12 17:29:57 Instead of storing the symbols lets just make sure
Oleksii Kadurin 2016/12/12 21:13:15 It could be optimized but I don't feel it could be
allada 2016/12/12 23:51:08 Lets first address the two way map from the other
Oleksii Kadurin 2016/12/13 19:43:37 Sorry, I still don't get the idea here :( I will j
+ priorityMap = SDK.NetworkRequest.GetSymbolicToNumericPriority();
+ result = result.sort((a, b) => priorityMap.get(b) - priorityMap.get(a));
+ result = result.map(priority => Components.uiLabelForPriority(priority));
+ } else {
+ result.sort();
+ }
+
return result;
}

Powered by Google App Engine
This is Rietveld 408576698