Bug 1273183 - Install a temporary add-on via remote debugging. r=jryans
authorKumar McMillan <kumar.mcmillan@gmail.com>
Fri, 20 May 2016 11:36:04 -0500
changeset 337691 4e09a535b6ae4025b77c3808a5e451a9ef787074
parent 337690 340aab956ac7953d1953dfb17ba6cb19c9254781
child 337692 67e6237c889546118da03cbf3512274f9db05562
push id6249
push userjlund@mozilla.com
push dateMon, 01 Aug 2016 13:59:36 +0000
treeherdermozilla-beta@bad9d4f5bf7e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjryans
bugs1273183
milestone49.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1273183 - Install a temporary add-on via remote debugging. r=jryans MozReview-Commit-ID: FoEZxDjldIA
devtools/server/actors/addons.js
devtools/server/actors/moz.build
devtools/server/docs/protocol.js.md
devtools/server/main.js
devtools/server/tests/unit/head_dbg.js
devtools/server/tests/unit/test_addons_actor.js
devtools/server/tests/unit/test_addons_actor/web-extension/manifest.json
devtools/server/tests/unit/xpcshell.ini
devtools/shared/fronts/addons.js
devtools/shared/fronts/moz.build
devtools/shared/specs/addons.js
devtools/shared/specs/moz.build
new file mode 100644
--- /dev/null
+++ b/devtools/server/actors/addons.js
@@ -0,0 +1,41 @@
+/* This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
+
+"use strict";
+
+const {AddonManager} = require("resource://gre/modules/AddonManager.jsm");
+const protocol = require("devtools/shared/protocol");
+const {FileUtils} = require("resource://gre/modules/FileUtils.jsm");
+const {Task} = require("devtools/shared/task");
+const {addonsSpec} = require("devtools/shared/specs/addons");
+
+const AddonsActor = protocol.ActorClassWithSpec(addonsSpec, {
+
+  initialize: function (conn) {
+    protocol.Actor.prototype.initialize.call(this, conn);
+  },
+
+  installTemporaryAddon: Task.async(function* (addonPath) {
+    let addonFile;
+    let addon;
+    try {
+      addonFile = new FileUtils.File(addonPath);
+      addon = yield AddonManager.installTemporaryAddon(addonFile);
+    } catch (error) {
+      throw new Error(`Could not install add-on at '${addonPath}': ${error}`);
+    }
+
+    // TODO: once the add-on actor has been refactored to use
+    // protocol.js, we could return it directly.
+    // return new BrowserAddonActor(this.conn, addon);
+
+    // Return a pseudo add-on object that a calling client can work
+    // with. Provide a flag that the client can use to detect when it
+    // gets upgraded to a real actor object.
+    return { id: addon.id, actor: false };
+
+  }),
+});
+
+exports.AddonsActor = AddonsActor;
--- a/devtools/server/actors/moz.build
+++ b/devtools/server/actors/moz.build
@@ -7,16 +7,17 @@
 DIRS += [
     'highlighters',
     'utils',
 ]
 
 DevToolsModules(
     'actor-registry.js',
     'addon.js',
+    'addons.js',
     'animation.js',
     'breakpoint.js',
     'call-watcher.js',
     'canvas.js',
     'child-process.js',
     'childtab.js',
     'chrome.js',
     'common.js',
--- a/devtools/server/docs/protocol.js.md
+++ b/devtools/server/docs/protocol.js.md
@@ -1,69 +1,87 @@
 Writing an Actor
 ================
 
 A Simple Hello World
 --------------------
 
 Here's a simple Hello World actor.  It is a global actor (not associated with a given browser tab).
-
-    let protocol = require("devtools/shared/protocol");
-    let {method, Arg, Option, RetVal} = protocol;
+It has two parts: a spec and an implementation. The spec would go somewhere like
+`devtools/shared/specs/hello-world.js` and would look like:
 
-    // This will be called by the framework when you call DebuggerServer.
-    // registerModule(), and adds the actor as a 'helloActor' property
-    // on the root actor.
-    exports.register = function(handle) {
-        handle.addGlobalActor(HelloActor, "helloActor");
-    }
+    const {Arg, RetVal, generateActorSpec} = require("devtools/shared/protocol");
 
-    // This will be called by the framework during shutdown/unload.
-    exports.unregister = function(handle) {
-        handle.removeGlobalActor(HelloActor); // This is optional, all registered actors will be removed automatically
-    }
-
-    // Create the hello actor.  This uses addon-sdk's heritage module
-    // behind the scenes in case you want to understand the class stuff
-    // a bit better.
+    const helloWorldSpec = generateActorSpec({
+      typeName: "helloWorld", // I'll explain types later, I promise.
 
-    let HelloActor = protocol.ActorClass({
-        typeName: "helloWorld", // I'll explain types later, I promise.
-        initialize: function(conn) {
-            protocol.Actor.prototype.initialize.call(this, conn); // This is the worst part of heritage.
-        },
-
-        sayHello: method(function() {
-           return "hello";
-        }, {
+      methods: {
+        sayHello: {
           // The request packet template.  There are no arguments, so
           // it is empty.  The framework will add the "type" and "to"
           // request properties.
           request: {},
 
           // The response packet template.  The return value of the function
           // will be plugged in where the RetVal() appears in the template.
           response: {
             greeting: RetVal("string") // "string" is the return value type.
           }
-        }
+        },
+      },
     });
 
-This actor now supports a `sayHello` request.  A request/reply will look like this:
+    // Expose the spec so it can be imported by the implementation.
+    exports.helloWorldSpec = helloWorldSpec;
+
+The actor implementation would go somewhere like
+`devtools/server/actors/hello-world.js` and would look like:
+
+    const protocol = require("devtools/shared/protocol");
+    const {helloWorldSpec} = require("devtools/shared/specs/hello-world");
+
+    const HelloActor = protocol.ActorClassWithSpec(helloWorldSpec, {
+      initialize: function (conn) {
+        protocol.Actor.prototype.initialize.call(this, conn); // This is the worst part of heritage.
+      },
+
+      sayHello: function () {
+       return "hello";
+      },
+    });
+
+    // You also need to export the actor class in your module for discovery.
+    exports.HelloActor = HelloActor;
+
+To activate your actor, register it in the `addBrowserActors` method in `server/main.js`.
+The registration code would look something like this:
+
+    this.registerModule("devtools/server/actors/hello-world", {
+      prefix: "hello",
+      constructor: "HelloActor",
+      type: { global: true }
+    });
+
+Your spec allows the actor to support a `sayHello` request.
+A request/reply will look like this:
 
     -> { to: <actorID>, type: "sayHello" }
     <- { from: <actorID>, greeting: "hello" }
 
-Now we can create a client side object.  We call these *front* objects.
+Now we can create a client side object.  We call these *front* objects and
+they typically go in `devtools/shared/fronts/`.
 
 Here's the front for the HelloActor:
 
-    let HelloFront = protocol.FrontClass(HelloActor, {
-      initialize: function(client, form) {
+    const HelloFront = protocol.FrontClassWithSpec(helloWorldSpec, {
+      initialize: function (client, form) {
         protocol.Front.prototype.initialize.call(this, client, form);
+        // This call may not be required but it's a good idea. It will
+        // guarantee that your instance is managed in the pool.
+        this.manage(this);
       }
     });
 
 Note that there is no `sayHello` method.  The FrontClass will generate a method on the Front object that matches the method declaration in the Actor class.
 
 The generated methods will return a Promise.  That promise will resolve to the RetVal of the actor method.
 
 So if we have a reference to a HelloFront object, we can issue a `sayHello` request:
@@ -81,24 +99,31 @@ Manually - If you're using a DebuggerCli
 
     let hello = HelloFront(this.client, { actor: <hello actorID> });
 
 Magically - Once you have an initial reference to a protocol.js object, it can return other protocol.js objects and fronts will automatically be created.
 
 Arguments
 ---------
 
-`sayHello` has no arguments, so let's add a method that does take arguments:
+`sayHello` has no arguments, so let's add a method that does take arguments.
+Here's an adjustment to the spec:
 
-    echo: method(function(str) {
-        return str + "... " + str + "...";
-    }, {
-      request: { echo: Arg(0, "string") },
-      response: { echoed: RetVal("string") }
-    })
+    methods: {
+      echo: {
+        request: { echo: Arg(0, "string") },
+        response: { echoed: RetVal("string") }
+      }
+    }
+
+Here's an adjustment to the implementation:
+
+    echo: function (str) {
+      return str + "... " + str + "...";
+    }
 
 This tells the library to place the 0th argument, which should be a string, in the `echo` property of the request packet.
 
 
 This will generate a request handler whose request and response packets look like this:
 
     { to: <actorID>, type: "echo", echo: <str> }
     { from: <actorID>, echoed: <str> }
@@ -107,24 +132,30 @@ The client usage should be predictable:
 
     echo.echo("hello").then(str => { assert(str === "hello... hello...") })
 
 The library tries hard to make using fronts feel like natural javascript (or as natural as you believe promises are, I guess).  When building the response it will put the return value of the function where RetVal() is specified in the response template, and on the client side it will use the value in that position when resolving the promise.
 
 Returning JSON
 --------------
 
-Maybe your response is an object:
+Maybe your response is an object. Here's an example of a spec:
 
-    addOneTwice: method(function(a, b) {
-        return { a: a + 1, b: b + 1 };
-    }, {
+    methods: {
+      addOneTwice: {
         request: { a: Arg(0, "number"), b: Arg(1, "number") },
         response: { ret: RetVal("json") }
-    });
+      }
+    }
+
+Here's an example implementation:
+
+    addOneTwice: function (a, b) {
+      return { a: a + 1, b: b + 1 };
+    }
 
 This will generate a response packet that looks like:
 
     { from: <actorID>, ret: { a: <number>, b: <number> } }
 
 That's probably unnecessary nesting (if you're sure you won't be returning an object with 'from' as a key!), so you can just replace `response` with:
 
     response: RetVal("json")
@@ -135,33 +166,39 @@ and now your packet will look like:
 
 Types and Marshalling
 ---------------------
 
 Things have been pretty simple up to this point - all the arguments we've passed in have been javascript primitives.  But for some types (most importantly Actor types, which I'll get to eventually), we can't just copy them into a JSON packet and expect it to work, we need to marshal things ourselves.
 
 Again, the protocol lib tries hard to provide a natural API to actors and clients, and sometime that natural API might involve object APIs. I'm going to use a wickedly contrived example, bear with me.  Let's say I have a small object that contains a number and has a few methods associated with it:
 
-    let Incrementor = function(i) {
+    let Incrementor = function (i) {
       this.value = value;
     }
     Incrementor.prototype = {
-        increment: function() { this.value++ },
-        decrement: function() { this.value-- }
+        increment: function () { this.value++ },
+        decrement: function () { this.value-- }
     };
 
 
 and I want to return it from a backend function:
 
-    getIncrementor: method(function(i) {
-        return new Incrementor(i)
-    }, {
+    // spec:
+    methods: {
+      getIncrementor: {
         request: { number: Arg(0, "number") },
         response: { value: RetVal("incrementor") } // We'll define "incrementor" below.
-    });
+      }
+    }
+
+    // implementation:
+    getIncrementor: function (i) {
+      return new Incrementor(i)
+    }
 
 I want that response to look like `{ from: <actorID>, value: <number> }`, but the client side needs to know to return an Incrementor, not a primitive number.  So let's tell the protocol lib about Incrementors:
 
     protocol.types.addType("incrementor", {
         // When writing to a protocol packet, just send the value
         write: (v) => v.value,
 
         // When reading from a protocol packet, wrap with an Incrementor
@@ -173,64 +210,83 @@ And now our client can use the API as ex
 
     front.getIncrementor(5).then(incrementor => {
         incrementor.increment();
         assert(incrementor.value === 6);
     });
 
 You can do the same thing with arguments:
 
-    passIncrementor: method(function(inc) {
-        w.increment();
-        assert(incrementor.value === 6);
-    }, {
+    // spec:
+    methods: {
+      passIncrementor: {
         request: { Arg(0, "incrementor") },
-    });
+      }
+    }
+
+    // implementation:
+    passIncrementor: function (inc) {
+      w.increment();
+      assert(incrementor.value === 6);
+    }
 
     front.passIncrementor(new Incrementor(5));
 
 The library provides primitiive `boolean`, `number`, `string`, and `json` types.
 
 Moving right along, let's say you want to pass/return an array of Incrementors.  You can just prepend `array:` to the type name:
 
-    incrementAll: method(function(incrementors) {
-        incrementors.forEach(incrementor => {
-          incrementor.increment();
-        }
-        return incrementors;
-    }, {
+    // spec:
+    methods: {
+      incrementAll: {
         request: { incrementors: Arg(0, "array:incrementor") },
         response: { incrementors: RetVal("array:incrementor") }
-    })
+      }
+    }
+
+    // implementation:
+    incrementAll: function (incrementors) {
+      incrementors.forEach(incrementor => {
+        incrementor.increment();
+      }
+      return incrementors;
+    }
 
 You can use an iterator in place of an array as an argument or return value, and the library will handle the conversion automatically.
 
 Or maybe you want to return a dictionary where one item is a incrementor.  To do this you need to tell the type system which members of the dictionary need custom marshallers:
 
     protocol.types.addDictType("contrivedObject", {
-        incrementor: "incrementor",
-        incrementorArray: "array:incrementor"
+      incrementor: "incrementor",
+      incrementorArray: "array:incrementor"
     });
 
-    reallyContrivedExample: method(function() {
-        return {
-            /* a and b are primitives and so don't need to be called out specifically in addDictType */
-            a: "hello", b: "world",
-            incrementor: new Incrementor(1), incrementorArray: [new Incrementor(2), new Incrementor(3)]
-        }
-    }, {
+    // spec:
+    methods: {
+      reallyContrivedExample: {
         response: RetVal("contrivedObject")
-    });
+      }
+    }
+
+    // implementations:
+    reallyContrivedExample: function () {
+      return {
+        /* a and b are primitives and so don't need to be called out specifically in addDictType */
+        a: "hello", b: "world",
+        incrementor: new Incrementor(1),
+        incrementorArray: [new Incrementor(2), new Incrementor(3)]
+      }
+    }
 
     front.reallyContrivedExample().then(obj => {
-        assert(obj.a == "hello");
-        assert(obj.b == "world");
-        assert(incrementor.i == 1);
-        assert(incrementorArray[0].i == 2);
-        assert(incrementorArray[1].i == 3);
+      assert(obj.a == "hello");
+      assert(obj.b == "world");
+      assert(incrementor.i == 1);
+      assert(incrementorArray[0].i == 2);
+      assert(incrementorArray[1].i == 3);
     });
 
 Nullables
 ---------
 
 If an argument, return value, or dict property can be null/undefined, you can prepend `nullable:` to the type name:
 
     "nullable:incrementor",  // Can be null/undefined or an incrementor
