Remove Sync JS generic request/reply framework

Remove Sync JS generic request/reply framework

With the conversion of GetAllNodes in r262193, there are no longer any
uses of the Sync JS framework's request + reply framework.

The generic framework was useful when we could guarantee that the
requests would be handled in order, in the SyncManager, and on the sync
thread, but new requirements have forced us to re-implement much of this
functionality in other ways.

Since no one uses this code, and no one plans to use this code, and it
depends on some semi-deprecated concepts (WeakHandle), the best course
of action seems to be to delete it.  We can always fetch it from SVN
history and clean it up if we find another use for it.

BUG=328606, 357821
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=263105

[rlarocque, 2014-04-09 20:30:43]

Please review.

[tim (not reviewing), 2014-04-10 02:23:33]

lgtm

[rlarocque, 2014-04-10 17:06:10]

The CQ bit was checked by rlarocque@chromium.org

[commit-bot: I haz the power, 2014-04-10 17:06:27]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rlarocque@chromium.org/231013003/20001

[commit-bot: I haz the power, 2014-04-10 18:00:25]

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

[commit-bot: I haz the power, 2014-04-10 18:00:25]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[rlarocque, 2014-04-10 18:05:00]

The CQ bit was checked by rlarocque@chromium.org

[commit-bot: I haz the power, 2014-04-10 18:05:07]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rlarocque@chromium.org/231013003/40001

[commit-bot: I haz the power, 2014-04-10 19:09:59]

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

[commit-bot: I haz the power, 2014-04-10 19:09:59]

Retried try job too often on win_rel for step(s) app_list_unittests,
ash_unittests, aura_unittests, base_unittests, browser_tests,
cacheinvalidation_unittests, cc_unittests, check_deps, check_deps2git,
chrome_elf_unittests, chromedriver_unittests, components_unittests,
compositor_unittests, content_browsertests, content_unittests, crypto_unittests,
events_unittests, gfx_unittests, google_apis_unittests, gpu_unittests,
installer_util_unittests, interactive_ui_tests, ipc_tests, jingle_unittests,
media_unittests, mini_installer_test, nacl_integration, net_unittests,
ppapi_unittests, printing_unittests, remoting_unittests, sql_unittests,
sync_integration_tests, sync_unit_tests, telemetry_perf_unittests,
telemetry_unittests, unit_tests, views_unittests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[rlarocque, 2014-04-10 19:58:35]

The CQ bit was checked by rlarocque@chromium.org

[commit-bot: I haz the power, 2014-04-10 19:58:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rlarocque@chromium.org/231013003/60001

[commit-bot: I haz the power, 2014-04-10 22:45:18]

Change committed as 263105