Sync the Presentation API interfaces with the spec

Sync the Presentation API interfaces with the spec

https://w3c.github.io/presentation-api/

Spec issues found in the process:
https://github.com/w3c/presentation-api/issues/137
https://github.com/w3c/presentation-api/issues/138
https://github.com/w3c/presentation-api/issues/139
https://github.com/w3c/presentation-api/issues/140
https://github.com/w3c/presentation-api/issues/141

BUG=460722
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=198408

[philipj_slow, 2015-07-07 12:47:05]

The CQ bit was checked by philipj@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2015-07-07 12:47:13]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1209423005/1

[philipj_slow, 2015-07-07 12:47:38]

philipj@opera.com changed reviewers:
+ avayvod@chromium.org

[philipj_slow, 2015-07-07 12:47:39]

PTAL. If you want them, I'll happily reassign the TODOs to you :)

[whywhat, 2015-07-07 13:00:07]

Thanks Philip!
I left a couple of comments about two TODOs.
You can assign the TODOs to me or mlamouri (CCed)

https://codereview.chromium.org/1209423005/diff/1/Source/modules/presentation...
File Source/modules/presentation/Presentation.idl (right):

https://codereview.chromium.org/1209423005/diff/1/Source/modules/presentation...
Source/modules/presentation/Presentation.idl:14: // TODO(philipj): The
|presentationId| argument is not in the spec.
https://codereview.chromium.org/1209423005/

https://codereview.chromium.org/1209423005/diff/1/Source/modules/presentation...
Source/modules/presentation/Presentation.idl:18: // TODO(philipj): The |url|
argument is ultimately unused.
This was only landed yesterday: https://codereview.chromium.org/1222623003

[whywhat, 2015-07-07 13:00:14]

lgtm

[philipj_slow, 2015-07-07 13:18:32]

https://codereview.chromium.org/1209423005/diff/1/Source/modules/presentation...
File Source/modules/presentation/Presentation.idl (right):

https://codereview.chromium.org/1209423005/diff/1/Source/modules/presentation...
Source/modules/presentation/Presentation.idl:14: // TODO(philipj): The
|presentationId| argument is not in the spec.
On 2015/07/07 13:00:07, whywhat wrote:
> https://codereview.chromium.org/1209423005/

Did you paste the wrong link?

https://codereview.chromium.org/1209423005/diff/1/Source/modules/presentation...
Source/modules/presentation/Presentation.idl:18: // TODO(philipj): The |url|
argument is ultimately unused.
On 2015/07/07 13:00:07, whywhat wrote:
> This was only landed yesterday: https://codereview.chromium.org/1222623003

Do you think that the URL argument will actually be used for something? Please
comment on https://github.com/w3c/presentation-api/issues/138 if I'm talking
crazy.

[philipj_slow, 2015-07-07 13:23:54]

Assign TODOs to Anton

[whywhat, 2015-07-07 13:53:58]

Commented on the GitHub issue.

https://codereview.chromium.org/1209423005/diff/1/Source/modules/presentation...
File Source/modules/presentation/Presentation.idl (right):

https://codereview.chromium.org/1209423005/diff/1/Source/modules/presentation...
Source/modules/presentation/Presentation.idl:14: // TODO(philipj): The
|presentationId| argument is not in the spec.
On 2015/07/07 at 13:18:31, philipj wrote:
> On 2015/07/07 13:00:07, whywhat wrote:
> > https://codereview.chromium.org/1209423005/
> 
> Did you paste the wrong link?

Yes, I meant: https://codereview.chromium.org/1208083004

[philipj_slow, 2015-07-07 14:31:02]

https://codereview.chromium.org/1209423005/diff/1/Source/modules/presentation...
File Source/modules/presentation/Presentation.idl (right):

https://codereview.chromium.org/1209423005/diff/1/Source/modules/presentation...
Source/modules/presentation/Presentation.idl:14: // TODO(philipj): The
|presentationId| argument is not in the spec.
On 2015/07/07 13:53:58, whywhat wrote:
> On 2015/07/07 at 13:18:31, philipj wrote:
> > On 2015/07/07 13:00:07, whywhat wrote:
> > > https://codereview.chromium.org/1209423005/
> > 
> > Did you paste the wrong link?
> 
> Yes, I meant: https://codereview.chromium.org/1208083004

I see. There will be conflicts here anyway so I'll just leave this to be removed
in a few days time :)

https://codereview.chromium.org/1209423005/diff/1/Source/modules/presentation...
Source/modules/presentation/Presentation.idl:18: // TODO(philipj): The |url|
argument is ultimately unused.
On 2015/07/07 13:18:31, philipj wrote:
> On 2015/07/07 13:00:07, whywhat wrote:
> > This was only landed yesterday: https://codereview.chromium.org/1222623003
> 
> Do you think that the URL argument will actually be used for something? Please
> comment on https://github.com/w3c/presentation-api/issues/138 if I'm talking
> crazy.

I'll revert this TODO, it's not actionable until the spec changes, if it does.

[philipj_slow, 2015-07-07 14:32:49]

The CQ bit was checked by philipj@opera.com

[philipj_slow, 2015-07-07 14:32:49]

The patchset sent to the CQ was uploaded after l-g-t-m from avayvod@chromium.org
Link to the patchset: https://codereview.chromium.org/1209423005/#ps60001
(title: "Drop speculative TODO")

[commit-bot: I haz the power, 2015-07-07 14:33:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1209423005/60001

[commit-bot: I haz the power, 2015-07-07 15:34:34]

Committed patchset #4 (id:60001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=198408