tree 1b1fd181c532482281ae57dcfe9a384af69c8185
parent a53392444f1c4b4612f8413917d96ae8765bc35a
author Santos Cordon <santoscordon@google.com> 1363634118 -0700
committer Santos Cordon <santoscordon@google.com> 1364593600 -0700

Update callLog on a failed outgoing call.

Did some calllog code consolidation into new CallLogger class.
Used new class to provide functionality into CallController which is
responsible for placing the call and reacting to failed outgoing calls.

It's worth noting that the consolidation wasn't necessary for the bug
fix, but it's a good change nonetheless.

The merging of the two codepaths in CallNotifier is hard to trace, but
looking through the execution path, both inevitably did the same thing
except for one case: We didn't previously call stripSeparators for the
CdmaCallWaitingReject calls.  It isn't necessary since we
would never find those separators coming from direct cdma calls. Having
that extra call now doesn't hurt since it is a no-op in that case.  It
should be the only difference between the two (thus merging made sense).

bug:7614900
Change-Id: Iac50cba53f4312715c803c2f375baed697c678a4