Revert 238348 "Clean up TestProfileSyncService and related tests"

This CL appeared to be the cause of a link failure on Win dbg: http://goo.gl/PTFpFF

> Clean up TestProfileSyncService and related tests
> 
> This CL refactors many of the tests in profile_sync_service_unittest.cc.
> It continues the refactoring work begun in r233533, r235661, and
> r235854.
> 
> The JsController tests have been deleted.  There is not much point in
> testing the JsController here; it can be more easily tested on its own
> in sync_js_controller_uniittest.cc.  The SyncJsController unit tests
> have been updated so they now cover the few cases that were formerly
> only exercised in the ProfileSyncService unit tests.
> 
> It converts all remaining uncoverted tests from relying on the
> TestProfileSyncService to using a real ProfileSyncService with an
> injected backend.  The injected backend makes it easier to create the
> scenarios we want to test.  We can inject a specially crafted SBH rather
> than fiddling with "synchronous init" and "fail initial download" flags.
> 
> Since the TestProfileSyncService no longer needs to support the wide
> variety of test scenarios required by the tests in
> profile_sync_service_unittest.cc, we can greatly simplify its
> implementation.  Many of its parameters and associated code have been
> removed.
> 
> BUG=140354,312994
> 
> Review URL: https://codereview.chromium.org/67683005

TBR=rlarocque@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@238368 0039d316-1c4b-4281-b951-d872f2087c98
SVN changes up to revision 238368
[rAC, OSX] Fix key handling for autofill popup.

BUG=323728
TEST=see bug
R=isherman@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@238369 0039d316-1c4b-4281-b951-d872f2087c98
Add a no-op measurement

Some pages you just want to let run while the profiler runs. This
no_op measurement does just that.

R=tonyg
CC=eseidel

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@238370 0039d316-1c4b-4281-b951-d872f2087c98
SVN changes up to revision 238370
Lower logging level to avoid spamming the logs

@tapted: assigning you as a reviewer for the sake of mixing it up (see @miket's email re assigning new reviewers to CLs) and because it's a trivial change.

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@238371 0039d316-1c4b-4281-b951-d872f2087c98
Compile libaddressinput on Mac.

All new symbols need to be in an .order file on Mac but because libaddressinput
is built statically it doesn't need to export symbols at all, so remove this
from its gyp file.

R=rouslan@chromium.org
BUG=317837
TEST=mac compiles when we actually start using libaddressinput in chrome

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@238372 0039d316-1c4b-4281-b951-d872f2087c98
SVN changes up to revision 238372