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

Unified Diff: README.md

Issue 29458601: Issue 5315 - Add support for Microsoft Edge (Closed)
Patch Set: Created June 7, 2017, 12:41 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | ext/background.js » ('j') | ext/common.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: README.md
===================================================================
--- a/README.md
+++ b/README.md
@@ -1,9 +1,9 @@
-Adblock Plus for Chrome and Opera
-=================================
+Adblock Plus for Chrome, Opera and Edge
Sebastian Noack 2017/06/07 14:42:14 I wonder whether we should spell out "Micorosft Ed
kzar 2017/06/08 10:50:27 FWIW I agree.
Oleksandr 2017/06/14 03:41:01 Done.
+=======================================
This repository contains the platform-specific Adblock Plus source code for
-Chrome and Opera. It can be used to build Adblock Plus for these platforms,
-generic Adblock Plus code will be extracted from other repositories
+Chrome, Opera and Edge. It can be used to build Adblock Plus for these
+platforms, generic Adblock Plus code will be extracted from other repositories
automatically (see _dependencies_ file).
Building
@@ -19,26 +19,34 @@
### Building the extension
-Run the following command in the project directory:
+Run one of the following commands in the project directory, depending on your
kzar 2017/06/08 10:50:27 Note: If the README changes for Firefox webext[1]
+target platform:
./build.py -t chrome build -k adblockpluschrome.pem
+ ./build.py -t edge build
Sebastian Noack 2017/06/07 14:42:14 Perhaps we should add a note below, that the gener
Oleksandr 2017/06/14 03:41:01 Done.
This will create a build with a name in the form
-_adblockpluschrome-1.2.3.nnnn.crx_
+_adblockpluschrome-1.2.3.nnnn.crx_ or _adblockplusedge-1.2.3.nnnn.appx_.
Note that you don't need an existing signing key for Chrome, a new key
will be created automatically if the file doesn't exist.
### Development environment
To simplify the process of testing your changes you can create an unpacked
-development environment. For that run the following command:
+development environment. For that run one of the following commands:
./build.py -t chrome devenv
+ ./build.py -t edge devenv
-This will create a _devenv.chrome_ directory in the repository. In Chrome you
-should load the directory as an unpacked extension. After making changes to the
-source code re-run the command to update the development environment, the
-extension should reload automatically after a few seconds.
+This will create a _devenv.platform_ directory in the repository. In Chrome you
kzar 2017/06/08 10:50:27 Nit: Blank line before "In Chrome"? I figure the f
+should load _devenv.chrome_ as an unpacked extension directory. After making
+changes to the source code re-run the command to update the development
+environment, the extension should reload automatically after a few seconds.
+
+For Edge you should load _devenv.edge/Extension_ as an unpacked extension
+directory. Edge build does not automatically detect changes, so after
+rebuilding the extension you should manually force reloading it in Edge by
+hitting the _Reload Extension_ button.
Running the unit tests
----------------------
« no previous file with comments | « no previous file | ext/background.js » ('j') | ext/common.js » ('J')

Powered by Google App Engine
This is Rietveld