Potential memory leak

[kennethlarsen]

When running the test suite in our app I see that the app doesn't tear down as expected. When investigating I saw that this was caused by ember-makeup. I haven't had time yet to do a proper investigation but Chrome points to the constructor here as the issue:

  // @TODO: This fails, when the primary host does not depend on `ember-makeup`
  // directly. What should the correct behavior be? I think we should throw a
  // build error, similar to `ember-cli-resolve-asset`.
  class MakeupService extends Ember.Service.extend(Ember.Evented) {
    constructor(...args) {
      super(...args);
      this.classNamePrefix = _config.default.options.contextClassNamePrefix;
      this.themePaths = _config.default.themePaths;
      this.linkElement = void 0;
      this.computedStyle = void 0;
      this.isReady = false;
    }