Declaring the weak_ptr_factory in proper order.

Declaring the weak_ptr_factory in proper order.

Cleaning up weak_ptr_factory destruction order in "src/ipc" module.
WeakPtrFactory should remain the last member so it'll be destroyed and
invalidate its weak pointers before any other members are destroyed.

BUG=303818
Committed: https://crrev.com/0184ced8d904d202ad7ab392ef8eca55d83d2937
Cr-Commit-Position: refs/heads/master@{#292334}

[anujsharma, 2014-08-27 09:25:35]

anujk.sharma@samsung.com changed reviewers:
+ l.gombos@samsung.com, sivagunturi@chromium.org, vivekg@chromium.org

[anujsharma, 2014-08-27 09:26:15]

On 2014/08/27 09:25:35, anujk.sharma wrote:
> mailto:anujk.sharma@samsung.com changed reviewers:
> + mailto:l.gombos@samsung.com, mailto:sivagunturi@chromium.org,
mailto:vivekg@chromium.org

PTAL

[anujsharma, 2014-08-28 03:18:51]

anujk.sharma@samsung.com changed reviewers:
+ dmichael@chromium.org, jam@chromium.org
- l.gombos@samsung.com, sivagunturi@chromium.org, vivekg@chromium.org

[anujsharma, 2014-08-28 03:18:51]

dmichael@chromium.org: Please review changes in src/ipc module

[anujsharma, 2014-08-28 03:20:53]

On 2014/08/28 03:18:51, anujsharma wrote:
> mailto:dmichael@chromium.org: Please review changes in src/ipc module

> mailto:jam@chromium.org: Please review changes in src/ipc module

PTAL

[jam, 2014-08-28 03:57:14]

lgtm

[anujsharma, 2014-08-28 04:10:45]

The CQ bit was checked by anujk.sharma@samsung.com

[anujsharma, 2014-08-28 04:10:57]

The CQ bit was unchecked by anujk.sharma@samsung.com

[anujsharma, 2014-08-28 04:11:06]

The CQ bit was checked by anujk.sharma@samsung.com

[commit-bot: I haz the power, 2014-08-28 04:11:20]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/anujk.sharma@samsung.com/508903002/1

[anujsharma, 2014-08-28 04:12:32]

On 2014/08/28 04:11:06, anujsharma wrote:
> The CQ bit was checked by mailto:anujk.sharma@samsung.com

@Thanks Jam for LGTM

[commit-bot: I haz the power, 2014-08-28 04:19:11]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/linux_gpu/builds/...)
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/47403)
  win_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu/builds/52758)
  android_aosp on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_aosp/bu...)
  android_arm64_dbg_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_arm64_d...)
  android_chromium_gn_compile_rel on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_chromiu...)
  android_clang_dbg_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_clang_d...)
  android_dbg_tests_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_dbg_tes...)
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
  linux_chromium_chromeos_rel_swarming on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_gn_rel on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_swarming on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  ios_dbg_simulator on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device/bu...)
  ios_rel_device_ninja on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  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_...)
  win8_chromium_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_rel...)
  win_chromium_compile_dbg on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)
  win_chromium_rel_swarming on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)
  win_chromium_x64_rel_swarming on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[commit-bot: I haz the power, 2014-08-28 04:20:51]

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

[commit-bot: I haz the power, 2014-08-28 04:20:52]

Try jobs failed on following builders:
  ios_rel_device on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device/bu...)
  ios_rel_device_ninja on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_rel_swarming on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[anujsharma, 2014-08-28 05:08:22]

On 2014/08/28 04:20:52, I haz the power (commit-bot) wrote:
> Try jobs failed on following builders:
>   ios_rel_device on tryserver.chromium.mac
>
(http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device/bu...)
>   ios_rel_device_ninja on tryserver.chromium.mac
>
(http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
>   mac_chromium_rel_swarming on tryserver.chromium.mac
>
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

Rebased as per the latest code.

[anujsharma, 2014-08-28 05:08:32]

The CQ bit was checked by anujk.sharma@samsung.com

[commit-bot: I haz the power, 2014-08-28 05:09:31]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/anujk.sharma@samsung.com/508903002/20001

[commit-bot: I haz the power, 2014-08-28 06:09:00]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_chromium_x64_rel_swarming on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[commit-bot: I haz the power, 2014-08-28 06:49:22]

Committed patchset #2 (id:20001) as 88c1a9834f64469b295157795047264a68c68ef5

[commit-bot: I haz the power, 2014-09-10 02:58:01]

Patchset 2 (id:??) landed as
https://crrev.com/0184ced8d904d202ad7ab392ef8eca55d83d2937
Cr-Commit-Position: refs/heads/master@{#292334}