Skip to content
This repository was archived by the owner on Aug 28, 2023. It is now read-only.

Add support for browser extensions code #1

Open
burivuhster wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add support for browser extensions code #1

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
2e1e0a4
Add chrome, chrome-extension and widget protocols to the stack-trace …
burivuhster Jun 18, 2013
9c0b478
Remove domain check before trying to load sources
burivuhster Jun 19, 2013
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 3 additions & 8 deletions tracekit.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -347,12 +347,7 @@ TraceKit.computeStackTrace = (function computeStackTraceWrapper() {
*/
function getSource(url) {
if (!_has(sourceCache, url)) {
// URL needs to be able to fetched within the acceptable domain. Otherwise,
// cross-domain errors will be triggered.
var source = '';
if (url.indexOf(document.domain) !== -1) {
source = loadSource(url);
}
var source = loadSource(url);
sourceCache[url] = source ? source.split('\n') : [];
}

Expand Down Expand Up @@ -618,8 +613,8 @@ TraceKit.computeStackTrace = (function computeStackTraceWrapper() {
return null;
}

var chrome = /^\s*at (?:((?:\[object object\])?\S+) )?\(?((?:file|http|https):.*?):(\d+)(?::(\d+))?\)?\s*$/i,
gecko = /^\s*(\S*)(?:\((.*?)\))?@((?:file|http|https).*?):(\d+)(?::(\d+))?\s*$/i,
var chrome = /^\s*at (?:((?:\[object object\])?\S+) )?\(?((?:file|http|https|chrome|widget|chrome-extension):.*?):(\d+)(?::(\d+))?\)?\s*$/i,
gecko = /^\s*(\S*)(?:\((.*?)\))?@((?:file|http|https|chrome|widget|chrome-extension).*?):(\d+)(?::(\d+))?\s*$/i,
lines = ex.stack.split('\n'),
stack = [],
parts,
Expand Down