O maior problema é que não está pegando os caracteres na ordem correta, está pegando o primeiro e fazendo o expoente com a menor grandeza quando o primeiro caractere é o de maior grandeza, então tem que inverter o cálculo do expoente ou pegar os caracteres ao contrário, que foi o que eu optei. Ainda tem como varrer o laço ao contrário começando do fim para o começo, mas não acho que vale a pena, ainda teria dois casos ao contrário e precisaria ter uma cálculo a fazer.
Não é um erro, mas não precisa de if
, basta matemática, se os valores só podem ser 0 ou 1 multiplicado por 0 sempre dá 0 e multiplicando por 1 sempre dá o próprio valor.
Só um último comentário: o que você está fazendo é um parse de um texto que tem um padrão que presume-se ser uma notação binária de um número e quer converter para número, ponto, não número. Número é número, notação é notação, então existe uma notação decimal, mas não existe um número decimal.
A forma simples, eficiente (não faz alocações de memória e processamento para manipular a string) e matemática pura seria assim (se é um exercício que não pode usar recursos da biblioteca acho que seria assim):
let bin = '1110';
let dec = 0;
for (let c = 0; c < bin.length; c++) dec += Math.pow(2, c) * bin[bin.length - c - 1]; //calcula para pegar do último ao primeiro
console.log(dec);
Coloquei no GitHub para referência futura.