Temporarily adding timers to one WebRTC browsertest.

Temporarily adding timers to one WebRTC browsertest.

Perhaps then we can see where it gets stuck or slowed down on Android
bots.

BUG=387895
NOTRY=true

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

[phoglund_chromium, 2014-07-07 08:50:49]

The CQ bit was checked by phoglund@chromium.org

[commit-bot: I haz the power, 2014-07-07 08:51:58]

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

[tommi (sloooow) - chröme, 2014-07-07 09:35:07]

https://codereview.chromium.org/371013002/diff/1/content/browser/media/webrtc...
File content/browser/media/webrtc_getusermedia_browsertest.cc (right):

https://codereview.chromium.org/371013002/diff/1/content/browser/media/webrtc...
content/browser/media/webrtc_getusermedia_browsertest.cc:86: LOG(INFO) <<
"Entering teardown; " << timer_.Elapsed().InSeconds()
We don't use LOG(INFO) anymore.  VLOG(1)?

https://codereview.chromium.org/371013002/diff/1/content/browser/media/webrtc...
content/browser/media/webrtc_getusermedia_browsertest.cc:249: base::ElapsedTimer
timer_;
protected?

https://codereview.chromium.org/371013002/diff/1/content/browser/media/webrtc...
content/browser/media/webrtc_getusermedia_browsertest.cc:255: 
remove

[phoglund_chromium, 2014-07-07 09:48:40]

https://codereview.chromium.org/371013002/diff/1/content/browser/media/webrtc...
File content/browser/media/webrtc_getusermedia_browsertest.cc (right):

https://codereview.chromium.org/371013002/diff/1/content/browser/media/webrtc...
content/browser/media/webrtc_getusermedia_browsertest.cc:86: LOG(INFO) <<
"Entering teardown; " << timer_.Elapsed().InSeconds()
On 2014/07/07 09:35:06, tommi wrote:
> We don't use LOG(INFO) anymore.  VLOG(1)?

VLOG(1) logs don't seem to be visible by default in the bot logs unfortunately,
and they need to be to be useful (since the issue I'm analyzing only repros on
the bots)

https://codereview.chromium.org/371013002/diff/1/content/browser/media/webrtc...
content/browser/media/webrtc_getusermedia_browsertest.cc:249: base::ElapsedTimer
timer_;
On 2014/07/07 09:35:06, tommi wrote:
> protected?

Done.

https://codereview.chromium.org/371013002/diff/1/content/browser/media/webrtc...
content/browser/media/webrtc_getusermedia_browsertest.cc:255: 
On 2014/07/07 09:35:06, tommi wrote:
> remove

Done.

[tommi (sloooow) - chröme, 2014-07-07 10:07:22]

ok, lgtm

[phoglund_chromium, 2014-07-07 11:07:37]

The CQ bit was checked by phoglund@chromium.org

[commit-bot: I haz the power, 2014-07-07 11:08:09]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/phoglund@chromium.org/371013002/20001

[commit-bot: I haz the power, 2014-07-07 11:09:05]

Change committed as 281518

[eseidel, 2014-07-07 22:04:24]

This broke content_browsertests on at least win7:
https://build.chromium.org/p/chromium.webkit/builders/Win7%20(dbg)/builds/144...

[eseidel, 2014-07-07 22:04:49]

A revert of this CL has been created in
https://codereview.chromium.org/377763002/ by eseidel@chromium.org.

The reason for reverting is: Broke content_browsertests on win7:
https://build.chromium.org/p/chromium.webkit/builders/Win7%20(dbg)/builds/144....

[phoglund_chromium, 2014-07-08 07:34:21]

On 2014/07/07 22:04:49, eseidel wrote:
> A revert of this CL has been created in
> https://codereview.chromium.org/377763002/ by mailto:eseidel@chromium.org.
> 
> The reason for reverting is: Broke content_browsertests on win7:
>
https://build.chromium.org/p/chromium.webkit/builders/Win7%20(dbg)/builds/144....

Huh, that's weird. Anyway, thanks for the revert!