Fix WeakPtrFactory member ordering in ui

Fix WeakPtrFactory member ordering in ui

Changing in the intialization order of WeakPtrFactory such that all
member variables should appear before the WeakPtrFactory to ensure
that any WeakPtrs to Controller are invalidated before its members
variable's destructors are executed, rendering them invalid.

BUG=303818
Committed: https://crrev.com/73b5ddebad0ec50c82e1551d77f34f9e5f3541e1
Cr-Commit-Position: refs/heads/master@{#296468}

[MRV, 2014-09-23 13:49:52]

mohan.reddy@samsung.com changed reviewers:
+ sky@chromium.org

[MRV, 2014-09-23 13:49:53]

Very few files left, after that we can incorporate changes for presubmit.

PTAL

[sky, 2014-09-23 21:40:40]

https://codereview.chromium.org/589413002/diff/1/ui/aura/client/screen_positi...
File ui/aura/client/screen_position_client.cc (right):

https://codereview.chromium.org/589413002/diff/1/ui/aura/client/screen_positi...
ui/aura/client/screen_position_client.cc:7: //#include
"ui/aura/window_event_dispatcher.h"
Fix this.

[MRV, 2014-09-24 02:47:45]

On 2014/09/23 21:40:40, sky wrote:
>
https://codereview.chromium.org/589413002/diff/1/ui/aura/client/screen_positi...
> File ui/aura/client/screen_position_client.cc (right):
> 
>
https://codereview.chromium.org/589413002/diff/1/ui/aura/client/screen_positi...
> ui/aura/client/screen_position_client.cc:7: //#include
> "ui/aura/window_event_dispatcher.h"
> Fix this.

Dear Mr. Sky, I have fixed the unwanted header inclusions.
PTAL

[MRV, 2014-09-24 02:47:57]

PTAL

https://codereview.chromium.org/589413002/diff/1/ui/aura/client/screen_positi...
File ui/aura/client/screen_position_client.cc (right):

https://codereview.chromium.org/589413002/diff/1/ui/aura/client/screen_positi...
ui/aura/client/screen_position_client.cc:7: //#include
"ui/aura/window_event_dispatcher.h"
On 2014/09/23 21:40:40, sky wrote:
> Fix this.

Done.

[sky, 2014-09-24 14:31:14]

LGTM

[MRV, 2014-09-24 17:17:06]

The CQ bit was checked by mohan.reddy@samsung.com

[commit-bot: I haz the power, 2014-09-24 17:18:14]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/589413002/20001

[commit-bot: I haz the power, 2014-09-24 18:22:01]

Committed patchset #2 (id:20001) as 0e3e8d7f6b654224ff7a00a9daa27fc00b63485a

[commit-bot: I haz the power, 2014-09-24 18:22:41]

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

[MRV, 2014-09-25 01:57:53]

On 2014/09/24 14:31:14, sky wrote:
> LGTM

Thanks mr.sky for lgtm.