Early return in ::SetTooltipText in case GetView returns null

Early return in ::SetTooltipText in case GetView returns null

Driven by change: there is no point in doing the wrapping of
the tooltip text with RTL/LTR formatting, in case there is no view.
Simply bail out earlier.

BUG=

Review-Url: https://codereview.chromium.org/2611943003
Cr-Commit-Position: refs/heads/master@{#441784}
Committed: https://chromium.googlesource.com/chromium/src/+/b21270407f03c618ce6b187dacd743e359d998d0

[tonikitoo, 2017-01-04 20:30:05]

The CQ bit was checked by tonikitoo@igalia.com to run a CQ dry run

[commit-bot: I haz the power, 2017-01-04 20:30:21]

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

[tonikitoo, 2017-01-04 20:30:23]

Description was changed from

==========
Early return in ::SetTooltipText in case GetView returns null

Driven by change: there is no point in doing the wrapping of
the tooltip text with RTL/LTR formatting, in case there is no view.
Simply bail out earlier.

BUG=
==========

to

==========
Early return in ::SetTooltipText in case GetView returns null

Driven by change: there is no point in doing the wrapping of
the tooltip text with RTL/LTR formatting, in case there is no view.
Simply bail out earlier.

BUG=
==========

[tonikitoo, 2017-01-04 20:30:23]

tonikitoo@igalia.com changed reviewers:
+ sadrul@chromium.org

[tonikitoo, 2017-01-04 21:04:48]

PTAL

[commit-bot: I haz the power, 2017-01-04 21:45:00]

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

[commit-bot: I haz the power, 2017-01-04 21:45:01]

Dry run: This issue passed the CQ dry run.

[sadrul, 2017-01-05 21:01:16]

lgtm

[tonikitoo, 2017-01-05 21:02:48]

The CQ bit was checked by tonikitoo@igalia.com

[commit-bot: I haz the power, 2017-01-05 21:03:33]

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, 2017-01-06 01:21:51]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1483650168233250, "parent_rev":
"482c1f85b922eeec0b9525838baca4b884c27392", "commit_rev":
"b21270407f03c618ce6b187dacd743e359d998d0"}

[commit-bot: I haz the power, 2017-01-06 01:22:19]

Description was changed from

==========
Early return in ::SetTooltipText in case GetView returns null

Driven by change: there is no point in doing the wrapping of
the tooltip text with RTL/LTR formatting, in case there is no view.
Simply bail out earlier.

BUG=
==========

to

==========
Early return in ::SetTooltipText in case GetView returns null

Driven by change: there is no point in doing the wrapping of
the tooltip text with RTL/LTR formatting, in case there is no view.
Simply bail out earlier.

BUG=

Review-Url: https://codereview.chromium.org/2611943003
Cr-Commit-Position: refs/heads/master@{#441784}
Committed:
https://chromium.googlesource.com/chromium/src/+/b21270407f03c618ce6b187dacd7...
==========

[commit-bot: I haz the power, 2017-01-06 01:22:20]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/b21270407f03c618ce6b187dacd7...