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

Side by Side Diff: lib/content/snippets.js

Issue 29886700: Issue 6969 - Implement abort-on-property-read snippet (Closed) Base URL: https://hg.adblockplus.org/adblockpluscore/
Patch Set: Fixed jsdoc Created Oct. 29, 2018, 4:33 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * This file is part of Adblock Plus <https://adblockplus.org/>, 2 * This file is part of Adblock Plus <https://adblockplus.org/>,
3 * Copyright (C) 2006-present eyeo GmbH 3 * Copyright (C) 2006-present eyeo GmbH
4 * 4 *
5 * Adblock Plus is free software: you can redistribute it and/or modify 5 * Adblock Plus is free software: you can redistribute it and/or modify
6 * it under the terms of the GNU General Public License version 3 as 6 * it under the terms of the GNU General Public License version 3 as
7 * published by the Free Software Foundation. 7 * published by the Free Software Foundation.
8 * 8 *
9 * Adblock Plus is distributed in the hope that it will be useful, 9 * Adblock Plus is distributed in the hope that it will be useful,
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of
(...skipping 473 matching lines...) Expand 10 before | Expand all | Expand 10 after
484 if ((!selector || target.matches(selector)) && 484 if ((!selector || target.matches(selector)) &&
485 (!re || re.test(target.textContent))) 485 (!re || re.test(target.textContent)))
486 { 486 {
487 event.preventDefault(); 487 event.preventDefault();
488 } 488 }
489 }, 489 },
490 true); 490 true);
491 } 491 }
492 492
493 exports["prevent-inline-scripts"] = preventInlineScripts; 493 exports["prevent-inline-scripts"] = preventInlineScripts;
494
495 /**
496 * Generates a random alphanumeric ID consisting of 6 base-36 digits
497 * from the range 100000..zzzzzz (both inclusive).
498 *
499 * @returns {string} The random ID.
500 */
501 function randomId()
502 {
503 // 2176782336 is 36^6 which mean 6 chars [a-z0-9]
504 // 60466176 is 36^5
505 // 2176782336 - 60466176 = 2116316160. This ensure to always have 6
506 // chars even if Math.random() returns its minimum value 0.0
507 //
508 return Math.floor(Math.random() * 2116316160 + 60466176).toString(36);
509 }
510
511 function wrapPropertyAccess(object, property, descriptor)
512 {
513 let currentDescriptor = Object.getOwnPropertyDescriptor(object, property);
514 if (!currentDescriptor)
515 currentDescriptor = {};
516 if (typeof descriptor.get != "undefined")
517 currentDescriptor.get = descriptor.get;
518 if (typeof descriptor.set != "undefined")
519 currentDescriptor.set = descriptor.set;
520 Object.defineProperty(object, property, currentDescriptor);
hub 2018/10/29 20:02:21 Here the problem is that if `currentDescriptor.val
521 }
522
523 /**
524 * Patches a property on the window object to abort execution when the
525 * property is read.
526 * No error will be printed in the console.
527 * The idea originates from
528 * {@link https://github.com/uBlockOrigin/uAssets/blob/80b195436f8f8d78ba713237b fc268ecfc9d9d2b/filters/resources.txt#L1703 uBlock Origin}
529 *
530 * @param {string} property The name of the property.
531 */
532 function abortOnPropertyRead(property)
533 {
534 if (!property)
535 return;
536
537 let rid = randomId();
538
539 function abort()
540 {
541 throw new ReferenceError(rid);
542 }
543
544 let {onerror} = window;
545 window.onerror = (message, ...rest) =>
546 {
547 if (typeof message == "string" && message.includes(rid))
548 return true;
549 if (typeof onerror == "function")
550 return (() => {}).call.call(onerror, this, message, ...rest);
551 };
552
553 wrapPropertyAccess(window, property, {get: abort, set() {}});
554 }
555
556 exports["abort-on-property-read"] = makeInjector(abortOnPropertyRead,
557 wrapPropertyAccess,
558 randomId);
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld