Make it clear that WeakPtrFactory is the last data member

Make it clear that WeakPtrFactory is the last data member

Move declaration of AttachmentServiceProxyForTest's destructor to make
it clear the WeakPtrFactory is the last data member.

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

[anujsharma, 2014-09-18 08:07:43]

anujk.sharma@samsung.com changed reviewers:
+ maniscalco@chromium.org

[anujsharma, 2014-09-18 08:07:43]

PTAL

[maniscalco, 2014-09-18 16:52:01]

On 2014/09/18 08:07:43, anujsharma wrote:
> PTAL

LGTM.  Thanks for doing this.  BTW, I updated the change description to be more
accurate since the layout of data members isn't actually changing.

[anujsharma, 2014-09-19 02:38:49]

On 2014/09/18 16:52:01, maniscalco wrote:
> On 2014/09/18 08:07:43, anujsharma wrote:
> > PTAL
> 
> LGTM.  Thanks for doing this.  BTW, I updated the change description to be
more
> accurate since the layout of data members isn't actually changing.

Thanks maniscalco for LGTM.
Yes, this updated description looks more clear. Thanks for updating it. :)

[anujsharma, 2014-09-19 02:38:58]

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

[commit-bot: I haz the power, 2014-09-19 02:39:50]

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

[commit-bot: I haz the power, 2014-09-19 03:35:34]

Committed patchset #1 (id:1) as 9a420da4c093287733b0d0f1a4a796b06dc9ff27

[commit-bot: I haz the power, 2014-09-19 03:36:15]

Patchset 1 (id:??) landed as
https://crrev.com/ed1230ad1dc862fd9d5c2bbba01ca88e2563faca
Cr-Commit-Position: refs/heads/master@{#295652}