Remove all the indexeddb-related utility process code

Remove all the indexeddb-related utility process code

BUG=129471
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=152873

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=153336

[alecflett, 2012-08-17 19:02:36]

There may be more than this, I'm still cutting away at code, but thought you
guys would want to take a first pass - this is basically just pure code removal,
but if you have any suggestions for further removal...

[dgrogan, 2012-08-17 20:15:00]

lgtm

https://chromiumcodereview.appspot.com/10834350/diff/1001/content/content_tes...
File content/content_tests.gypi (right):

https://chromiumcodereview.appspot.com/10834350/diff/1001/content/content_tes...
content/content_tests.gypi:576: #       
'browser/indexed_db/idbbindingutilities_browsertest.cc',
What's the plan here?

[dgrogan, 2012-08-17 20:54:51]

lgtm

http://codereview.chromium.org/10834350/diff/1001/webkit/glue/idb_bindings.h
File webkit/glue/idb_bindings.h (right):

http://codereview.chromium.org/10834350/diff/1001/webkit/glue/idb_bindings.h#...
webkit/glue/idb_bindings.h:13: namespace webkit_glue {
Can you also remove idb_bindings.*?

[alecflett, 2012-08-21 18:37:22]

This next patch removes a little bit more - the empty files that dgrogan noticed
as well as outright removal of the idbbindingutilities_browsertest.

The story with that test: that one tests three things: the webkit/chromium api,
the ability to call into the utility process using the (now deleted) IDB utility
process client, and the actual injection/extraction. 

The first two parts are now irrelevant, so I am these over to the webkit
API/unit tests (WebKit/Source/WebKit/chromium/tests/IDB*) - hopefully that is
reasonably painless.

When that lands (or at least exists in patch form) in webkit, then this will be
ready to land.

[jsbell, 2012-08-21 18:53:13]

lgtm as well, this is awesome

https://chromiumcodereview.appspot.com/10834350/diff/1002/content/utility/uti...
File content/utility/utility_thread_impl.cc (right):

https://chromiumcodereview.appspot.com/10834350/diff/1002/content/utility/uti...
content/utility/utility_thread_impl.cc:14: #include
"content/common/indexed_db/indexed_db_key.h"
Can this include be removed?

https://chromiumcodereview.appspot.com/10834350/diff/1002/content/utility/uti...
content/utility/utility_thread_impl.cc:15: #include
"content/common/indexed_db/indexed_db_key_path.h"
Ditto?

[commit-bot: I haz the power, 2012-08-22 00:28:07]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/alecflett@chromium.org/10834350/9001

[commit-bot: I haz the power, 2012-08-22 00:28:27]

Presubmit check for 10834350-9001 failed and returned exit status 1.


Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    webkit/support
    content/utility
    content/browser
    content
    content/ppapi_plugin
    content/renderer
    webkit
    content/common

Presubmit checks took 1.8s to calculate.

[alecflett, 2012-08-22 00:37:49]

Darin/John - Mind a review? This is entirely code removal (yay!) now that I've
refactored IDB not to need the utility process anymore.

[jam, 2012-08-22 01:34:20]

lgtm

[alecflett, 2012-08-22 19:11:41]

oops, darin is on vacation - tony@ mind a review as an OWNER of webkit/ ?

[tony, 2012-08-22 20:51:03]

webkit lgtm

[commit-bot: I haz the power, 2012-08-22 20:58:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/alecflett@chromium.org/10834350/9001

[commit-bot: I haz the power, 2012-08-22 23:49:00]

Change committed as 152873

[commit-bot: I haz the power, 2012-08-24 18:32:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/alecflett@chromium.org/10834350/16003

[commit-bot: I haz the power, 2012-08-24 22:57:31]

Change committed as 153336