[Presentation API] Remove references to presentation sessions.

[Presentation API] Remove references to presentation sessions.

"Sessions" are from an early draft of the Presentation API and they are confusing to define APIs around as they are not defined by the current spec.

This mechanical change updates the PresentationService mojo API, and upstream and downstream APIs, to remove references to them.

Renamings in presentation.mojom:

  PresentationSessionInfo => PresentationInfo
  SESSION_REQUEST_CANCELLED => PRESENTATION_REQUEST_CANCELLED
  OnDefaultSessionStarted => OnDefaultPresentationStarted
  StartSession => StartPresentation
  JoinSession => ReconnectPresentation
  OnDefaultSessionStarted => OnDefaultPresentationStarted

In WebPresentationClient:

  terminateConnection => terminatePresentation
  closeSession => closeConnection

In WebPresentationController:

  didStartDefaultSession => didStartDefaultPresentation
  didChangeSessionState => didChangeConnectionState
  didReceiveSessionTextMessage => didReceiveConnectionTextMessage
  didReceiveSessionBinaryMessage => didReceiveConnectionBinaryMessage

In WebPresentationReceiver:

  didChangeSessionState => didChangeConnectionState

In PresentationServiceDelegate:

  PresentationSessionStartedCallback => PresentationConnectionCallback
  PresentationSessionErrorCallback => PresentationConnectionErrorCallback

  
  


BUG=647290
Review-Url: https://codereview.chromium.org/2737413003
Cr-Commit-Position: refs/heads/master@{#458243}
Committed: https://chromium.googlesource.com/chromium/src/+/11fc4df8400a28054b5f5081e67985e83c387643

[mark a. foltz, 2017-03-09 21:19:42]

Description was changed from

==========
[Presentation API] Replace references to sessions.

"Sessions" are from an early draft of the Presentation API and their use
throughout content/ is confusing.  This mechanical replaces "session" with
"presentation" or "connection" depending on context.

Specific renamings:

BUG=647290
==========

to

==========
[Presentation API] Remove references to presentation sessions.

"Sessions" are from an early draft of the Presentation API and their use
throughout content/ is confusing.  This mechanical replaces "session" with
"presentation" or "connection" depending on context.

Specific renamings:

BUG=647290
==========

[mark a. foltz, 2017-03-09 22:34:23]

Description was changed from

==========
[Presentation API] Remove references to presentation sessions.

"Sessions" are from an early draft of the Presentation API and their use
throughout content/ is confusing.  This mechanical replaces "session" with
"presentation" or "connection" depending on context.

Specific renamings:

BUG=647290
==========

to

==========
[Presentation API] Remove references to presentation sessions.

"Sessions" are from an early draft of the Presentation API and they are
confusing to define APIs around as they are not defined by the current spec.

This mechanical change updates the PresentationService mojo API, and upstream
and downstream APIs, to remove references to them.

Renamings in presentation.mojom:

  PresentationSessionInfo => PresentationInfo
  SESSION_REQUEST_CANCELLED => PRESENTATION_REQUEST_CANCELLED
  OnDefaultSessionStarted => OnDefaultPresentationStarted
  StartSession => StartPresentation
  JoinSession => ReconnectPresentation
  OnDefaultSessionStarted => OnDefaultPresentationStarted

In WebPresentationClient:

  terminateConnection => terminatePresentation
  closeSession => closeConnection

In WebPresentationController:

  didStartDefaultSession => didStartDefaultPresentation
  didChangeSessionState => didChangeConnectionState
  didReceiveSessionTextMessage => didReceiveConnectionTextMessage
  didReceiveSessionBinaryMessage => didReceiveConnectionBinaryMessage

In WebPresentationReceiver:

  didChangeSessionState => didChangeConnectionState

In PresentationServiceDelegate:

  PresentationSessionStartedCallback => PresentationConnectionCallback
  PresentationSessionErrorCallback => PresentationConnectionErrorCallback

  
  


BUG=647290
==========

[mark a. foltz, 2017-03-09 22:35:07]

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

[commit-bot: I haz the power, 2017-03-09 22:36:03]

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, 2017-03-09 22:51:41]

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

[commit-bot: I haz the power, 2017-03-09 22:51:42]

Dry run: Try jobs failed on following builders:
  android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)

[mark a. foltz, 2017-03-09 23:13:05]

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

[commit-bot: I haz the power, 2017-03-09 23:13:34]

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

[mark a. foltz, 2017-03-10 00:39:22]

The CQ bit was unchecked by mfoltz@chromium.org

