[webcrypto] Remove the mock implementation of WebCrypto.

[webcrypto] Remove the mock implementation of WebCrypto.

Instead let the layout tests run with WebCryptoImpl.

R=jamesr@chromium.org, jochen@chromium.org
TBR=joi
BUG=245025
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=225665

[eroman, 2013-09-26 00:59:57]

This is a dependency for https://codereview.chromium.org/24467004/.

My plan is to (1) disable the crypto layout tests (2) Land this CL (3) Land
https://codereview.chromium.org/24467004/ and re-enable the layouttests

[jamesr, 2013-09-26 01:01:12]

lgtm, although i have no content/shell powers.  adding jochen for that

[jochen (gone - plz use gerrit), 2013-09-26 20:19:37]

lgtm for content/shell

https://codereview.chromium.org/24702003/diff/1/content/renderer/renderer_web...
File content/renderer/renderer_webkitplatformsupport_impl.cc (left):

https://codereview.chromium.org/24702003/diff/1/content/renderer/renderer_web...
content/renderer/renderer_webkitplatformsupport_impl.cc:1044:
GetContentClient()->renderer()->OverrideWebCrypto();
can you rip out the OverrideWebCrypto() method from ContentRendererClient as
well?

[eroman, 2013-09-26 20:54:46]

https://codereview.chromium.org/24702003/diff/1/content/renderer/renderer_web...
File content/renderer/renderer_webkitplatformsupport_impl.cc (left):

https://codereview.chromium.org/24702003/diff/1/content/renderer/renderer_web...
content/renderer/renderer_webkitplatformsupport_impl.cc:1044:
GetContentClient()->renderer()->OverrideWebCrypto();
On 2013/09/26 20:19:37, jochen wrote:
> can you rip out the OverrideWebCrypto() method from ContentRendererClient as
> well?

Done. Thanks, that was an omission on my part.

[commit-bot: I haz the power, 2013-09-27 00:30:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eroman@chromium.org/24702003/6001

[commit-bot: I haz the power, 2013-09-27 01:20:14]

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

[eroman, 2013-09-27 01:30:31]

+joi for approval of content/public/renderer/*

(I will likely submit this TBR since it is removing interfaces not adding them)

[commit-bot: I haz the power, 2013-09-27 01:36:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eroman@chromium.org/24702003/6001

[commit-bot: I haz the power, 2013-09-27 03:48:26]

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

[commit-bot: I haz the power, 2013-09-27 04:09:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eroman@chromium.org/24702003/6001

[eroman, 2013-09-27 05:16:08]

Committed patchset #2 manually as r225665 (presubmit successful).

[Jói, 2013-09-27 10:49:14]

//content/public LGTM.