Opened 3 years ago

Last modified 3 years ago

#6020 closed defect

Unnecessary code being included in webpack bundles — at Version 3

Reported by: kzar Assignee: kzar
Priority: P2 Milestone:
Module: Automation Keywords:
Cc: trev, sebastian, tlucas, sylvaing@… Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Ross Verified working: yes
Review URL(s):

https://codereview.adblockplus.org/29602573/

Description (last modified by trev)

Environment

adblockpluschrome ecdd8caaf81a

How to reproduce

  1. Open the Chrome developer tools for the background page
  2. Click the Sources pane and expand the webpack:// section

Observed behaviour

The unused (webpack)/buildin/global.js is listed.

Expected behaviour

That file should not be listed, since we don't use it!

Notes

The code in question uses eval which is flagged by Mozilla's add-on linter. It is never being used and wouldn't work anyway because of CSP.

Change History (3)

comment:1 Changed 3 years ago by kzar

  • Cc awagner@… kewisch@… removed
  • Summary changed from Unnesissary code being included in webpack bundles to Unnecessary code being included in webpack bundles

comment:2 Changed 3 years ago by tlucas

  • Ready set

comment:3 Changed 3 years ago by trev

  • Description modified (diff)
Note: See TracTickets for help on using tickets.