Closed Bug 1292057 Opened 3 years ago Closed 3 years ago

[Presentation WebAPI] add complete NSPR log for core service and webidl implementation

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla51
Tracking Status
firefox51 --- fixed

People

(Reporter: schien, Assigned: schien)

References

(Blocks 2 open bugs)

Details

(Whiteboard: [ETA 9/2])

Attachments

(1 file)

No description provided.
Assignee: nobody → schien
Whiteboard: [ETA 9/2]
Comment on attachment 8781076 [details]
Bug 1292057 - add NSPR log for Presentation API.

@kershaw, please let me know if any other log you think is useful.
Comment on attachment 8781076 [details]
Bug 1292057 - add NSPR log for Presentation API.

https://reviewboard.mozilla.org/r/71576/#review69290

::: dom/presentation/PresentationConnection.cpp:354
(Diff revision 1)
>  }
>  
>  NS_IMETHODIMP
>  PresentationConnection::NotifyMessage(const nsAString& aSessionId,
>                                        const nsACString& aData)
>  {

Adding a log here could be helpful.

::: dom/presentation/PresentationConnection.cpp:382
(Diff revision 1)
>  }
>  
>  NS_IMETHODIMP
>  PresentationConnection::NotifyReplaced()
>  {
>    return NotifyStateChange(mId,

Also here.

::: dom/presentation/PresentationService.cpp:305
(Diff revision 1)
>    }
>  }
>  
>  nsresult
>  PresentationService::HandleDeviceChange()
>  {

Not sure if we are insterested in device change. If yes, you might want to add a log here.

::: dom/presentation/PresentationService.cpp:486
(Diff revision 1)
>  }
>  
>  nsresult
>  PresentationService::HandleReconnectRequest(nsIPresentationSessionRequest* aRequest)
>  {
>    nsCOMPtr<nsIPresentationControlChannel> ctrlChannel;

Add here to log reconnecting at receiver side.

::: dom/presentation/PresentationService.cpp:985
(Diff revision 1)
>  PresentationService::UntrackSessionInfo(const nsAString& aSessionId,
>                                          uint8_t aRole)
>  {
>    MOZ_ASSERT(aRole == nsIPresentationService::ROLE_CONTROLLER ||
>               aRole == nsIPresentationService::ROLE_RECEIVER);
>    // Remove the session info.

Also add a log here.

::: dom/presentation/PresentationSessionInfo.cpp:902
(Diff revision 1)
>  // nsIServerSocketListener
>  NS_IMETHODIMP
>  PresentationControllingInfo::OnSocketAccepted(nsIServerSocket* aServerSocket,
>                                              nsISocketTransport* aTransport)
>  {
>    int32_t port;

Add a log here could be helpful.

::: dom/presentation/PresentationSessionInfo.cpp:925
(Diff revision 1)
>  
>  NS_IMETHODIMP
>  PresentationControllingInfo::OnStopListening(nsIServerSocket* aServerSocket,
>                                             nsresult aStatus)
>  {
>    MOZ_ASSERT(NS_IsMainThread());

Also here.

::: dom/presentation/PresentationSessionInfo.cpp:1222
(Diff revision 1)
>            false;
>  }
>  
>  nsresult
>  PresentationPresentingInfo::NotifyResponderReady()
>  {

Add a log here.
Comment on attachment 8781076 [details]
Bug 1292057 - add NSPR log for Presentation API.

https://reviewboard.mozilla.org/r/71578/#review69296

Overall looks good to me.
I just want to add more logs in callback functions.
Attachment #8781076 - Flags: review?(kechang) → review+
Pushed by schien@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f63971df6d31
add NSPR log for Presentation API. r=kershaw
https://hg.mozilla.org/mozilla-central/rev/f63971df6d31
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.