Em JS, indexOf retorna -1 se a string não tiver uma substring. Se você quiser verificar apenas a existência de substring, escreva o código como este:
if (string.indexOf(substring) != -1) {/*actions*/}
Mas para encurtar você pode escrever:
if (~string.indexOf(substring)) {/*actions*/}