Drop all uses of //base except ObserverList from view mgr client lib

Drop all uses of //base except ObserverList from view mgr client lib

This removes all uses of //base from the view manager client C++ library
interface and implementation except for ObserverList. The tests still
use //base fairly extensively to set up message loops and whatnot.  Main
changes:

*) use int64_t + <stdint.h> instead of //base/basictypes.h + int64. This
is what //base/basictypes.h recommends doing anyway *) use lambdas to
construct callbacks instead of base::Bind. We were using
base::Unretained() anyway so bind's lifetime stuff didn't help us.  *)
Hand-expand the one use of DISALLOW_COPY_AND_ASSIGN *) Remove the one
used of scoped_ptr in the ViewManagerClientFactory variant used by the
window manager. This one is a bit unfortunate in that it needs a raw
pointer but we should be able to update it to std::unique_ptr<>
relatively soon and it's only used in two places (the window manager
common lib and one test).

The ObserverList dependency is a bit bigger.

R=sky@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/3aefd1bccae360f76b91162edb8de8698e34e98b

[viettrungluu, 2015-02-10 22:47:10]

viettrungluu@chromium.org changed reviewers:
+ viettrungluu@chromium.org

[viettrungluu, 2015-02-10 22:47:11]

FWIW, you can use MOJO_DISALLOW_COPY_AND_ASSIGN if you want.

(And I think you missed various DISALLOW_COPY_AND_ASSIGNs.)

https://codereview.chromium.org/911073005/diff/1/mojo/services/view_manager/p...
File mojo/services/view_manager/public/cpp/lib/view_manager_client_impl.h
(right):

https://codereview.chromium.org/911073005/diff/1/mojo/services/view_manager/p...
mojo/services/view_manager/public/cpp/lib/view_manager_client_impl.h:161:
DISALLOW_COPY_AND_ASSIGN(ViewManagerClientImpl);
Hmmm?

[jamesr, 2015-02-10 22:48:36]

Ah, that's nicer.  That macro must be defined through other includes.

[sky, 2015-02-10 22:51:15]

LGTM

https://codereview.chromium.org/911073005/diff/1/mojo/services/view_manager/p...
File mojo/services/view_manager/public/cpp/view_tracker.h (right):

https://codereview.chromium.org/911073005/diff/1/mojo/services/view_manager/p...
mojo/services/view_manager/public/cpp/view_tracker.h:42: ViewTracker(const
ViewTracker&);
MOJO_DISALLOW_COPY_AND_ASSIGN?

[jamesr, 2015-02-11 00:24:08]

New patchsets have been uploaded after l-g-t-m from sky@chromium.org

[jamesr, 2015-02-11 01:11:50]

Committed patchset #3 (id:40001) manually as
3aefd1bccae360f76b91162edb8de8698e34e98b.