Revert of [Presentation API] Allow PresentationConnection state change to "Connecting"

Revert of [Presentation API] Allow PresentationConnection state change to "Connecting" (patchset #5 id:180001 of https://codereview.chromium.org/2643473002/ )

Reason for revert:
presentation/presentation-close-reconnect.html is timing out consistently on e.g. WebKit Linux Trusty https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty/builds/22580

Original issue's description:
> [Presentation API] Allow PresentationConnection state change to "Connecting"
> 
> As per the Presentation API spec [1], when we reconnect to a closed
> PresentationConnection, its status changes from Closed to Connecting. This CL
> removes a NOTREACHED() called at state change to Connecting.
> 
> This CL also replaces the use of the var keyword with const/let in
> third_party/WebKit/LayoutTests/presentation/.
> 
> [1] https://w3c.github.io/presentation-api/#reconnecting-to-a-presentation
> 
> BUG=677540
> 
> Review-Url: https://codereview.chromium.org/2643473002
> Cr-Commit-Position: refs/heads/master@{#446458}
> Committed: https://chromium.googlesource.com/chromium/src/+/a8f908ca6ab0abe86af75c688099d8c9dee1685c

TBR=mfoltz@chromium.org,zhaobin@chromium.org,takumif@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=677540
Review-Url: https://codereview.chromium.org/2655073007
Cr-Commit-Position: refs/heads/master@{#446578}
Committed: https://chromium.googlesource.com/chromium/src/+/fb6e9d1fd67d2ee811d5276e6d07940cb2faf342

[suzyh_UTC10 (ex-contributor), 2017-01-27 03:40:00]

The CQ bit was checked by suzyh@chromium.org

[suzyh_UTC10 (ex-contributor), 2017-01-27 03:40:00]

Created Revert of [Presentation API] Allow PresentationConnection state change
to "Connecting"

[commit-bot: I haz the power, 2017-01-27 03:40:24]

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, 2017-01-27 03:41:19]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485488400053650, "parent_rev":
"b28df93e8b400bb452521c8e9a21a41de8771df9", "commit_rev":
"fb6e9d1fd67d2ee811d5276e6d07940cb2faf342"}

[commit-bot: I haz the power, 2017-01-27 03:41:48]

Description was changed from

==========
Revert of [Presentation API] Allow PresentationConnection state change to
"Connecting" (patchset #5 id:180001 of
https://codereview.chromium.org/2643473002/ )

Reason for revert:
presentation/presentation-close-reconnect.html is timing out consistently on
e.g. WebKit Linux Trusty
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty...

Original issue's description:
> [Presentation API] Allow PresentationConnection state change to "Connecting"
> 
> As per the Presentation API spec [1], when we reconnect to a closed
> PresentationConnection, its status changes from Closed to Connecting. This CL
> removes a NOTREACHED() called at state change to Connecting.
> 
> This CL also replaces the use of the var keyword with const/let in
> third_party/WebKit/LayoutTests/presentation/.
> 
> [1] https://w3c.github.io/presentation-api/#reconnecting-to-a-presentation
> 
> BUG=677540
> 
> Review-Url: https://codereview.chromium.org/2643473002
> Cr-Commit-Position: refs/heads/master@{#446458}
> Committed:
https://chromium.googlesource.com/chromium/src/+/a8f908ca6ab0abe86af75c688099...

TBR=mfoltz@chromium.org,zhaobin@chromium.org,takumif@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=677540
==========

to

==========
Revert of [Presentation API] Allow PresentationConnection state change to
"Connecting" (patchset #5 id:180001 of
https://codereview.chromium.org/2643473002/ )

Reason for revert:
presentation/presentation-close-reconnect.html is timing out consistently on
e.g. WebKit Linux Trusty
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty...

Original issue's description:
> [Presentation API] Allow PresentationConnection state change to "Connecting"
> 
> As per the Presentation API spec [1], when we reconnect to a closed
> PresentationConnection, its status changes from Closed to Connecting. This CL
> removes a NOTREACHED() called at state change to Connecting.
> 
> This CL also replaces the use of the var keyword with const/let in
> third_party/WebKit/LayoutTests/presentation/.
> 
> [1] https://w3c.github.io/presentation-api/#reconnecting-to-a-presentation
> 
> BUG=677540
> 
> Review-Url: https://codereview.chromium.org/2643473002
> Cr-Commit-Position: refs/heads/master@{#446458}
> Committed:
https://chromium.googlesource.com/chromium/src/+/a8f908ca6ab0abe86af75c688099...

TBR=mfoltz@chromium.org,zhaobin@chromium.org,takumif@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=677540

Review-Url: https://codereview.chromium.org/2655073007
Cr-Commit-Position: refs/heads/master@{#446578}
Committed:
https://chromium.googlesource.com/chromium/src/+/fb6e9d1fd67d2ee811d5276e6d07...
==========

[commit-bot: I haz the power, 2017-01-27 03:41:51]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/fb6e9d1fd67d2ee811d5276e6d07...