Revert of Revert of Fix the ppapi test for FlashDRM (patchset #1 id:1 of https:/...
commit0edd109f14d49b8580627bfbf13404d5523d9c3e
authorjackhou <jackhou@chromium.org>
Tue, 7 Oct 2014 03:22:50 +0000 (6 20:22 -0700)
committerCommit bot <commit-bot@chromium.org>
Tue, 7 Oct 2014 03:23:43 +0000 (7 03:23 +0000)
treee3919bc7388b15ffef7ca428f741c42678072651
parentd42a8f8372389ff80a2f0f55b8826318a6ed2c93
Revert of Revert of Fix the ppapi test for FlashDRM (patchset #1 id:1 of https://codereview.chromium.org/632133002/)

Reason for revert:
This CL was not the cause.

Actual fix here: https://codereview.chromium.org/630963003

Original issue's description:
> Revert of Fix the ppapi test for FlashDRM (patchset #1 id:1 of https://codereview.chromium.org/626883002/)
>
> Reason for revert:
> Suspecting this CL broke the following:
>
> https://build.chromium.org/p/chromium.memory/builders/Mac%20ASan%2064%20Tests%20%281%29/builds/2456
>
> Original issue's description:
> > Fix the ppapi test for FlashDRM
> >
> > This fixes the FlashDRM ppapi test which was failing because the test was assuming
> > that the "Hmonitor" value would never be returned on Mac, but at some point it
> > was implemented but the test was never changed. This changes the test to assume
> > that a valid Hmonitor value will be returned on Mac (but doesn't actually check
> > what the value is).
> >
> > BUG=419863
> >
> > Committed: https://crrev.com/5a4fc2503ca9d45c28c60723fdc71cba0f7ea968
> > Cr-Commit-Position: refs/heads/master@{#298298}
>
> TBR=bbudge@chromium.org,raymes@chromium.org
> NOTREECHECKS=true
> NOTRY=true
> BUG=419863
>
> Committed: https://crrev.com/886aeccd3824618309ccce5eabb2a70241073bc9
> Cr-Commit-Position: refs/heads/master@{#298316}

TBR=bbudge@chromium.org,raymes@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=419863

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

Cr-Commit-Position: refs/heads/master@{#298349}
chrome/test/ppapi/ppapi_browsertest.cc
ppapi/tests/test_flash_drm.cc