Fix uninit memory access in TranslateManagerRenderViewHostTests.

Fix uninit memory access in TranslateManagerRenderViewHostTests.

BUG=464837
Committed: https://crrev.com/63231b5edb0d74f7997924801698dda869899e6c
Cr-Commit-Position: refs/heads/master@{#319614}

[Lei Zhang, 2015-03-07 04:14:49]

thestig@chromium.org changed reviewers:
+ hajimehoshi@chromium.org

[Lei Zhang, 2015-03-07 04:17:56]

BTW, I ran TranslateManagerRenderViewHostTest locally under MSAN with this
patch. All tests pass.

[hajimehoshi, 2015-03-09 03:30:32]

lgtm, thanks

[hajimehoshi, 2015-03-09 03:32:24]

Sorry but not lgtm.

Can you do that on the constructor of LanguageDetectionDetails?

[Lei Zhang, 2015-03-09 03:34:50]

On 2015/03/09 03:32:24, hajimehoshi wrote:
> Sorry but not lgtm.
> 
> Can you do that on the constructor of LanguageDetectionDetails?

Sure, is is_cld_reliable = false the better default?

[hajimehoshi, 2015-03-09 03:49:15]

On 2015/03/09 03:34:50, Lei Zhang wrote:
> On 2015/03/09 03:32:24, hajimehoshi wrote:
> > Sorry but not lgtm.
> > 
> > Can you do that on the constructor of LanguageDetectionDetails?
> 
> Sure, is is_cld_reliable = false the better default?

Right.

[Lei Zhang, 2015-03-09 04:31:21]

See patch set 2.

[hajimehoshi, 2015-03-09 04:41:22]

lgtm, thanks!

[Lei Zhang, 2015-03-09 04:42:23]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2015-03-09 04:42:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/983283002/20001

[commit-bot: I haz the power, 2015-03-09 06:19:43]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-03-09 06:19:45]

Failed to apply the patch.

[commit-bot: I haz the power, 2015-03-09 06:20:00]

The CQ bit was unchecked by commit-bot@chromium.org

[Sergiy Byelozyorov, 2015-03-09 07:25:34]

The CQ bit was checked by sergiyb@chromium.org

[commit-bot: I haz the power, 2015-03-09 07:26:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/983283002/20001

[commit-bot: I haz the power, 2015-03-09 07:26:51]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-03-09 07:26:53]

Failed to apply the patch.

[commit-bot: I haz the power, 2015-03-09 07:27:08]

The CQ bit was unchecked by commit-bot@chromium.org

[Sergiy Byelozyorov, 2015-03-09 11:37:13]

The CQ bit was checked by sergiyb@chromium.org

[commit-bot: I haz the power, 2015-03-09 11:37:20]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/983283002/20001

[commit-bot: I haz the power, 2015-03-09 11:38:36]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-03-09 11:38:38]

Failed to commit the patch.

[commit-bot: I haz the power, 2015-03-09 11:38:49]

The CQ bit was unchecked by commit-bot@chromium.org

[Sergiy Byelozyorov, 2015-03-09 13:06:45]

The CQ bit was checked by sergiyb@chromium.org

[commit-bot: I haz the power, 2015-03-09 13:07:07]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/983283002/20001

[commit-bot: I haz the power, 2015-03-09 13:08:22]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-09 13:09:30]

Patchset 2 (id:??) landed as
https://crrev.com/63231b5edb0d74f7997924801698dda869899e6c
Cr-Commit-Position: refs/heads/master@{#319614}