[mark a. foltz, 2017-03-10 21:23:43]

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

[commit-bot: I haz the power, 2017-03-10 21:24:01]

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, 2017-03-10 23:26:06]

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

[commit-bot: I haz the power, 2017-03-10 23:26:07]

Dry run: This issue passed the CQ dry run.

[mark a. foltz, 2017-03-10 23:35:16]

mfoltz@chromium.org changed reviewers:
+ avi@chromium.org, dcheng@chromium.org, imcheng@chromium.org,
mlamouri@chromium.org, zhaobin@chromium.org

[mark a. foltz, 2017-03-10 23:35:17]

imcheng@, zhaobin@: PTAL everything
dcheng@:            PTAL at .mojom and struct_traits changes
avi@:               PTAL at content/public
mlamouri@:          PTAL at chrome/browser/media/android

[Avi (use Gerrit), 2017-03-11 00:09:27]

content/public lgtm

[dcheng, 2017-03-11 07:04:50]

lgtm

(i mostly only skimmed, since this CL touches so many lines)

[imcheng, 2017-03-13 21:17:55]

lgtm

https://codereview.chromium.org/2737413003/diff/60001/content/browser/present...
File content/browser/presentation/presentation_service_impl.cc (right):

https://codereview.chromium.org/2737413003/diff/60001/content/browser/present...
content/browser/presentation/presentation_service_impl.cc:501:
PresentationServiceImpl::NewPresentationCallbackWrapper ::
Please fix formatting

https://codereview.chromium.org/2737413003/diff/60001/content/browser/present...
content/browser/presentation/presentation_service_impl.cc:505:
PresentationServiceImpl::NewPresentationCallbackWrapper ::
Please fix formatting

https://codereview.chromium.org/2737413003/diff/60001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/presentation/presentation-controller-terminate-connection.html
(right):

https://codereview.chromium.org/2737413003/diff/60001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/presentation/presentation-controller-terminate-connection.html:13:
var button = document.querySelector('button');
const?

[zhaobin, 2017-03-13 21:23:26]

lgtm

[mlamouri (slow - plz ping), 2017-03-16 11:56:21]

lgtm

[mark a. foltz, 2017-03-16 19:48:13]

On 2017/03/13 at 21:17:55, imcheng wrote:
> lgtm
> 
>
https://codereview.chromium.org/2737413003/diff/60001/content/browser/present...
> File content/browser/presentation/presentation_service_impl.cc (right):
> 
>
https://codereview.chromium.org/2737413003/diff/60001/content/browser/present...
> content/browser/presentation/presentation_service_impl.cc:501:
PresentationServiceImpl::NewPresentationCallbackWrapper ::
> Please fix formatting
> 
>
https://codereview.chromium.org/2737413003/diff/60001/content/browser/present...
> content/browser/presentation/presentation_service_impl.cc:505:
PresentationServiceImpl::NewPresentationCallbackWrapper ::
> Please fix formatting
> 
>
https://codereview.chromium.org/2737413003/diff/60001/third_party/WebKit/Layo...
> File
third_party/WebKit/LayoutTests/presentation/presentation-controller-terminate-connection.html
(right):
> 
>
https://codereview.chromium.org/2737413003/diff/60001/third_party/WebKit/Layo...
>
third_party/WebKit/LayoutTests/presentation/presentation-controller-terminate-connection.html:13:
var button = document.querySelector('button');
> const?

Done.

[mark a. foltz, 2017-03-16 19:50:09]

https://codereview.chromium.org/2737413003/diff/60001/content/browser/present...
File content/browser/presentation/presentation_service_impl.cc (right):

https://codereview.chromium.org/2737413003/diff/60001/content/browser/present...
content/browser/presentation/presentation_service_impl.cc:501:
PresentationServiceImpl::NewPresentationCallbackWrapper ::
On 2017/03/13 at 21:17:55, imcheng wrote:
> Please fix formatting

Done.

https://codereview.chromium.org/2737413003/diff/60001/content/browser/present...
content/browser/presentation/presentation_service_impl.cc:505:
PresentationServiceImpl::NewPresentationCallbackWrapper ::
On 2017/03/13 at 21:17:55, imcheng wrote:
> Please fix formatting

Done.

[mark a. foltz, 2017-03-20 18:48:52]

The CQ bit was checked by mfoltz@chromium.org

[mark a. foltz, 2017-03-20 18:48:52]

The patchset sent to the CQ was uploaded after l-g-t-m from avi@chromium.org,
imcheng@chromium.org, mlamouri@chromium.org, dcheng@chromium.org,
zhaobin@chromium.org
Link to the patchset: https://codereview.chromium.org/2737413003/#ps140001
(title: "Successful rebase :-)")

