[omnibox] Return reference to ImageView instead of LocationIconView (2)

[omnibox] Return reference to ImageView instead of LocationIconView (2)

Found another place where we need to return a reference to the
ImageView, not the LocationIconView.

From the previous CL:

The latter is quite wide when we fill in the label half of it, which we
now do with site security level in some cases.  The bubble (in the bug
that is in the wrong place) anchors to the middle of whatever View you
return to it, so we really want to give it the padlock here.

BUG=650140
Committed: https://crrev.com/b34fa9422ed84323d7d20ebc9eb69bb946207e87
Cr-Commit-Position: refs/heads/master@{#422094}

[Kevin Bailey, 2016-09-29 15:10:03]

Description was changed from

==========
[omnibox] Return reference to ImageView instead of LocationIconView (2)

Found another place where we need to return a reference to the
ImageView, not the LocationIconView.

From the previous CL:

The latter is quite wide when we fill in the label half of it, which we
now do with site security level in some cases.  The bubble (in the bug
that is in the wrong place) anchors to the middle of whatever View you
return to it, so we really want to give it the padlock here.

BUG=650140
==========

to

==========
[omnibox] Return reference to ImageView instead of LocationIconView (2)

Found another place where we need to return a reference to the
ImageView, not the LocationIconView.

From the previous CL:

The latter is quite wide when we fill in the label half of it, which we
now do with site security level in some cases.  The bubble (in the bug
that is in the wrong place) anchors to the middle of whatever View you
return to it, so we really want to give it the padlock here.

BUG=650140
==========

[Kevin Bailey, 2016-09-29 15:10:04]

krb@chromium.org changed reviewers:
+ benwells@chromium.org

[Kevin Bailey, 2016-09-29 15:12:21]

On 2016/09/29 15:10:04, Kevin Bailey wrote:
> mailto:krb@chromium.org changed reviewers:
> + mailto:benwells@chromium.org

Hi Ben,

I found another place where we need to tweak what we return.
Can you take a look?

thx,
krb

[Kevin Bailey, 2016-09-29 15:16:49]

On 2016/09/29 15:12:21, Kevin Bailey wrote:
> On 2016/09/29 15:10:04, Kevin Bailey wrote:
> > mailto:krb@chromium.org changed reviewers:
> > + mailto:benwells@chromium.org
> 
> Hi Ben,
> 
> I found another place where we need to tweak what we return.
> Can you take a look?
> 
> thx,
> krb

oops, previous CL: codereview.chromium.org/2328983002/

[benwells, 2016-09-30 01:21:29]

lgtm

[Kevin Bailey, 2016-09-30 02:14:06]

The CQ bit was checked by krb@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-30 02:14:27]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-30 03:05:39]

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

[commit-bot: I haz the power, 2016-09-30 03:05:40]

Dry run: This issue passed the CQ dry run.

[Kevin Bailey, 2016-09-30 13:09:19]

The CQ bit was checked by krb@chromium.org

[commit-bot: I haz the power, 2016-09-30 13:09:35]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-30 13:14:52]

Description was changed from

==========
[omnibox] Return reference to ImageView instead of LocationIconView (2)

Found another place where we need to return a reference to the
ImageView, not the LocationIconView.

From the previous CL:

The latter is quite wide when we fill in the label half of it, which we
now do with site security level in some cases.  The bubble (in the bug
that is in the wrong place) anchors to the middle of whatever View you
return to it, so we really want to give it the padlock here.

BUG=650140
==========

to

==========
[omnibox] Return reference to ImageView instead of LocationIconView (2)

Found another place where we need to return a reference to the
ImageView, not the LocationIconView.

From the previous CL:

The latter is quite wide when we fill in the label half of it, which we
now do with site security level in some cases.  The bubble (in the bug
that is in the wrong place) anchors to the middle of whatever View you
return to it, so we really want to give it the padlock here.

BUG=650140
==========

[commit-bot: I haz the power, 2016-09-30 13:14:54]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-30 13:17:41]

Description was changed from

==========
[omnibox] Return reference to ImageView instead of LocationIconView (2)

Found another place where we need to return a reference to the
ImageView, not the LocationIconView.

From the previous CL:

The latter is quite wide when we fill in the label half of it, which we
now do with site security level in some cases.  The bubble (in the bug
that is in the wrong place) anchors to the middle of whatever View you
return to it, so we really want to give it the padlock here.

BUG=650140
==========

to

==========
[omnibox] Return reference to ImageView instead of LocationIconView (2)

Found another place where we need to return a reference to the
ImageView, not the LocationIconView.

From the previous CL:

The latter is quite wide when we fill in the label half of it, which we
now do with site security level in some cases.  The bubble (in the bug
that is in the wrong place) anchors to the middle of whatever View you
return to it, so we really want to give it the padlock here.

BUG=650140

Committed: https://crrev.com/b34fa9422ed84323d7d20ebc9eb69bb946207e87
Cr-Commit-Position: refs/heads/master@{#422094}
==========

[commit-bot: I haz the power, 2016-09-30 13:17:42]

Patchset 1 (id:??) landed as
https://crrev.com/b34fa9422ed84323d7d20ebc9eb69bb946207e87
Cr-Commit-Position: refs/heads/master@{#422094}