Revert of Remove unneeded calls to plugin.SetParentVisible.

Revert of Remove unneeded calls to plugin.SetParentVisible. (patchset #3 id:40001 of https://codereview.chromium.org/2808723002/ )

Reason for revert:
Although no tests failed from this change, and it seems that pepper plugins do nothing for visibility changes:
https://cs.chromium.org/chromium/src/content/renderer/pepper/pepper_webplugin_impl.cc?l=206&rcl=0cc4c62a163318c06636916b7e2b2c6a56f16b74

I think that this is still important for browser_plugin:
https://cs.chromium.org/chromium/src/content/renderer/browser_plugin/browser_plugin.cc?l=427&rcl=b6dc62f53d4c3b14c0f541897f8d2970be89b84b

Original issue's description:
> Remove unneeded calls to plugin.SetParentVisible.
> 
> Plugin (WebPluginContainerImpl) does not keep its own state about
> visibility of parent, and once it is no longer inheriting from
> FrameViewBase, it will not implement any ParentVisible methods.
> If required, it could always call parent_.IsVisible.
> 
> BUG=637460
> 
> Review-Url: https://codereview.chromium.org/2808723002
> Cr-Commit-Position: refs/heads/master@{#463409}
> Committed: https://chromium.googlesource.com/chromium/src/+/e62eb16ec0d97fe0b59d39e3c5f9c0669e6d9cff

TBR=dcheng@chromium.org,haraken@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=637460
Review-Url: https://codereview.chromium.org/2809913004
Cr-Commit-Position: refs/heads/master@{#463953}
Committed: https://chromium.googlesource.com/chromium/src/+/db77d9c68e2df6c5d5c626455a19d9a1aef76598

[joelhockey, 2017-04-12 04:34:25]

The CQ bit was checked by joelhockey@chromium.org

[joelhockey, 2017-04-12 04:34:25]

Created Revert of Remove unneeded calls to plugin.SetParentVisible.

[commit-bot: I haz the power, 2017-04-12 04:34:39]

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-04-12 04:34:39]

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

[commit-bot: I haz the power, 2017-04-12 04:34:40]

No L-G-T-M from a valid reviewer yet.
CQ run can only be started once the patch has received an L-G-T-M from a full
committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,_not_ a
full super star committer.
Committers are members of the group "project-chromium-committers".
Note that this has nothing to do with OWNERS files.

[haraken, 2017-04-12 04:37:55]

LGTM

[joelhockey, 2017-04-12 04:53:17]

The CQ bit was checked by joelhockey@chromium.org

[commit-bot: I haz the power, 2017-04-12 04:53:38]

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

[dcheng, 2017-04-12 06:39:36]

lgtm

[joelhockey, 2017-04-12 07:15:47]

The CQ bit was unchecked by joelhockey@chromium.org

[joelhockey, 2017-04-12 07:15:49]

The CQ bit was checked by joelhockey@chromium.org

[commit-bot: I haz the power, 2017-04-12 07:15:57]

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-04-12 07:58:18]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491981349476840, "parent_rev":
"1f46c3f8167ab959fa80fa2e8b1d9a56e5e8bd46", "commit_rev":
"db77d9c68e2df6c5d5c626455a19d9a1aef76598"}

[commit-bot: I haz the power, 2017-04-12 07:59:07]

Description was changed from

==========
Revert of Remove unneeded calls to plugin.SetParentVisible. (patchset #3
id:40001 of https://codereview.chromium.org/2808723002/ )

Reason for revert:
Although no tests failed from this change, and it seems that pepper plugins do
nothing for visibility changes:
https://cs.chromium.org/chromium/src/content/renderer/pepper/pepper_webplugin...

I think that this is still important for browser_plugin:
https://cs.chromium.org/chromium/src/content/renderer/browser_plugin/browser_...

Original issue's description:
> Remove unneeded calls to plugin.SetParentVisible.
> 
> Plugin (WebPluginContainerImpl) does not keep its own state about
> visibility of parent, and once it is no longer inheriting from
> FrameViewBase, it will not implement any ParentVisible methods.
> If required, it could always call parent_.IsVisible.
> 
> BUG=637460
> 
> Review-Url: https://codereview.chromium.org/2808723002
> Cr-Commit-Position: refs/heads/master@{#463409}
> Committed:
https://chromium.googlesource.com/chromium/src/+/e62eb16ec0d97fe0b59d39e3c5f9...

TBR=dcheng@chromium.org,haraken@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=637460
==========

to

==========
Revert of Remove unneeded calls to plugin.SetParentVisible. (patchset #3
id:40001 of https://codereview.chromium.org/2808723002/ )

Reason for revert:
Although no tests failed from this change, and it seems that pepper plugins do
nothing for visibility changes:
https://cs.chromium.org/chromium/src/content/renderer/pepper/pepper_webplugin...

I think that this is still important for browser_plugin:
https://cs.chromium.org/chromium/src/content/renderer/browser_plugin/browser_...

Original issue's description:
> Remove unneeded calls to plugin.SetParentVisible.
> 
> Plugin (WebPluginContainerImpl) does not keep its own state about
> visibility of parent, and once it is no longer inheriting from
> FrameViewBase, it will not implement any ParentVisible methods.
> If required, it could always call parent_.IsVisible.
> 
> BUG=637460
> 
> Review-Url: https://codereview.chromium.org/2808723002
> Cr-Commit-Position: refs/heads/master@{#463409}
> Committed:
https://chromium.googlesource.com/chromium/src/+/e62eb16ec0d97fe0b59d39e3c5f9...

TBR=dcheng@chromium.org,haraken@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=637460

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

[commit-bot: I haz the power, 2017-04-12 07:59:08]

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