[zoom bubble] First part of speculative fix for zoom bubble crashes.

[zoom bubble] First part of speculative fix for zoom bubble crashes.

R=avi@chromium.org
BUG=144879
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=168086

[Dan Beam, 2012-11-14 06:06:38]

I still don't really think this makes a ton of sense to commit on trunk and the
branch considering how different the code is, but it doesn't hurt to put these
CHECK()s temporarily in both places.

[Dan Beam, 2012-11-14 06:07:23]

https://chromiumcodereview.appspot.com/11359205/diff/1/chrome/browser/ui/zoom...
File chrome/browser/ui/zoom/OWNERS (right):

https://chromiumcodereview.appspot.com/11359205/diff/1/chrome/browser/ui/zoom...
chrome/browser/ui/zoom/OWNERS:1: dbeam@chromium.org
I created this so people know who to blame for this and who to file bugs
against.

[Avi (use Gerrit), 2012-11-14 17:20:34]

Uh, OK. I thought we were suspecting a bogus delegate. I don't believe this is a
useful CHECK, but it's harmless.

lgtm

[Avi (use Gerrit), 2012-11-14 17:22:04]

I'm still waiting for a plausible explanation of how this being null would crash
in the way we're seeing. I'd call this a wild chase for an answer rather than a
speculative fix, as this fixes nothing.

[commit-bot: I haz the power, 2012-11-14 23:59:00]

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

[commit-bot: I haz the power, 2012-11-15 07:34:08]

Retried try job too often for step(s) sync_integration_tests

[commit-bot: I haz the power, 2012-11-15 08:25:08]

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