Disable tests broken by WK r118001

Disable tests broken by WK r118001

BUG=129235
R=vandebo@chromium.org

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

[commit-bot: I haz the power, 2012-05-22 22:27:36]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jsbell@chromium.org/10414060/1

[commit-bot: I haz the power, 2012-05-22 22:56:55]

Try job failure for 10414060-1 (retry) on linux_chromeos for step "compile"
(clobber build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...

[vandebo (ex-Chrome), 2012-05-22 23:03:48]

https://chromiumcodereview.appspot.com/10414060/diff/9002/chrome/browser/brow...
File chrome/browser/browser_keyevents_browsertest.cc (right):

https://chromiumcodereview.appspot.com/10414060/diff/9002/chrome/browser/brow...
chrome/browser/browser_keyevents_browsertest.cc:296: #elif defined(OS_LINUX)
This stuff is more commonly done with a MAYBE pattern see for example
base/synchronization/waitable_event_watcher_unittest.cc:165

[jsbell, 2012-05-22 23:18:12]

https://chromiumcodereview.appspot.com/10414060/diff/9002/chrome/browser/brow...
File chrome/browser/browser_keyevents_browsertest.cc (right):

https://chromiumcodereview.appspot.com/10414060/diff/9002/chrome/browser/brow...
chrome/browser/browser_keyevents_browsertest.cc:296: #elif defined(OS_LINUX)
On 2012/05/22 23:03:49, vandebo wrote:
> This stuff is more commonly done with a MAYBE pattern

Yeah, initially I wanted a less intrusive change but you're right it's more
readable this way.

One more look?

[vandebo (ex-Chrome), 2012-05-22 23:20:35]

LGTM