[commit-bot: I haz the power, 2017-03-20 18:49:38]

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-03-20 18:54:43]

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

[commit-bot: I haz the power, 2017-03-20 18:54:45]

Try jobs failed on following builders:
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[mark a. foltz, 2017-03-20 20:40:44]

The CQ bit was checked by mfoltz@chromium.org

[mark a. foltz, 2017-03-20 20:40:45]

The patchset sent to the CQ was uploaded after l-g-t-m from avi@chromium.org,
imcheng@chromium.org, mlamouri@chromium.org, dcheng@chromium.org,
zhaobin@chromium.org
Link to the patchset: https://codereview.chromium.org/2737413003/#ps180001
(title: "Update PresentationServiceDelegateImpl unittest")

[commit-bot: I haz the power, 2017-03-20 20:41:20]

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-03-20 23:49:22]

CQ is committing da patch.

Bot data: {"patchset_id": 180001, "attempt_start_ts": 1490042444669380,
"parent_rev": "79fa3471cea36082fa33d26b452878490b9cba84", "commit_rev":
"11fc4df8400a28054b5f5081e67985e83c387643"}

[commit-bot: I haz the power, 2017-03-20 23:50:05]

Description was changed from

==========
[Presentation API] Remove references to presentation sessions.

"Sessions" are from an early draft of the Presentation API and they are
confusing to define APIs around as they are not defined by the current spec.

This mechanical change updates the PresentationService mojo API, and upstream
and downstream APIs, to remove references to them.

Renamings in presentation.mojom:

  PresentationSessionInfo => PresentationInfo
  SESSION_REQUEST_CANCELLED => PRESENTATION_REQUEST_CANCELLED
  OnDefaultSessionStarted => OnDefaultPresentationStarted
  StartSession => StartPresentation
  JoinSession => ReconnectPresentation
  OnDefaultSessionStarted => OnDefaultPresentationStarted

In WebPresentationClient:

  terminateConnection => terminatePresentation
  closeSession => closeConnection

In WebPresentationController:

  didStartDefaultSession => didStartDefaultPresentation
  didChangeSessionState => didChangeConnectionState
  didReceiveSessionTextMessage => didReceiveConnectionTextMessage
  didReceiveSessionBinaryMessage => didReceiveConnectionBinaryMessage

In WebPresentationReceiver:

  didChangeSessionState => didChangeConnectionState

In PresentationServiceDelegate:

  PresentationSessionStartedCallback => PresentationConnectionCallback
  PresentationSessionErrorCallback => PresentationConnectionErrorCallback

  
  


BUG=647290
==========

to

==========
[Presentation API] Remove references to presentation sessions.

"Sessions" are from an early draft of the Presentation API and they are
confusing to define APIs around as they are not defined by the current spec.

This mechanical change updates the PresentationService mojo API, and upstream
and downstream APIs, to remove references to them.

Renamings in presentation.mojom:

  PresentationSessionInfo => PresentationInfo
  SESSION_REQUEST_CANCELLED => PRESENTATION_REQUEST_CANCELLED
  OnDefaultSessionStarted => OnDefaultPresentationStarted
  StartSession => StartPresentation
  JoinSession => ReconnectPresentation
  OnDefaultSessionStarted => OnDefaultPresentationStarted

In WebPresentationClient:

  terminateConnection => terminatePresentation
  closeSession => closeConnection

In WebPresentationController:

  didStartDefaultSession => didStartDefaultPresentation
  didChangeSessionState => didChangeConnectionState
  didReceiveSessionTextMessage => didReceiveConnectionTextMessage
  didReceiveSessionBinaryMessage => didReceiveConnectionBinaryMessage

In WebPresentationReceiver:

  didChangeSessionState => didChangeConnectionState

In PresentationServiceDelegate:

  PresentationSessionStartedCallback => PresentationConnectionCallback
  PresentationSessionErrorCallback => PresentationConnectionErrorCallback

  
  


BUG=647290

Review-Url: https://codereview.chromium.org/2737413003
Cr-Commit-Position: refs/heads/master@{#458243}
Committed:
https://chromium.googlesource.com/chromium/src/+/11fc4df8400a28054b5f5081e679...
==========

[commit-bot: I haz the power, 2017-03-20 23:50:07]

Committed patchset #10 (id:180001) as
https://chromium.googlesource.com/chromium/src/+/11fc4df8400a28054b5f5081e679...