Fix TextInputClientMac related crashes of Fullscreen RenderWidget

Fix TextInputClientMac related crashes of Fullscreen RenderWidget.

TextInputClientObsever should skip commands from fullscreen RenderWidgetHosts corresponding to plugins.

TextInputClientObserver assumes that the RenderWidget::GetWebWidget() always
returns a WebFrameWidget.

This is not true when the RenderWidget is a RenderWidgetFullScreen. Therefore,
pressing Ctrl+Command+D
on a fullscreen flash will lead to a DCHECK firing in dev builds as well as
crashes in current Canary.

This CL will avoid sending any IPCs from TextInputClientMac to the fullscreen
RenderWidget.

BUG=656525, 663384
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2422973003
Cr-Commit-Position: refs/heads/master@{#430842}
(cherry picked from commit aeb3c5c70a6ab7472e802d424c9c4a628e819283)

[EhsanK, 2016-11-16 07:05:08]

Description was changed from

==========
TextInputClientObsever should skip commands from fullscreen RenderWidgetHosts
corresponding to plugins.

TextInputClientObserver assumes that the RenderWidget::GetWebWidget() always
returns a WebFrameWidget.

This is not true when the RenderWidget is a RenderWidgetFullScreen. Therefore,
pressing Ctrl+Command+D
on a fullscreen flash will lead to a DCHECK firing in dev builds as well as
crashes in current Canary.

This CL will avoid sending any IPCs from TextInputClientMac to the fullscreen
RenderWidget.

BUG=656525, 663384

Review-Url: https://codereview.chromium.org/2422973003
Cr-Commit-Position: refs/heads/master@{#430842}
(cherry picked from commit aeb3c5c70a6ab7472e802d424c9c4a628e819283)
==========

to

==========
Fix TextInputClientMac related crashes of Fullscreen RenderWidget.

TextInputClientObsever should skip commands from fullscreen RenderWidgetHosts
corresponding to plugins.

TextInputClientObserver assumes that the RenderWidget::GetWebWidget() always
returns a WebFrameWidget.

This is not true when the RenderWidget is a RenderWidgetFullScreen. Therefore,
pressing Ctrl+Command+D
on a fullscreen flash will lead to a DCHECK firing in dev builds as well as
crashes in current Canary.

This CL will avoid sending any IPCs from TextInputClientMac to the fullscreen
RenderWidget.

BUG=656525, 663384
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2422973003
Cr-Commit-Position: refs/heads/master@{#430842}
(cherry picked from commit aeb3c5c70a6ab7472e802d424c9c4a628e819283)
==========

[EhsanK, 2016-11-16 07:06:44]

ekaramad@chromium.org changed reviewers:
+ avi@chromium.org

[EhsanK, 2016-11-16 07:06:44]

Avi,

Could you please take a quick look for sanity check? Thanks!

[Avi (use Gerrit), 2016-11-16 15:28:41]

lgtm for your cherrypick.

[EhsanK, 2016-11-16 21:35:35]

Thanks Avi. Tested locally and things seem green. Trying CQ.

[EhsanK, 2016-11-16 21:35:38]

The CQ bit was checked by ekaramad@chromium.org

[commit-bot: I haz the power, 2016-11-16 21:36:37]

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-11-16 21:41:36]

Description was changed from

==========
Fix TextInputClientMac related crashes of Fullscreen RenderWidget.

TextInputClientObsever should skip commands from fullscreen RenderWidgetHosts
corresponding to plugins.

TextInputClientObserver assumes that the RenderWidget::GetWebWidget() always
returns a WebFrameWidget.

This is not true when the RenderWidget is a RenderWidgetFullScreen. Therefore,
pressing Ctrl+Command+D
on a fullscreen flash will lead to a DCHECK firing in dev builds as well as
crashes in current Canary.

This CL will avoid sending any IPCs from TextInputClientMac to the fullscreen
RenderWidget.

BUG=656525, 663384
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2422973003
Cr-Commit-Position: refs/heads/master@{#430842}
(cherry picked from commit aeb3c5c70a6ab7472e802d424c9c4a628e819283)
==========

to

==========
Fix TextInputClientMac related crashes of Fullscreen RenderWidget.

TextInputClientObsever should skip commands from fullscreen RenderWidgetHosts
corresponding to plugins.

TextInputClientObserver assumes that the RenderWidget::GetWebWidget() always
returns a WebFrameWidget.

This is not true when the RenderWidget is a RenderWidgetFullScreen. Therefore,
pressing Ctrl+Command+D
on a fullscreen flash will lead to a DCHECK firing in dev builds as well as
crashes in current Canary.

This CL will avoid sending any IPCs from TextInputClientMac to the fullscreen
RenderWidget.

BUG=656525, 663384
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2422973003
Cr-Commit-Position: refs/heads/master@{#430842}
(cherry picked from commit aeb3c5c70a6ab7472e802d424c9c4a628e819283)
==========

[commit-bot: I haz the power, 2016-11-16 21:41:37]

Committed patchset #1 (id:1)