Remove current webext implementation

Author: rosahbruno

.circleci/config.yml

@@ -101,29 +101,6 @@ jobs:
           name: Build docs
           command: npm --prefix ./glean run build:docs
 
-  sample-webext-test:
-    docker:
-      - image: cimg/python:<< pipeline.parameters.python-version >>
-    steps:
-      - checkout
-      - install-node
-      - run:
-          name: Install JavaScript dependencies for Glean
-          command: npm --prefix ./glean install
-      - run:
-          name: Install Typescript dependencies for the TS web extension sample
-          command: |
-            npm --prefix ./samples/browser/webext install
-            npm --prefix ./samples/browser/webext run link:glean
-      - run:
-          name: Build the TS web extension sample
-          command: |
-            npm --prefix ./samples/browser/webext run build
-      - run:
-          name: Run TS web extension sample test
-          command: |
-            export PATH=.:$PATH && npm --prefix ./samples/browser/webext run test
-
   sample-node-test:
     docker:
       - image: cimg/python:<< pipeline.parameters.python-version >>
@@ -237,9 +214,6 @@ workflows:
       - test:
           requires:
             - build
-      - sample-webext-test:
-          requires:
-            - test
       - sample-node-test:
           requires:
             - test
@@ -249,7 +223,6 @@ workflows:
             - lint
             - build
             - test
-            - sample-webext-test
             - sample-node-test
           filters:
             branches:

.github/dependabot.yml

@@ -8,10 +8,6 @@ updates:
     directory: "/automation"
     schedule:
       interval: "monthly"
-  - package-ecosystem: "npm"
-    directory: "/samples/browser/webext"
-    schedule:
-      interval: "monthly"
   - package-ecosystem: "npm"
     directory: "/samples/browser/web"
     schedule:
@@ -20,7 +16,3 @@ updates:
     directory: "/samples/node"
     schedule:
       interval: "monthly"
-  - package-ecosystem: "npm"
-    directory: "/glean/tests/unit/platform/utils/webext/sample"
-    schedule:
-      interval: "monthly"

ARCHITECTURE.md

@@ -16,7 +16,7 @@ An end-to-end data collection platform developed by Mozilla and primarily target
 Glean provides multiple client SDKs for different programming languages and platforms.
 One of the aspects that guide Glean SDK development is cross-platform consistency and the Glean
 JavaScript SDK is no exception to that. It is built to work on multiple JavaScript platforms --
-websites, web extensions, Node.js and QML as of the time of writing -- and to be easily extendable
+websites, Node.js and QML as of the time of writing -- and to be easily extendable
 to other platforms as well.
 
 The Glean JavaScript SDK is the latest addition to the family of Glean SDKs. The other Glean SDKs,
