Add PepperAddressResolver.

Add PepperAddressResolver.

The new PepperAddressResolver provides AsyncResolverInterface
implementation to P2PTransportChannel and is used there to resolve
address of the STUN server. It's required after recent changes in
libjingle. Also remove STUN address resolution from PepperPortAllocator
as it is not needed there anymore.

BUG=401102
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=288549

[dcaiafa, 2014-08-08 19:45:21]

lgtm

https://codereview.chromium.org/452923005/diff/20001/remoting/client/plugin/p...
File remoting/client/plugin/pepper_address_resolver.cc (right):

https://codereview.chromium.org/452923005/diff/20001/remoting/client/plugin/p...
remoting/client/plugin/pepper_address_resolver.cc:59: delete this;
I assume that pp::CompletionCallbackFactory handles the cancelation, correct.
I.e., there is no risk of OnResolved getting called after Destroy()

[Sergey Ulanov, 2014-08-08 19:49:09]

https://codereview.chromium.org/452923005/diff/20001/remoting/client/plugin/p...
File remoting/client/plugin/pepper_address_resolver.cc (right):

https://codereview.chromium.org/452923005/diff/20001/remoting/client/plugin/p...
remoting/client/plugin/pepper_address_resolver.cc:59: delete this;
On 2014/08/08 19:45:20, dcaiafa wrote:
> I assume that pp::CompletionCallbackFactory handles the cancelation, correct.
> I.e., there is no risk of OnResolved getting called after Destroy()

Correct, that's the reason we need CompletionCallbackFactory.

[Sergey Ulanov, 2014-08-08 19:49:12]

The CQ bit was checked by sergeyu@chromium.org

[commit-bot: I haz the power, 2014-08-08 19:50:48]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sergeyu@chromium.org/452923005/20001

[commit-bot: I haz the power, 2014-08-09 00:33:50]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/39408)
  mac_chromium_compile_dbg on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_swarming on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[commit-bot: I haz the power, 2014-08-09 00:40:47]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-08-09 00:40:48]

Try jobs failed on following builders:
  mac_chromium_compile_dbg on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_swarming on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Sergey Ulanov, 2014-08-09 01:07:34]

The CQ bit was checked by sergeyu@chromium.org

[commit-bot: I haz the power, 2014-08-09 01:11:45]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sergeyu@chromium.org/452923005/40001

[commit-bot: I haz the power, 2014-08-09 10:20:37]

Change committed as 288549