Don't update progress when Frame::isLoading() is false

Don't update progress when Frame::isLoading() is false

This can lead to spurious progress change notifications, which can lead to underestimatation of progress, even at the time when the only page is complete.

This was regressed in https://codereview.chromium.org/1860743002

BUG=650781
Committed: https://crrev.com/aa70a46d3bfff89100ab5732812dbb7a8dac7953
Cr-Commit-Position: refs/heads/master@{#425178}

[Nate Chapin, 2016-10-13 19:21:32]

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

[commit-bot: I haz the power, 2016-10-13 19:21:55]

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

[Nate Chapin, 2016-10-13 20:04:23]

Description was changed from

==========
Don't update progress after loading complete

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

to

==========
Don't update progress when Frame::isLoading() is false

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

[Nate Chapin, 2016-10-13 20:23:16]

Description was changed from

==========
Don't update progress when Frame::isLoading() is false

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

to

==========
Don't update progress when Frame::isLoading() is false

This can lead to spurious progress change notifications, which can lead to
underestimatation of progress, even at the time when the only page is complete.

This was regressed in https://codereview.chromium.org/1860743002

BUG=650781
==========

[Nate Chapin, 2016-10-13 20:23:22]

japhet@chromium.org changed reviewers:
+ dcheng@chromium.org

[commit-bot: I haz the power, 2016-10-13 20:32:42]

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

[commit-bot: I haz the power, 2016-10-13 20:32:42]

Dry run: This issue passed the CQ dry run.

[dcheng, 2016-10-13 21:52:16]

lgtm

[Nate Chapin, 2016-10-13 21:52:45]

The CQ bit was checked by japhet@chromium.org

[commit-bot: I haz the power, 2016-10-13 21:53:16]

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-10-13 22:00:13]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-13 22:02:10]

Description was changed from

==========
Don't update progress when Frame::isLoading() is false

This can lead to spurious progress change notifications, which can lead to
underestimatation of progress, even at the time when the only page is complete.

This was regressed in https://codereview.chromium.org/1860743002

BUG=650781
==========

to

==========
Don't update progress when Frame::isLoading() is false

This can lead to spurious progress change notifications, which can lead to
underestimatation of progress, even at the time when the only page is complete.

This was regressed in https://codereview.chromium.org/1860743002

BUG=650781

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

[commit-bot: I haz the power, 2016-10-13 22:02:11]

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