JavaScript Promise.all()の正しい書き方

Question

前提・実現したいこと

JavaScriptで非同期処理（具体的には画像の読み込み）を待ってからrequestAnimationFrameを走らせたいと思い、Promiseを初めて試してみました。
試行錯誤の結果、想定通りの動作になりましたが、そこに至るまでのコードが何故失敗していたかを知りたい（Promiseに対する理解を深めたい）です。

該当のソースコード

  //失敗していたコード
  //promisesはPromiseの配列
  Promise.all(promises).then(requestAnimationFrame(update););
  //promisesの各要素がpendingのままupdateが実行されてしまう

  //成功したコード
  //promisesはPromiseの配列
  Promise.all(promises).then(function(results){
    requestAnimationFrame(update);
  });  //promisesの各要素がresolveしてからupdateが実行される（想定通り）

違いは明確なのですが、PromiseおよびPromise.all()の正しい理解の仕方、もしくは初心者が参考にすべき資料などがありましたらご教示下さい。

Answer 1

Promiseがどうという話ではなく、関数の渡し方および文法の問題です。

//                                       実行している↓    ↓セミコロンは文法的におかしい
Promise.all(promises).then(requestAnimationFrame(update););

関数は()をつけると実行できます。

function test() {}
test(); // ←実行

Comments

[yamakane]

回答ありがとうございました。ご教示いただいて、Promise.all()でなくPromise.prototype.then()の解説を見てからようやくぼんやり見えた気がしました。
https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Promise/then
"then() メソッドは２つの引数を持ち、それらは、 Promise が成功した場合と、失敗した場合のコールバック関数です。"
"片方または両方の引数が省略されるか関数でない場合、then にはハンドラーが足りていないのですが、如何なるエラーも生成されません。"

ご指摘の通り文法もよく分かっていませんが引き続き勉強しながら進めていきます。
ありがとうございました。