Presentation API: ignore presentation id passed to startSession().

content/common/presentation/presentation_service.mojom

Index: content/common/presentation/presentation_service.mojom
diff --git a/content/common/presentation/presentation_service.mojom b/content/common/presentation/presentation_service.mojom
index d033a482cb80d280926902f9261a8e850ebbc8f2..3b528f08ffb917feae24fa26237b115a07b9eaed 100644
--- a/content/common/presentation/presentation_service.mojom
+++ b/content/common/presentation/presentation_service.mojom
@@ -75,11 +75,10 @@ interface PresentationService {
   // session failed.
   // The presentation id is always returned along with the initialized

[whywhat, 2015/07/02 19:54:31]

Perhaps, you should mention that the presentation id is always generated by the
UA here?
I.e. replace the sentence with "The presentation id returned in |sessionInfo| on
success is generated by the UA."

[mlamouri (slow - plz ping), 2015/07/08 14:37:09]

Done.

   // session on success.
-  // If the UA identifies a matching session (same presentation url and id),
-  // the user may choose this existing session and the page will join it
-  // rather than get a new one. An empty presentation id means that the
-  // UA will generate the presentation id.
-  StartSession(string presentation_url, string? presentation_id)
+  // If the UA identifies a matching session (same presentation url), the user
+  // may choose this existing session and the page will join it rather than get
+  // a new one.
+  StartSession(string presentation_url)
       => (PresentationSessionInfo? sessionInfo, PresentationError? error);
 
   // Called when joinSession() is called by the frame. The result callback
@@ -95,7 +94,7 @@ interface PresentationService {
   // the send requests and invalidate all pending requests. This occurs
   // for eg., when frame is deleted or navigated away.
   SendSessionMessage(SessionMessage message_request) => (bool success);
-  
+

[whywhat, 2015/07/02 19:54:31]

what's this diff?

[mlamouri (slow - plz ping), 2015/07/08 14:37:09]

Can't find that in my diff locally.

   // Called when closeSession() is called by the frame.
   CloseSession(string presentation_url, string presentation_id);