Content refactoring start - checkStatus POC

[beshur]

Refactoring based on functions:

• Move checkStatus into class
• Move start into class
• Move pause into class
• Move getTitle into class
• index.js cleanup

[beshur]

this is obsolete

[beshur]

Moved out of index.js without changes

[endway]

Since you moved customLastPlayerSelector into a separate class it won't work anymore.

[beshur]

I wanted to pass it as a param to provider methods

[endway]

I like the idea to have mixins, as most of the status/pause/play functions can be grouped into few mixins, like:

status:

document.querySelector(PROVIDER_SPECIFIC_QUERY).classList.contains(PROVIDER_SPECIFIC_CLASS)

play/pause:

document.querySelector(PROVIDER_SPECIFIC_QUERY).click();

But, what I don't like is creating a separate class for each provider.
Previously, I thought that this is ok, but now it seems to me that that's a lot of redundant classes.

I'm thinking about a Strategy pattern, which may fix this:

class Service {
    constructor(statusStrategy, controlStrategy, titleStrategy) {
    }
    
    getStatus() {
        return this.statusStrategy.getStatus();
    }

    play() {
        this.controlStrategy.play();
    }

    pause() {
        this.controlStrategy.pause();
    }
}

const servicesRegistry = [
    {
        hostPattern: "audible.**", // we can add support fields, like `host: string`, `hosts: Array<string>`
        statusStrategy: oneOfeTheVideos.getCheckStatusStrategy(".path .to .player"),
        controlStrategy: someOtherType.getControlStrategy("#player"),
    },
];

function getService(domain) {
    const matchedService = servicesRegistry.find(serviceConfig => {
        return serviceConfig.host === domain
            || serviceConfig.hosts.include(domain)
            || serviceConfig.hostPattern.match(domain);
    });

    if (!matchedService) {
        return;
    }

    return new Service(matchedService.statusStrategy, matchedService.controlStrategy);
}

Not sure about registry.
@beshur What do you think?

[beshur]

Clean branch #157