CallbackPromiseAdapter types should be more compatible with WebCallbacks (2/3).

CallbackPromiseAdapter types should be more compatible with WebCallbacks (2/3).

This series of CLs makes CallbackPromiseAdapter (CPA) more compatible with
WebCallbacks (WC) in terms of type parameters.

Before this CL, CPA<S, T> corresponded to WC<S::WebType*, T::WebType*>. CPAs
took ownership of passed pointers but there was no place to represent that in
WebCallbacks.

This series of CLs changes that: CPA<S, T> correspond to WC<S::WebType,
T::WebType>. CPA users can specify if he/she wants the parameter ownership by
specifying the type parameter. For example, setting S::WebType to OwnPtr<X>
means it takes ownership. Setting S::WebType to |const X&| means it doesn't
take ownership.
WebCallbacks is exposed to chromium side, so we use WebPassOwnPtr as the
counterpart of PassOwnPtr.

[1] https://codereview.chromium.org/1234603003/
[2] This CL.
[3] https://codereview.chromium.org/1240763002/

BUG=493531
Committed: https://crrev.com/7f862b475953a0d6a1af42f27adf38062fe11b10
Cr-Commit-Position: refs/heads/master@{#342577}

[yhirano, 2015-07-15 06:18:28]

yhirano@chromium.org changed reviewers:
+ kinuko@chromium.org

[kinuko, 2015-07-26 15:15:18]

lgtm

[yhirano, 2015-07-27 12:37:27]

yhirano@chromium.org changed reviewers:
+ mek@chromium.org, mlamouri@chromium.org, mvanouwerkerk@chromium.org,
peter@chromium.org, scheib@chromium.org

[yhirano, 2015-07-27 12:38:57]

PTAL.

mlamouri@ for app_banner, presentation
mvanouwerkerk@ for push_messaging
scheib@ for bluetooth
mek@ for geofencing.
peter@ for notifications

[Peter Beverloo, 2015-07-27 12:53:01]

Ooh nice :)

push and notifications lgtm

[Marijn Kruisselbrink, 2015-07-27 16:58:37]

geofencing LGTM

[scheib, 2015-07-27 17:29:25]

bluetooth LGTM

[Michael van Ouwerkerk, 2015-07-28 10:14:58]

push_messaging lgtm

[yhirano, 2015-07-29 04:05:00]

yhirano@chromium.org changed reviewers:
+ avayvod@chromium.org
- mlamouri@chromium.org

[yhirano, 2015-07-29 04:05:00]

I reverted app_banner related changes because it no longer uses
CallbackPromiseAdapter.
avavyvod@, can you take a look at the presentation change?

[yhirano, 2015-08-04 03:36:39]

On 2015/07/29 04:05:00, yhirano wrote:
> I reverted app_banner related changes because it no longer uses
> CallbackPromiseAdapter.
> avavyvod@, can you take a look at the presentation change?

ping

[whywhat, 2015-08-05 12:45:42]

On 2015/08/04 at 03:36:39, yhirano wrote:
> On 2015/07/29 04:05:00, yhirano wrote:
> > I reverted app_banner related changes because it no longer uses
> > CallbackPromiseAdapter.
> > avavyvod@, can you take a look at the presentation change?
> 
> ping

presentation/ lgtm

[yhirano, 2015-08-07 11:12:11]

The CQ bit was checked by yhirano@chromium.org

[yhirano, 2015-08-07 11:12:11]

The patchset sent to the CQ was uploaded after l-g-t-m from peter@chromium.org,
kinuko@chromium.org, scheib@chromium.org, mvanouwerkerk@chromium.org,
mek@chromium.org, avayvod@chromium.org
Link to the patchset: https://codereview.chromium.org/1235083006/#ps100001
(title: " ")

[commit-bot: I haz the power, 2015-08-07 11:12:16]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1235083006/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1235083006/100001

[commit-bot: I haz the power, 2015-08-07 11:13:46]

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

[commit-bot: I haz the power, 2015-08-07 11:13:47]

Try jobs failed on following builders:
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[yhirano, 2015-08-10 01:32:04]

The CQ bit was checked by yhirano@chromium.org

[yhirano, 2015-08-10 01:32:05]

The patchset sent to the CQ was uploaded after l-g-t-m from peter@chromium.org,
kinuko@chromium.org, scheib@chromium.org, mvanouwerkerk@chromium.org,
avayvod@chromium.org, mek@chromium.org
Link to the patchset: https://codereview.chromium.org/1235083006/#ps140001
(title: " ")

[commit-bot: I haz the power, 2015-08-10 01:32:07]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1235083006/140001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1235083006/140001

[commit-bot: I haz the power, 2015-08-10 02:05:21]

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

[commit-bot: I haz the power, 2015-08-10 02:05:22]

Try jobs failed on following builders:
  win8_chromium_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_ng/...)

[yhirano, 2015-08-10 03:37:57]

The CQ bit was checked by yhirano@chromium.org

[yhirano, 2015-08-10 03:37:58]

The patchset sent to the CQ was uploaded after l-g-t-m from peter@chromium.org,
kinuko@chromium.org, scheib@chromium.org, mvanouwerkerk@chromium.org,
avayvod@chromium.org, mek@chromium.org
Link to the patchset: https://codereview.chromium.org/1235083006/#ps160001
(title: " ")

[commit-bot: I haz the power, 2015-08-10 03:38:00]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1235083006/160001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1235083006/160001

[commit-bot: I haz the power, 2015-08-10 04:29:13]

Committed patchset #9 (id:160001)

[commit-bot: I haz the power, 2015-08-10 04:29:50]

Patchset 9 (id:??) landed as
https://crrev.com/7f862b475953a0d6a1af42f27adf38062fe11b10
Cr-Commit-Position: refs/heads/master@{#342577}