Cleanup blimp/client/core code organization.

Cleanup blimp/client/core code organization.

The //blimp/client/core directory had some files that were located
directly there without being in a sub-directory. This ended up being
confusing for other developers, so this CL cleans this up by adding a
//blimp/client/core/context directory. It also creates a directory for
the switches.

This CL also temporarily adds a common directory that both the context
classes and other code can depend on. That in turn can then of course
not depend on anything else in //blimp/client/core.

Lastly, it also cleans up the //blimp/client/core/settings directory
to now contain the Java-files it refers to.

In addition, the documentation is updated to include information about
this change, plus add information about the relatively new support
directory.

BUG=647848
Committed: https://crrev.com/5026f9cb132373a27d9660d8beab832dc90d7184
Cr-Commit-Position: refs/heads/master@{#421968}

[nyquist, 2016-09-27 01:24:58]

Description was changed from

==========
Cleanup blimp/client/core code organization.

The //blimp/client/core directory had some files that were located
directly there without being in a sub-directory. This ended up being
confusing for other developers, so this CL cleans this up by adding a
//blimp/client/core/context directory.

This CL also temporarily adds a common directory that both the context
classes and other code can depend on. That in turn can then of course
not depend on anything else in //blimp/client/core.

Lastly, it also cleans up the //blimp/client/core/settings directory
to now contain the Java-files it refers to.

In addition, the documentation is updated to include information about
this change, plus add information about the relatively new support
directory.

BUG=647848
==========

to

==========
Cleanup blimp/client/core code organization.

The //blimp/client/core directory had some files that were located
directly there without being in a sub-directory. This ended up being
confusing for other developers, so this CL cleans this up by adding a
//blimp/client/core/context directory. It also creates a directory for
the switches.

This CL also temporarily adds a common directory that both the context
classes and other code can depend on. That in turn can then of course
not depend on anything else in //blimp/client/core.

Lastly, it also cleans up the //blimp/client/core/settings directory
to now contain the Java-files it refers to.

In addition, the documentation is updated to include information about
this change, plus add information about the relatively new support
directory.

BUG=647848
==========

[nyquist, 2016-09-27 01:25:09]

The CQ bit was checked by nyquist@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-27 01:25:49]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-27 04:03:12]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-27 04:03:13]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[nyquist, 2016-09-27 06:54:33]

The CQ bit was checked by nyquist@chromium.org to run a CQ dry run

[nyquist, 2016-09-27 06:54:45]

nyquist@chromium.org changed reviewers:
+ dtrainor@chromium.org

[nyquist, 2016-09-27 06:54:46]

dtrainor: PTAL

[commit-bot: I haz the power, 2016-09-27 06:54:49]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-27 07:47:35]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-27 07:47:35]

Dry run: This issue passed the CQ dry run.

[David Trainor- moved to gerrit, 2016-09-29 20:03:51]

lgtm % one nit!

https://codereview.chromium.org/2376573002/diff/20001/blimp/client/core/conte...
File blimp/client/core/context/BUILD.gn (right):

https://codereview.chromium.org/2376573002/diff/20001/blimp/client/core/conte...
blimp/client/core/context/BUILD.gn:47: source_set("dummy_context") {
"dummy" might be more straightforward.  We're already in "context."  wdyt?

[nyquist, 2016-09-29 21:48:24]

https://codereview.chromium.org/2376573002/diff/20001/blimp/client/core/conte...
File blimp/client/core/context/BUILD.gn (right):

https://codereview.chromium.org/2376573002/diff/20001/blimp/client/core/conte...
blimp/client/core/context/BUILD.gn:47: source_set("dummy_context") {
On 2016/09/29 20:03:51, David Trainor wrote:
> "dummy" might be more straightforward.  We're already in "context."  wdyt?

I think yes.

[nyquist, 2016-09-29 21:48:29]

The CQ bit was checked by nyquist@chromium.org

[nyquist, 2016-09-29 21:48:30]

The patchset sent to the CQ was uploaded after l-g-t-m from
dtrainor@chromium.org
Link to the patchset: https://codereview.chromium.org/2376573002/#ps60001
(title: "merged origin/master")

[commit-bot: I haz the power, 2016-09-29 21:48:51]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-29 22:57:38]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-09-29 23:01:29]

Description was changed from

==========
Cleanup blimp/client/core code organization.

The //blimp/client/core directory had some files that were located
directly there without being in a sub-directory. This ended up being
confusing for other developers, so this CL cleans this up by adding a
//blimp/client/core/context directory. It also creates a directory for
the switches.

This CL also temporarily adds a common directory that both the context
classes and other code can depend on. That in turn can then of course
not depend on anything else in //blimp/client/core.

Lastly, it also cleans up the //blimp/client/core/settings directory
to now contain the Java-files it refers to.

In addition, the documentation is updated to include information about
this change, plus add information about the relatively new support
directory.

BUG=647848
==========

to

==========
Cleanup blimp/client/core code organization.

The //blimp/client/core directory had some files that were located
directly there without being in a sub-directory. This ended up being
confusing for other developers, so this CL cleans this up by adding a
//blimp/client/core/context directory. It also creates a directory for
the switches.

This CL also temporarily adds a common directory that both the context
classes and other code can depend on. That in turn can then of course
not depend on anything else in //blimp/client/core.

Lastly, it also cleans up the //blimp/client/core/settings directory
to now contain the Java-files it refers to.

In addition, the documentation is updated to include information about
this change, plus add information about the relatively new support
directory.

BUG=647848

Committed: https://crrev.com/5026f9cb132373a27d9660d8beab832dc90d7184
Cr-Commit-Position: refs/heads/master@{#421968}
==========

[commit-bot: I haz the power, 2016-09-29 23:01:31]

Patchset 4 (id:??) landed as
https://crrev.com/5026f9cb132373a27d9660d8beab832dc90d7184
Cr-Commit-Position: refs/heads/master@{#421968}