[promises] remove if kPending check

[promises] remove if kPending check

FulfillPromise is always called when a promise is in a pending state
which makes this check redundant.

Committed: https://crrev.com/d390e11e02dd94d6602c4beb5baf61fd210ca761
Cr-Commit-Position: refs/heads/master@{#40556}

[gsathya, 2016-10-24 17:30:30]

The CQ bit was checked by gsathya@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-24 17:30:33]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-24 18:02:51]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-24 18:02:52]

Dry run: This issue passed the CQ dry run.

[gsathya, 2016-10-25 08:24:41]

gsathya@chromium.org changed reviewers:
+ adamk@chromium.org

[gsathya, 2016-10-25 10:49:16]

Description was changed from

==========
[promises] remove if kPending check
==========

to

==========
[promises] remove if kPending check

FulfillPromise is always called when a promise is in a pending state 
which makes this check redundant.
==========

[adamk, 2016-10-25 11:02:18]

lgtm

Looks like this is an Assert in the spec, so seems reasonable that this is a
no-op in our code.

[gsathya, 2016-10-25 11:04:12]

The CQ bit was checked by gsathya@chromium.org

[commit-bot: I haz the power, 2016-10-25 11:04:19]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-25 11:06:00]

Description was changed from

==========
[promises] remove if kPending check

FulfillPromise is always called when a promise is in a pending state 
which makes this check redundant.
==========

to

==========
[promises] remove if kPending check

FulfillPromise is always called when a promise is in a pending state 
which makes this check redundant.
==========

[commit-bot: I haz the power, 2016-10-25 11:06:01]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-17 22:12:16]

Description was changed from

==========
[promises] remove if kPending check

FulfillPromise is always called when a promise is in a pending state 
which makes this check redundant.
==========

to

==========
[promises] remove if kPending check

FulfillPromise is always called when a promise is in a pending state
which makes this check redundant.

Committed: https://crrev.com/d390e11e02dd94d6602c4beb5baf61fd210ca761
Cr-Commit-Position: refs/heads/master@{#40556}
==========

[commit-bot: I haz the power, 2016-11-17 22:12:18]

Patchset 1 (id:??) landed as
https://crrev.com/d390e11e02dd94d6602c4beb5baf61fd210ca761
Cr-Commit-Position: refs/heads/master@{#40556}