Remove CanChallengePlatform() and CheckPlatformState().

Remove CanChallengePlatform() and CheckPlatformState().

This API did not meet its intended purpose.  We've decided to instead
just use a real challenge and wait for the response.

BUG=307707, 309158
TEST=compile, unittests pass.
TBR=pastarmovj

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=229648

[DaleCurtis, 2013-10-18 21:34:18]

pastarmovj, dkrahn: chrome/browser/chromeos/attestation/
tsepez: ppapi/proxy/ppapi_message.h
dmichael: Pepper changes.

[Darren Krahn, 2013-10-18 21:38:47]

On 2013/10/18 21:34:18, DaleCurtis wrote:
> pastarmovj, dkrahn: chrome/browser/chromeos/attestation/
> tsepez: ppapi/proxy/ppapi_message.h
> dmichael: Pepper changes.

This will conflict with https://chromiumcodereview.appspot.com/27399004/.  If it
ever makes it through the CQ it should be trivial to rebase.

[DaleCurtis, 2013-10-18 21:39:43]

Want to just drop that one in favor of this complete cleanup?

[Darren Krahn, 2013-10-18 21:41:37]

On 2013/10/18 21:39:43, DaleCurtis wrote:
> Want to just drop that one in favor of this complete cleanup?

Sure, if you want to pull in the additional changes.  There are some
dependencies only used by CheckPlatformState() that I also cleaned up in that
CL.

[DaleCurtis, 2013-10-18 21:45:46]

+tsepez for real for ppapi/proxy/ppapi_messages.h

dkrahn: Done, all pulled in.

[Tom Sepez, 2013-10-18 21:50:09]

Rubberstamp LGTM on ppapi_messages.h for deleting code.

[Darren Krahn, 2013-10-18 21:50:38]

On 2013/10/18 21:45:46, DaleCurtis wrote:
> +tsepez for real for ppapi/proxy/ppapi_messages.h
> 
> dkrahn: Done, all pulled in.

chrome/browser/chromeos/attestation: lgtm (unfortunately, I'm not a chrome
committer so you'll need a stamp from pastarmovj@)

[dmichael (off chromium), 2013-10-18 22:00:29]

https://chromiumcodereview.appspot.com/29943003/diff/70001/ppapi/api/private/...
File ppapi/api/private/ppb_platform_verification_private.idl (right):

https://chromiumcodereview.appspot.com/29943003/diff/70001/ppapi/api/private/...
ppapi/api/private/ppb_platform_verification_private.idl:14: M31 = 0.1,
Since you're not providing backwards compatibility, you can just delete this
line. You'll have to re-generate.

[DaleCurtis, 2013-10-18 22:07:13]

Since pastarmovj already approved the CL I pulled in a TBR should be fine.

https://chromiumcodereview.appspot.com/29943003/diff/70001/ppapi/api/private/...
File ppapi/api/private/ppb_platform_verification_private.idl (right):

https://chromiumcodereview.appspot.com/29943003/diff/70001/ppapi/api/private/...
ppapi/api/private/ppb_platform_verification_private.idl:14: M31 = 0.1,
On 2013/10/18 22:00:29, dmichael wrote:
> Since you're not providing backwards compatibility, you can just delete this
> line. You'll have to re-generate.

Done.

[dmichael (off chromium), 2013-10-18 22:12:21]

lgtm

[commit-bot: I haz the power, 2013-10-18 22:58:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dalecurtis@chromium.org/29943003/140001

[commit-bot: I haz the power, 2013-10-19 11:58:04]

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

[commit-bot: I haz the power, 2013-10-19 20:03:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dalecurtis@chromium.org/29943003/140001

[commit-bot: I haz the power, 2013-10-20 01:34:29]

Change committed as 229648