ref: Remove a redundant call to updateEstimate().

[bgrozev]

No description provided.

[bgrozev]

There's a comment about reducing how often we call updateEstimate, and now it's called twice for every packet reported in TCC: once in GoogCcEstimator.doProcessPacketArrival and once inside updateReceiverBlock. I'm not sure which one it makes more sense to keep.
cc @JonathanLennox

edit: I just noticed there is an early return in the one in updateReceiverBlock, so we should keep the one outside (like in the PR).

[JonathanLennox]

I think a better way to reduce the number of times we call updateEstimate() would be to implement the feedbackComplete part of my old PR #155 -- right now we call updateEstimate for each packet reported on in the TCC feedback.

I can look at that, or you can if you want, but in the meantime, this is probably fine.

[JonathanLennox]

See my PR #291 for the code I mentioned.

I don't think that conflicts with this PR; we can and should do both.