Set WebPagePopupImpl's bounds even if they don't appear to have changed.

Set WebPagePopupImpl's bounds even if they don't appear to have changed.

In r389984 I tried to make WebPagePopupImpl::resize only call into
setWindowRect (which sends an IPC to the browser process) if the bounds have
actually changed. This uncovered a bug in Aura that was papered up by the fact
that we repeatedly set the bounds. The proper way to fix this is in Aura itself
but we'd like to merge this for release so I'm landing this quick fix to
restore the previous behavior.

I'm working on a proper fix in crrev.com/2228093003

BUG=633140
Committed: https://crrev.com/10dc8b7652e6d30337d9bea0ff8bba2592942751
Cr-Commit-Position: refs/heads/master@{#413288}

[bokan, 2016-08-19 21:23:10]

Description was changed from

==========
Set WebPagePopupImpl's bounds even if they don't appear to have changed.

In r389984 I tried to make WebPagePopupImpl::resize only call into
setWindowRect (which sends an IPC to the browser process) if the bounds have
actually changed. This uncovered a bug in Aura that was papered up by the fact
that we repeatedly set the bounds. The proper way to fix this is in Aura itself
but we'd like to merge this for release so I'm landing this quick fix to
restore the previous behavior.

I'm working on a proper fix in crrev.com/2228093003

BUG=633140
==========

to

==========
Set WebPagePopupImpl's bounds even if they don't appear to have changed.

In r389984 I tried to make WebPagePopupImpl::resize only call into
setWindowRect (which sends an IPC to the browser process) if the bounds have
actually changed. This uncovered a bug in Aura that was papered up by the fact
that we repeatedly set the bounds. The proper way to fix this is in Aura itself
but we'd like to merge this for release so I'm landing this quick fix to
restore the previous behavior.

I'm working on a proper fix in crrev.com/2228093003

BUG=633140
==========

[bokan, 2016-08-19 21:23:10]

bokan@chromium.org changed reviewers:
+ aelias@chromium.org

[bokan, 2016-08-19 21:23:40]

Alexandre, could you please take a look today? We'd like to merge this to M53
which is cutting soon.

[aelias_OOO_until_Jul13, 2016-08-19 21:29:19]

lgtm

[bokan, 2016-08-19 21:29:39]

The CQ bit was checked by bokan@chromium.org

[commit-bot: I haz the power, 2016-08-19 21:30:29]

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-08-19 23:45:37]

Description was changed from

==========
Set WebPagePopupImpl's bounds even if they don't appear to have changed.

In r389984 I tried to make WebPagePopupImpl::resize only call into
setWindowRect (which sends an IPC to the browser process) if the bounds have
actually changed. This uncovered a bug in Aura that was papered up by the fact
that we repeatedly set the bounds. The proper way to fix this is in Aura itself
but we'd like to merge this for release so I'm landing this quick fix to
restore the previous behavior.

I'm working on a proper fix in crrev.com/2228093003

BUG=633140
==========

to

==========
Set WebPagePopupImpl's bounds even if they don't appear to have changed.

In r389984 I tried to make WebPagePopupImpl::resize only call into
setWindowRect (which sends an IPC to the browser process) if the bounds have
actually changed. This uncovered a bug in Aura that was papered up by the fact
that we repeatedly set the bounds. The proper way to fix this is in Aura itself
but we'd like to merge this for release so I'm landing this quick fix to
restore the previous behavior.

I'm working on a proper fix in crrev.com/2228093003

BUG=633140
==========

[commit-bot: I haz the power, 2016-08-19 23:45:39]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-19 23:47:09]

Description was changed from

==========
Set WebPagePopupImpl's bounds even if they don't appear to have changed.

In r389984 I tried to make WebPagePopupImpl::resize only call into
setWindowRect (which sends an IPC to the browser process) if the bounds have
actually changed. This uncovered a bug in Aura that was papered up by the fact
that we repeatedly set the bounds. The proper way to fix this is in Aura itself
but we'd like to merge this for release so I'm landing this quick fix to
restore the previous behavior.

I'm working on a proper fix in crrev.com/2228093003

BUG=633140
==========

to

==========
Set WebPagePopupImpl's bounds even if they don't appear to have changed.

In r389984 I tried to make WebPagePopupImpl::resize only call into
setWindowRect (which sends an IPC to the browser process) if the bounds have
actually changed. This uncovered a bug in Aura that was papered up by the fact
that we repeatedly set the bounds. The proper way to fix this is in Aura itself
but we'd like to merge this for release so I'm landing this quick fix to
restore the previous behavior.

I'm working on a proper fix in crrev.com/2228093003

BUG=633140

Committed: https://crrev.com/10dc8b7652e6d30337d9bea0ff8bba2592942751
Cr-Commit-Position: refs/heads/master@{#413288}
==========

[commit-bot: I haz the power, 2016-08-19 23:47:10]

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