@@ -40,10 +40,10 @@ storage. Each metric can have different [lifetimes](https://mozilla.github.io/gl
 and the SDK will manage its storage so that data does not remain in storage after it's lifetime is expired.
 
 The Glean SDK tries to do all of this is the least disruptive way possible to users. There are two separate
-implementations for the SDK based on the platform: async (node, web extensions) and sync (browser). The implementation
+implementations for the SDK based on the platform: async (node) and sync (browser). The implementation
 is set inside of Glean itself and is not configurable by the user.
 
-### async (Web Extensions, Node)
+### async (Node)
 
 All of the SDKs tasks are queued and executed asynchronously. The APIs exposed by the Glean SDK will only do
 the en-queuing of tasks, a quick synchronous operation. Internally, the Glean SDK will handle the
@@ -104,7 +104,7 @@ To see all the exposed entry points, check out Glean.js' `package.json` file.
 ### `entry/`
 
 The `entry/` folder contains the main entry points for the Glean.js package per platform.
-For example, when a user does `import Glean from @mozilla/glean/webext` it's the `entry/webext.ts`
+For example, when a user does `import Glean from @mozilla/glean/web` it's the `entry/web.ts`
 file that they are getting and not `core/glean.ts`.
 
 The main difference between each platform's file is that a different `Platform` implementation is
@@ -123,7 +123,6 @@ It also makes testing easier, because the exact same suite of tests can be run f
 
 The storage module varies for each platform. The storage mechanism used by each platform is as follows:
 - `web` - [`localStorage`](https://developer.mozilla.org/en-US/docs/Web/API/Window/localStorage)
-- `webext` - [`storage`](https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/storage)
 - `Node` - None, everything is stored in memory
 
 ### `plugins/`

automation/size/report.js

@@ -14,11 +14,6 @@ import {
 
 async function buildStatsMap() {
   return [
-    {
-      name: "Web Extension",
-      max: await getCustomLibSize("webext", METRIC_TYPES, PLUGINS),
-      min: await getCustomLibSize("webext")
-    },
     {
       name: "Website",
       max: await getCustomLibSize("web", METRIC_TYPES, PLUGINS),

automation/size/utils.js

@@ -34,7 +34,7 @@ export const METRIC_TYPES = [
 
 export const PLUGINS = [ "encryption" ];
 
-export const PLATFORMS = [ "web", "webext", "node" ];
+export const PLATFORMS = [ "web", "node" ];
 
 const UNITS = ["bytes", "KB", "MB", "GB", "TB", "PB", "EB", "ZB", "YB"];
 /**
@@ -81,7 +81,7 @@ export function executeCmd(cmd) {
  *
  * Note: This function will use whatever `@mozilla/glean` package is linked at its runtime.
  *
- * @param {string} platform The platform to build for, supported platforms are: `web`, `webext` and `node`.
+ * @param {string} platform The platform to build for, supported platforms are: `web`, and `node`.
  * @param {[string]} metrics An array of metric types. Supported metrics types are listed in `METRIC_TYPES`.
  *        If not provided the final bundle won't contain any additional metric types.
  * @param {[string]} plugins An array of plugins. Supported plugins are listed in `PLUGINS`.

automation/size/webpack.config.js

@@ -3,9 +3,7 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 export default (platform, entry) => {
-  const target = platform === "node" ? "node"
-  : ["webext", "web"].includes(platform) ? "web"
-  : null;
+  const target = platform === "node" ? "node" : platform === "web" ? "web" : null;
 
   return {
     target,

docs/reference/ci.md

@@ -37,7 +37,7 @@ Runs all necessary lints on the code. This includes:
 
 Runs unit and integration test on the `@mozilla/glean` library.
 
-### `sample-webext-test`, `sample-node-test`
+### `sample-node-test`
 
 Runs unit tests on the Glean.js sample apps.
 

docs/reference/compat.md

@@ -20,10 +20,6 @@ All of the listed versions are explicitly tested.
 The Opera and Brave browsers are not explicitly tested, however are unofficially supported
 as Chromium based browsers.
 
-## Web Extensions
-
-> TODO: see [Bug 1752549](https://bugzilla.mozilla.org/show_bug.cgi?id=1752549)
-
 ## Node.js
 
 > TODO: see [Bug 1752548](https://bugzilla.mozilla.org/show_bug.cgi?id=1752548)

glean/package.json

@@ -13,14 +13,10 @@
     "./uploader": "./dist/core/upload/uploader.js",
     "./testing": "./dist/core/testing/index.js",
     "./node": "./dist/entry/node.js",
-    "./webext": "./dist/entry/webext.js",
     "./web": "./dist/entry/web.js"
   },
   "typesVersions": {
     "*": {
-      "webext": [
-        "./dist/types/entry/webext.d.ts"
-      ],
       "node": [
         "./dist/types/entry/node.d.ts"
       ],
@@ -62,8 +58,7 @@
     "test:unit": "run-s test:unit:core test:unit:platform test:unit:plugins",
     "test:unit:core": "npm run test:base -- \"tests/unit/core/**/*.spec.ts\" --recursive",
     "test:unit:plugins": "npm run test:base -- \"tests/unit/plugins/**/*.spec.ts\" --recursive",
-    "test:unit:platform": "npm run test:base -- \"tests/unit/platform/**/*.spec.ts\" --recursive --timeout 0 --exclude tests/unit/platform/utils/webext/sample/node_modules/**/*.spec.ts",
-    "pretest:unit:platform": "cd tests/unit/platform/utils/webext/sample/ && npm install && npm run build:xpi",
+    "test:unit:platform": "npm run test:base -- \"tests/unit/platform/**/*.spec.ts\" --recursive --timeout 0",
     "test:base": "node --experimental-modules --experimental-specifier-resolution=node --loader=ts-node/esm node_modules/mocha/lib/cli/cli.js",
     "lint": "run-s lint:eslint lint:circular-deps lint:glinter",
     "lint:eslint": "eslint . --ext .ts,.js,.json --max-warnings=0",