Note that this issue was fixed previously in this CL: https://codereview.chromium.org/2822383002/

Issue 722545 has been merged into this issue.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e65dd51e10accc80aa2be79129912f47357cb958

commit e65dd51e10accc80aa2be79129912f47357cb958
Author: googleo <googleo@chromium.org>
Date: Tue May 23 03:21:50 2017

Replace OnPageTranslate Observer by a responder of delegate.

In order to be notified that page is translated, we simply implement CompactInfobar as observer of Translate Driver.
But now we found from the translate button is clicked to the page is translated, there are many steps. If any step is returned, we are not be notified at all.

In order to fix the loose relationship, we create a responder inside the delegate.
So no matter what delegate is going to do, we will be notified, which keep infobar always consistent with delegate.

What's more, on some corner cases listed in bug/723426, the relationship between infobar and translate driver causes crash. And the new strong relationship will fix it.

BUG= 720164 , 703887 ,723426, 724428
TBR=dfalcantara@chromium.org

Review-Url: https://codereview.chromium.org/2894553002
Cr-Commit-Position: refs/heads/master@{#473799}

[modify] https://crrev.com/e65dd51e10accc80aa2be79129912f47357cb958/chrome/browser/ui/android/infobars/translate_compact_infobar.cc
[modify] https://crrev.com/e65dd51e10accc80aa2be79129912f47357cb958/chrome/browser/ui/android/infobars/translate_compact_infobar.h
[modify] https://crrev.com/e65dd51e10accc80aa2be79129912f47357cb958/components/translate/core/browser/translate_infobar_delegate.cc
[modify] https://crrev.com/e65dd51e10accc80aa2be79129912f47357cb958/components/translate/core/browser/translate_infobar_delegate.h

Should be fixed, Please help me verify it.

Thanks