Eliminate BrowserProcess dependency from sign-in production code.

Eliminate BrowserProcess dependency from sign-in production code.

TBR=atwilson@chromium.org,phajdan.jr@chromium.org
BUG=233552
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=200931

[Roger Tawa OOO till Jul 10th, 2013-05-17 17:43:43]

lgtm

Curious Joi: I guess to get those lists in the DEPS file to zero, you'll need to
move some files out of chrome\browser\signin.  Which are are staying and which
ones are moving?

[Jói, 2013-05-17 17:50:15]

The ones I've identified so far as not moving are listed at the bottom
of the DEPS file (chrome_signin_manager_delegate.*,
signin_names_io_thread.*, signin_manager_factory.*), and browser tests
will also stay.  I haven't done an exhaustive analysis yet to find
whether there might be more files that should stay, but nothing else
pops out so far.

I'm planning to do this the way we've usually componentized, which is
to start by breaking the production code's dependencies and moving it
along with unit tests (which may still depend back on //chrome and be
built in //chrome/chrome_tests_unit.gypi), then breaking the unit
tests' dependencies.

Cheers,
Jói


On Fri, May 17, 2013 at 10:43 AM,  <rogerta@chromium.org> wrote:
> lgtm
>
> Curious Joi: I guess to get those lists in the DEPS file to zero, you'll
> need to
> move some files out of chrome\browser\signin.  Which are are staying and
> which
> ones are moving?
>
> https://codereview.chromium.org/15024007/

[commit-bot: I haz the power, 2013-05-17 18:02:46]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/joi@chromium.org/15024007/18001

[Jói, 2013-05-17 18:09:35]

TBR=atwilson@chromium.org,phajdan.jr@chromium.org for trivial usage updates.

Cheers,
Jói

[commit-bot: I haz the power, 2013-05-17 18:10:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/joi@chromium.org/15024007/18001

[Paweł Hajdan Jr., 2013-05-17 22:02:49]

chrome/test LGTM

[commit-bot: I haz the power, 2013-05-18 00:07:09]

Change committed as 200931