tree 8782d5326e5ae26330c89898b83901c4f8e1fcc7
parent abe3e1a6977d09313b0f7afbd74f289aeb20a72a
author Viet-Trung Luu <viettrungluu@chromium.org> 1463528596 -0700
committer Viet-Trung Luu <viettrungluu@chromium.org> 1463528596 -0700

Factor stuff from ApplicationImpl out to a new class, ApplicationImplBase.

ApplicationImpl is now a rather dumb class that just calls its
ApplicationDelegate.

Using base/implementation classes is preferable to the delegate pattern
because:
* It's simpler (see below).
* With the delgate pattern, the impl and the delegate both need pointers
  to each other. And so there are lifetime problems/considerations.
* The delegate just ends up calling into the impl a lot.

Unfortunately, it's rather hard (impossible) to use ApplicationImplBase
right now (as intended): I'll need to refactor
ApplicationRunner(Chromium) to not just instantiate ApplicationImpl,
etc.

(I'll also need a story for the crazy that is
ApplicationImpl(Base)::Terminate().)

R=vardhan@google.com

Review URL: https://codereview.chromium.org/1985223003 .