@@ -238,222 +294,256 @@ If an argument, return value, or dict pr
     "nullable:array:incrementor" // Either null/undefined or an array of incrementors without holes.
 
 
 Actors
 ------
 
 Probably the most common objects that need custom martialing are actors themselves.  These are more interesting than the Incrementor object, but by default they're somewhat easy to work with.  Let's add a ChildActor implementation that will be returned by the HelloActor (which is rapidly becoming the OverwhelminglyComplexActor):
 
-    let ChildActor = protocol.ActorClass({
-        actorType: "childActor",
-        initialize: function(conn, id) {
-            protocol.Actor.prototype.initialize.call(this, conn);
-            this.greeting = "hello from " + id;
-        },
-        getGreeting: method(function() {
-            return this.greeting;
-        }, {
-            response: { greeting: RetVal("string") },
+    // spec:
+    const childActorSpec = generateActorSpec({
+      actorType: "childActor",
+      methods: {
+        getGreeting: {
+          response: { greeting: RetVal("string") },
         }
+      }
     });
 
-    let ChildFront = protocol.FrontClass(ChildActor, {
-        initialize: function(client, form) {
-            protocol.Front.prototype.initialize.call(this, client, form);
-        },
+    // implementation:
+    const ChildActor = protocol.ActorClassWithSpec(childActorSpec, {
+      initialize: function (conn, id) {
+        protocol.Actor.prototype.initialize.call(this, conn);
+        this.greeting = "hello from " + id;
+      },
+      getGreeting: function () {
+        return this.greeting;
+      },
+    });
+
+    exports.ChildActor = ChildActor;
+
+    const ChildFront = protocol.FrontClassWithSpec(childActorSpec, {
+      initialize: function (client, form) {
+        protocol.Front.prototype.initialize.call(this, client, form);
+      },
     });
 
 The library will register a marshaller for the actor type itself, using typeName as its tag.
 
 So we can now add the following code to HelloActor:
 
-    getChild: method(function(id) {
-        return ChildActor(this.conn, id);
-    }, {
+    // spec:
+    methods: {
+      getChild: {
         request: { id: Arg(0, "string") },
         response: { child: RetVal("childActor") }
-    });
+      }
+    }
+
+    // implementation:
+    getChild: function (id) {
+      return ChildActor(this.conn, id);
+    }
 
     front.getChild("child1").then(childFront => {
-        return childFront.getGreeting();
+      return childFront.getGreeting();
     }).then(greeting => {
-        assert(id === "hello from child1");
+      assert(id === "hello from child1");
     });
 
 The conversation will look like this:
 
     { to: <actorID>, type: "getChild", id: "child1" }
     { from: <actorID>, child: { actor: <childActorID> }}
     { to: <childActorID>, type: "getGreeting" }
     { from: <childActorID>, greeting: "hello from child1" }
 
 But the ID is the only interesting part of this made-up example.  You're never going to want a reference to a ChildActor without checking its ID.  Making an extra request just to get that id is wasteful.  You really want the first response to look like `{ from: <actorID>, child: { actor: <childActorID>, greeting: "hello from child1" } }`
 
 You can customize the marshalling of an actor by providing a `form` method in the `ChildActor` class:
 
-    form: function() {
+    form: function () {
         return {
             actor: this.actorID,
             greeting: this.greeting
         }
     },
 
 And you can demarshal in the `ChildFront` class by implementing a matching `form` method:
 
-    form: function(form) {
+    form: function (form) {
         this.actorID = form.actor;
         this.greeting = form.greeting;
     }
 
 Now you can use the id immediately:
 
     front.getChild("child1").then(child => { assert(child.greeting === "child1) });
 
 You may come across a situation where you want to customize the output of a `form` method depending on the operation being performed.  For example, imagine that ChildActor is a bit more complex, with a, b, c, and d members:
 
     ChildActor:
-        form: function() {
+        form: function () {
             return {
                 actor: this.actorID,
                 greeting: this.greeting,
                 a: this.a,
                 b: this.b,
                 c: this.c,
                 d: this.d
             }
         }
     ChildFront:
-        form: function(form) {
+        form: function (form) {
             this.actorID = form.actorID;
             this.id = form.id;
             this.a = form.a;
             this.b = form.b;
             this.c = form.c;
             this.d = form.d;
         }
 
 And imagine you want to change 'c' and return the object:
 
     // Oops!  If a type is going to return references to itself or any other
     // type that isn't fully registered yet, you need to predeclare the type.
     types.addActorType("childActor");
 
     ...
 
-    changeC: method(function(newC) {
-        c = newC;
-        return this;
-    }, {
+    // spec:
+    methods: {
+      changeC: {
         request: { newC: Arg(0) },
         response: { self: RetVal("childActor") }
-    });
+      }
+    }
+
+    // implementation:
+    changeC: function (newC) {
+      c = newC;
+      return this;
+    }
 
     ...
 
     childFront.changeC('hello').then(ret => { assert(ret === childFront); assert(childFront.c === "hello") });
 
 Now our response will look like:
 
     { from: <childActorID>, self: { actor: <childActorID>, greeting: <id>, a: <a>, b: <b>, c: "hello", d: <d> }
 
 But that's wasteful.  Only c changed.  So we can provide a *detail* to the type using `#`:
 
     response: { self: RetVal("childActor#changec") }
 
 and update our form methods to make use of that data:
 
-    ChildActor:
-    form: function(detail) {
+    // In ChildActor:
+    form: function (detail) {
         if (detail === "changec") {
             return { actor: this.actorID, c: this.c }
         }
-        ... // the rest of the form method stays the same.
+        // ... the rest of the form method stays the same.
     }
 
-    ChildFront:
-    form: function(form, detail) {
+    // In ChildFront:
+    form: function (form, detail) {
         if (detail === "changec") {
             this.actorID = form.actor;
             this.c = form.c;
             return;
         }
-        ... // the rest of the form method stays the same.
+        // ... the rest of the form method stays the same.
     }
 
 Now the packet looks like a much more reasonable `{ from: <childActorID>, self: { actor: <childActorID>, c: "hello" } }`
 
 Lifetimes
 ---------
 
 No, I don't want to talk about lifetimes quite yet.
 
 Events
 ------
 
 Your actor has great news!
 
-Actors are subclasses of jetpack `EventTarget`, so you can just emit:
-
-    let event = require("sdk/event/core");
-
-    giveGoodNews: method(function(news) {
-        event.emit(this, "good-news", news);
-    }, {
-        request: { news: Arg(0) }
-    });
-
-... but nobody will really care, because that's not going over the protocol.  But you can describe the packet in an `events` member, the same way you would specify a request:
+Actors are subclasses of jetpack `EventTarget`, so you can just emit events.
+Here's how you'd set it up in a spec:
 
     events: {
-        "good-news": {
-            type: "goodNews", // event target naming and packet naming are at odds, and we want both to be natural!
-            news: Arg(0)
-        }
+      "good-news": {
+        type: "goodNews", // event target naming and packet naming are at odds, and we want both to be natural!
+        news: Arg(0)
+      }
+    }
+
+    methods: {
+      giveGoodNews: {
+        request: { news: Arg(0) }
+      }
     }
 
-And now you can listen to events on a front:
+Here's how the implementation would look:
+
+    const event = require("sdk/event/core");
+
+    // In your protocol.ActorClassWithSpec definition:
+    giveGoodNews: function (news) {
+      event.emit(this, "good-news", news);
+    }
+
+Now you can listen to events on a front:
 
     front.on("good-news", news => {
-        console.log("Got good news: " + news + "\n");
+      console.log(`Got some good news: ${news}\n`);
     });
     front.giveGoodNews().then(() => { console.log("request returned.") });
 
 You might want to update your front's state when an event is fired, before emitting it against the front.  You can use `preEvent` in the front definition for that:
 
-    countGoodNews: protocol.preEvent("good-news", function(news) {
+    countGoodNews: protocol.preEvent("good-news", function (news) {
         this.amountOfGoodNews++;
     });
 
 You can have events wait until an asynchronous action completes before firing by returning a promise. If you have multiple preEvents defined for a specific event, and atleast one fires asynchronously, then all preEvents most resolve before all events are fired.
 
-    countGoodNews: protocol.preEvent("good-news", function(news) {
+    countGoodNews: protocol.preEvent("good-news", function (news) {
         return this.updateGoodNews().then(() => this.amountOfGoodNews++);
     });
 
 On a somewhat related note, not every method needs to be request/response.  Just like an actor can emit a one-way event, a method can be marked as a one-way request.  Maybe we don't care about giveGoodNews returning anything:
 
-    giveGoodNews: method(function(news) {
-        emit(this, "good-news", news);
-    }, {
+    // spec:
+    methods: {
+      giveGoodNews: {
         request: { news: Arg(0, "string") },
         oneway: true
-    });
+      }
+    }
+
+    // implementation:
+    giveGoodNews: function (news) {
+      emit(this, "good-news", news);
+    }
 
 Lifetimes
 ---------
 
 No, let's talk about custom front methods instead.
 
 Custom Front Methods
 --------------------
 
 You might have some bookkeeping to do before issuing a request.  Let's say you're calling `echo`, but you want to count the number of times you issue that request.  Just use the `custom` tag in your front implementation:
 
-    echo: custom(function(str) {
+    echo: custom(function (str) {
         this.numEchos++;
         return this._echo(str);
     }, {
         impl: "_echo"
     })
 
 This puts the generated implementation in `_echo` instead of `echo`, letting you implement `echo` as needed.  If you leave out the `impl`, it just won't generate the implementation at all.  You're on your own.
 
@@ -463,93 +553,120 @@ Lifetimes
 OK, I can't think of any more ways to put this off.  The remote debugging protocol has the concept of a *parent* for each actor.  This is to make distributed memory management a bit easier.  Basically, any descendents of an actor will be destroyed if the actor is destroyed.
 
 Other than that, the basic protocol makes no guarantees about lifetime.  Each interface defined in the protocol will need to discuss and document its approach to lifetime management (although there are a few common patterns).
 
 The protocol library will maintain the child/parent relationships for you, but it needs some help deciding what the child/parent relationships are.
 
 The default parent of an object is the first object that returns it after it is created.  So to revisit our earlier HelloActor `getChild` implementation:
 
-    getChild: method(function(id) {
-        return new ChildActor(this.conn, id);
-    }, {
+    // spec:
+    methods: {
+      getChild: {
         request: { id: Arg(0) },
         response: { child: RetVal("childActor") }
-    });
+      }
+    }
+
+    // implementation:
+    getChild: function (id) {
+      return new ChildActor(this.conn, id);
+    }
 
 The ChildActor's parent is the HelloActor, because it's the one that created it.
 
 You can customize this behavior in two ways.  The first is by defining a `marshallPool` property in your actor.  Imagine a new ChildActor method:
 
-    getSibling: method(function(id) {
-        return new ChildActor(this.conn, id);
-    }, {
+    // spec:
+    methods: {
+      getSibling: {
         request: { id: Arg(0) },
         response: { child: RetVal("childActor") }
-    });
+      }
+    }
+
+    // implementation:
+    getSibling: function (id) {
+      return new ChildActor(this.conn, id);
+    }
 
 This creates a new child actor owned by the current child actor.  But in this example we want all actors created by the child to be owned by the HelloActor.  So we can define a `defaultParent` property that makes use of the `parent` proeprty provided by the Actor class:
 
     get marshallPool() { return this.parent }
 
 The front needs to provide a matching `defaultParent` property that returns an owning front, to make sure the client and server lifetimes stay synced.
 
 For more complex situations, you can define your own lifetime properties.  Take this new pair of HelloActor methods:
 
     // When the "temp" lifetime is specified, look for the _temporaryParent attribute as the owner.
     types.addLifetime("temp", "_temporaryParent");
 
-    getTemporaryChild: method(function(id) {
-        if (!this._temporaryParent) {
-            // Create an actor to serve as the parent for all temporary children and explicitly
-            // add it as a child of this actor.
-            this._temporaryParent = this.manage(new Actor(this.conn));
-        }
-        return new ChildActor(this.conn, id);
-    }, {
+    // spec:
+    methods: {
+      getTemporaryChild: {
         request: { id: Arg(0) },
         response: {
-            child: RetVal("temp:childActor") // use the lifetime name here to specify the expected lifetime.
+          child: RetVal("temp:childActor") // use the lifetime name here to specify the expected lifetime.
         }
-    });
+      },
+      clearTemporaryChildren: {
+        oneway: true
+      }
+    }
 
-    clearTemporaryChildren: method(function(id) {
-        if (this._temporaryParent) {
-            this._temporaryParent.destroy();
-            delete this._temporaryParent;
-        }
-    });
+    // implementation:
+    getTemporaryChild: function (id) {
+      if (!this._temporaryParent) {
+        // Create an actor to serve as the parent for all temporary children and explicitly
+        // add it as a child of this actor.
+        this._temporaryParent = this.manage(new Actor(this.conn));
+      }
+      return new ChildActor(this.conn, id);
+    }
+
+    clearTemporaryChildren: function () {
+      if (this._temporaryParent) {
+        this._temporaryParent.destroy();
+        delete this._temporaryParent;
+      }
+    }
 
 This will require some matching work on the front:
 
-    getTemporaryChild: protocol.custom(function(id) {
+    getTemporaryChild: protocol.custom(function (id) {
         if (!this._temporaryParent) {
             this._temporaryParent = this.manage(new Front(this.client));
         }
         return this._getTemporaryChild(id);
     }, {
         impl: "_getTemporaryChild"
     }),
 
-    clearTemporaryChildren: protocol.custom(function(id) {
+    clearTemporaryChildren: protocol.custom(function (id) {
         if (this._temporaryParent) {
             this._temporaryParent.destroy();
             delete this._temporaryParent;
         }
         return this._clearTemporaryChildren();
     }, {
         impl: "_clearTemporaryChildren"
     })
 
 Telemetry
 ---------
 
 You can specify a telemetry probe id in your method spec:
 
-    echo: method(function(str) {
-        return str;
-    }, {
+    // spec:
+    methods: {
+      echo: {
         request: { str: Arg(0) },
         response: { str: RetVal() },
         telemetry: "ECHO"
-    });
+      }
+    }
+
+    // implementation:
+    echo: function (str) {
+      return str;
+    }
 
 ... and the time to execute that request will be included as a telemetry probe.
--- a/devtools/server/main.js
+++ b/devtools/server/main.js
@@ -395,16 +395,21 @@ var DebuggerServer = {
     }
     if (Services.prefs.getBoolPref("dom.mozSettings.enabled")) {
       this.registerModule("devtools/server/actors/settings", {
         prefix: "settings",
         constructor: "SettingsActor",
         type: { global: true }
       });
     }
+    this.registerModule("devtools/server/actors/addons", {
+      prefix: "addons",
+      constructor: "AddonsActor",
+      type: { global: true }
+    });
     this.registerModule("devtools/server/actors/webapps", {
       prefix: "webapps",
       constructor: "WebappsActor",
       type: { global: true }
     });
     this.registerModule("devtools/server/actors/device", {
       prefix: "device",
       constructor: "DeviceActor",
--- a/devtools/server/tests/unit/head_dbg.js
+++ b/devtools/server/tests/unit/head_dbg.js
@@ -3,16 +3,28 @@
 
 "use strict";
 var Cc = Components.classes;
 var Ci = Components.interfaces;
 var Cu = Components.utils;
 var Cr = Components.results;
 var CC = Components.Constructor;
 
+// Populate AppInfo before anything (like the shared loader) accesses
+// System.appinfo, which is a lazy getter.
+const _appInfo = {};
+Cu.import("resource://testing-common/AppInfo.jsm", _appInfo);
+_appInfo.updateAppInfo({
+  ID: "devtools@tests.mozilla.org",
+  name: "devtools-tests",
+  version: "1",
+  platformVersion: "42",
+  crashReporter: true,
+});
+
 const { require, loader } = Cu.import("resource://devtools/shared/Loader.jsm", {});
 const { worker } = Cu.import("resource://devtools/shared/worker/loader.js", {});
 const promise = require("promise");
 const { Task } = require("devtools/shared/task");
 
 const Services = require("Services");
 // Always log packets when running tests. runxpcshelltests.py will throw
 // the output away anyway, unless you give it the --verbose flag.
new file mode 100644
--- /dev/null
+++ b/devtools/server/tests/unit/test_addons_actor.js
@@ -0,0 +1,61 @@
+/* Any copyright is dedicated to the Public Domain.
+   http://creativecommons.org/publicdomain/zero/1.0/ */
+
+const protocol = require("devtools/shared/protocol");
+const {AddonsActor} = require("devtools/server/actors/addons");
+const {AddonsFront} = require("devtools/shared/fronts/addons");
+
+function startupAddonsManager() {
+  const internalManager = Cc["@mozilla.org/addons/integration;1"]
+    .getService(Ci.nsIObserver)
+    .QueryInterface(Ci.nsITimerCallback);
+
+  internalManager.observe(null, "addons-startup", null);
+}
+
+const profileDir = do_get_profile().clone();
+profileDir.append("extensions");
+startupAddonsManager();
+
+function* connect() {
+  const client = yield new Promise(resolve => {
+    get_chrome_actors(client => resolve(client));
+  });
+  const root = yield listTabs(client);
+  const addonsActor = root.addonsActor;
+  ok(addonsActor, "Got AddonsActor instance");
+
+  const addons = AddonsFront(client, {addonsActor});
+  return [client, addons];
+}
+
+add_task(function* testSuccessfulInstall() {
+  const [client, addons] = yield connect();
+
+  const allowMissing = false;
+  const usePlatformSeparator = true;
+  const addonPath = getFilePath("test_addons_actor/web-extension",
+                                allowMissing, usePlatformSeparator);
+  const installedAddon = yield addons.installTemporaryAddon(addonPath);
+  equal(installedAddon.id, "test-addons-actor@mozilla.org");
+  // The returned object is currently not a proper actor.
+  equal(installedAddon.actor, false);
+
+  const addonList = yield client.listAddons();
+  ok(addonList && addonList.addons && addonList.addons.map(a => a.name),
+     "Received list of add-ons");
+  const addon = addonList.addons.filter(a => a.id === installedAddon.id)[0];
+  ok(addon, "Test add-on appeared in root install list");
+
+  yield close(client);
+});
+
+add_task(function* testNonExistantPath() {
+  const [client, addons] = yield connect();
+
+  yield Assert.rejects(
+    addons.installTemporaryAddon("some-non-existant-path"),
+    /Could not install add-on.*Component returned failure/);
+
+  yield close(client);
+});
new file mode 100644
--- /dev/null
+++ b/devtools/server/tests/unit/test_addons_actor/web-extension/manifest.json
@@ -0,0 +1,10 @@
+{
+  "manifest_version": 2,
+  "name": "Test Addons Actor",
+  "version": "1.0",
+  "applications": {
+    "gecko": {
+      "id": "test-addons-actor@mozilla.org"
+    }
+  }
+}
--- a/devtools/server/tests/unit/xpcshell.ini
+++ b/devtools/server/tests/unit/xpcshell.ini
@@ -25,17 +25,19 @@ support-files =
   setBreakpoint-on-column-with-no-offsets-at-end-of-line.js
   setBreakpoint-on-column-with-no-offsets-in-gcd-script.js
   setBreakpoint-on-line.js
   setBreakpoint-on-line-in-gcd-script.js
   setBreakpoint-on-line-with-multiple-offsets.js
   setBreakpoint-on-line-with-multiple-statements.js
   setBreakpoint-on-line-with-no-offsets.js
   setBreakpoint-on-line-with-no-offsets-in-gcd-script.js
+  test_addons_actor/web-extension/manifest.json
 
+[test_addons_actor.js]
 [test_animation_name.js]
 [test_animation_type.js]
 [test_ScriptStore.js]
 [test_actor-registry-actor.js]
 [test_nesting-01.js]
 [test_nesting-02.js]
 [test_nesting-03.js]
 [test_forwardingprefix.js]
new file mode 100644
--- /dev/null
+++ b/devtools/shared/fronts/addons.js
@@ -0,0 +1,17 @@
+/* This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
+"use strict";
+
+const {addonsSpec} = require("devtools/shared/specs/addons");
+const protocol = require("devtools/shared/protocol");
+
+const AddonsFront = protocol.FrontClassWithSpec(addonsSpec, {
+  initialize: function (client, {addonsActor}) {
+    protocol.Front.prototype.initialize.call(this, client);
+    this.actorID = addonsActor;
+    this.manage(this);
+  }
+});
+
+exports.AddonsFront = AddonsFront;
--- a/devtools/shared/fronts/moz.build
+++ b/devtools/shared/fronts/moz.build
@@ -1,14 +1,15 @@
 # -*- Mode: python; c-basic-offset: 4; indent-tabs-mode: nil; tab-width: 40 -*-
 # vim: set filetype=python:
 # This Source Code Form is subject to the terms of the Mozilla Public
 # License, v. 2.0. If a copy of the MPL was not distributed with this
 # file, You can obtain one at http://mozilla.org/MPL/2.0/.
 
 DevToolsModules(
+    'addons.js',
     'animation.js',
     'highlighters.js',
     'inspector.js',
     'storage.js',
     'styles.js',
     'stylesheets.js'
 )
new file mode 100644
--- /dev/null
+++ b/devtools/shared/specs/addons.js
@@ -0,0 +1,20 @@
+/* This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
+"use strict";
+
+const {Arg, RetVal, generateActorSpec} = require("devtools/shared/protocol");
+
+const addonsSpec = generateActorSpec({
+  typeName: "addons",
+
+  methods: {
+    installTemporaryAddon: {
+      request: { addonPath: Arg(0, "string") },
+      response: { addon: RetVal("json") },
+      telemetry: "INSTALL_TEMPORARY_ADDON"
+    },
+  },
+});
+
+exports.addonsSpec = addonsSpec;
--- a/devtools/shared/specs/moz.build
+++ b/devtools/shared/specs/moz.build
@@ -1,14 +1,15 @@
 # -*- Mode: python; c-basic-offset: 4; indent-tabs-mode: nil; tab-width: 40 -*-
 # vim: set filetype=python:
 # This Source Code Form is subject to the terms of the Mozilla Public
 # License, v. 2.0. If a copy of the MPL was not distributed with this
 # file, You can obtain one at http://mozilla.org/MPL/2.0/.
 
 DevToolsModules(
+    'addons.js',
     'animation.js',
     'highlighters.js',
     'inspector.js',
     'storage.js',
     'styles.js',
     'stylesheets.js'
 )