Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

QBWEBSDK-124 #44

Merged
merged 21 commits into from
Oct 20, 2015
Merged

QBWEBSDK-124 #44

merged 21 commits into from
Oct 20, 2015

Conversation

dimaspirit
Copy link
Contributor

No description provided.

@soulfly
Copy link
Contributor

soulfly commented Oct 16, 2015

  1. acceptCallTime не нужно делать глобальным, ты его используешь только в WebRTCSession.prototype.accept
  2. startCallTime лучше инициализировать не в _createPeer, а в _callInternal и processOnCall. Иначе эта переменная у тебя инициализируется N paз (N=количество оппонентов)
  3. Переименуй _startOfferTimer и offerTimer в _startWaitingOfferOrAnswerTimer и waitingOfferOrAnswerTimer
  4. У тебя нет кода, который кенселит этот таймер если другой юзер все-таки сделал accept

@soulfly
Copy link
Contributor

soulfly commented Oct 16, 2015

  1. Тут еще можно закрывать пир
        if(peerConnection.state !== RTCPeerConnection.State.CONNECTED) {
              peerConnection._clearDialingTimer();
              peerConnection.release();
              self.processOnNotAnswer(peerConnection);
        }        

@soulfly
Copy link
Contributor

soulfly commented Oct 16, 2015

  1. Вообще у нас много где есть места с кодом
   peerConnection._clearDialingTimer();
   peerConnection.release();

нужно сделать спец функцию типа closePeerConnection(peer)

…uickBlox/quickblox-javascript-sdk into develop.webrtc.QBWEBSDK-124_v2

Conflicts:
	js/modules/webrtc/qbWebRTCSession.js
	quickblox.min.js
@dimaspirit
Copy link
Contributor Author

  1. Ответ в комментарии
  2. _createPeer потому что нужно дата создания последнего пира.
    _callInternal и processOnCall не подходят для инициализации.
    processOnCall - циклично вызывается,
    _callInternal - инициализуется у инициатора звонка;
  3. fixed
  4. fixed
  5. Метод processOnNotAnswer закрывает сессию и очищает таймер.
    http://joxi.ru/Drlaw0pUpVbWmP

soulfly pushed a commit that referenced this pull request Oct 20, 2015
@soulfly soulfly merged commit 531894d into develop.webrtc Oct 20, 2015
@soulfly soulfly deleted the develop.webrtc.QBWEBSDK-124_v2 branch October 20, 2015 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants