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

Unified Diff: third_party/WebKit/Source/devtools/front_end/coverage/CoverageModel.js

Issue 2890863003: DevTools/Coverage: filter out content scripts (Closed)
Patch Set: DevTools/Coverage: filter out content scripts Created 3 years, 7 months 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/coverage/CoverageModel.js
diff --git a/third_party/WebKit/Source/devtools/front_end/coverage/CoverageModel.js b/third_party/WebKit/Source/devtools/front_end/coverage/CoverageModel.js
index a8c37f5ff8846489f47a4dd6efa36171695ae65a..b33928f5203ac3428bacedbe25b7657f18fa8da8 100644
--- a/third_party/WebKit/Source/devtools/front_end/coverage/CoverageModel.js
+++ b/third_party/WebKit/Source/devtools/front_end/coverage/CoverageModel.js
@@ -263,6 +263,7 @@ Coverage.URLCoverageInfo = class {
this._usedSize = 0;
/** @type {!Coverage.CoverageType} */
this._type;
+ this._isContentScript = false;
}
/**
@@ -301,6 +302,13 @@ Coverage.URLCoverageInfo = class {
}
/**
+ * @return {boolean}
+ */
+ isContentScript() {
+ return this._isContentScript;
+ }
+
+ /**
* @param {!Common.ContentProvider} contentProvider
* @param {number} contentLength
* @param {number} lineOffset
@@ -311,12 +319,17 @@ Coverage.URLCoverageInfo = class {
var key = `${lineOffset}:${columnOffset}`;
var entry = this._coverageInfoByLocation.get(key);
- if (!entry) {
- entry = new Coverage.CoverageInfo(contentProvider, contentLength);
- this._coverageInfoByLocation.set(key, entry);
- this._size += contentLength;
- this._type |= entry.type();
- }
+ if (entry)
+ return entry;
+
+ entry = new Coverage.CoverageInfo(contentProvider, contentLength);
+
+ if (entry.type() === Coverage.CoverageType.JavaScript && !this._coverageInfoByLocation.size)
+ this._isContentScript = /** @type {!SDK.Script} */ (contentProvider).isContentScript();
+
+ this._coverageInfoByLocation.set(key, entry);
+ this._size += contentLength;
+ this._type |= entry.type();
return entry;
}
};

Powered by Google App Engine
This is Rietveld 408576698