Use Reflect for script.src

[jonathanKingston]

Use reflect for script.src as some scripts bind this to call getAttribute causing a stack overflow.

Follow the steps in: duckduckgo/duckduckgo-privacy-extension#1913

cookielaw is exempted from stack tracing and pandadoc are exceptions; when they're enabled you'll see the stack overflow mentioned in the issue.

When you place a debugger within the class you see cookielaw has overriden us:

Object.getOwnPropertyDescriptor(this, 'src').get.toString()
'function(){return a.getAttribute("src")||""}'

Object.getOwnPropertyDescriptor(this, 'src').set.toString()
'function(c){var b="";"string"==typeof c?b=c:c instanceof Object&&(b=c.toString());b=n(b);!b.categoryIds.length&&!b.vsCatIds.length||\n"script"!==d[0].toLowerCase()||p(a)||m(b.categoryIds,b.vsCatIds)||q(a)?!b.categoryIds.length||-1===w.indexOf(d[0].toLowerCase())||p(a)||m(b.categoryIds,b.vsCatIds)||q(a)?f("src",c):(a.removeAttribute("src"),f("data-src",c),c=a.getAttribute("class"),c||(c=B(b.categoryIds,c||"",b.vsCatIds),f("class",c))):(f("type","text/plain"),f("src",c));return!0}'

We're doing the reverse wishing to call this[name] which in turn calls getAttribute creating the infinite loop.