Revert of Remove unnecessary page loading in ContentShellActivity

Revert of Remove unnecessary page loading in ContentShellActivity (patchset #2 id:20001 of https://codereview.chromium.org/2199223002/ )

Reason for revert:
ContentViewZoomingTest.testJoystickZoomOut is now very flaky in almost all configurations, including on the main waterfall. crbug.com/635940

Original issue's description:
> Remove unnecessary page loading in ContentShellActivity
>
> In ContentShellActivity#onCreate, we load the page twice: one is the
> default page and the other is the target page.  The second loading can
> be avoided if we load the target page in the first loading. Thus, we can
> be sure that the target page will be ready once the page stops loading.
>
> assertWaitForPageScaleFactorMatch() used to be a workaround for the
> flaky bug crbug.com/179511 . The tests are not flaky anymore after
> applying this patch. Thus, we can remove it.
>
> BUG=580419, 179511
>
> Committed: https://crrev.com/797581eba513a69c80fbcbff43817b6fc6c6a9c0
> Cr-Commit-Position: refs/heads/master@{#410519}

TBR=changwan@chromium.org,aelias@chromium.org,sievers@chromium.org,yabinh@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=580419, 179511
Committed: https://crrev.com/a7c4fa85692c0201f91eaa82a4161ee2e4958201
Cr-Commit-Position: refs/heads/master@{#410677}

[jbudorick, 2016-08-09 15:28:24]

The CQ bit was checked by jbudorick@chromium.org

[jbudorick, 2016-08-09 15:28:24]

Created Revert of Remove unnecessary page loading in ContentShellActivity

[commit-bot: I haz the power, 2016-08-09 15:28:47]

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-09 15:29:52]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-09 15:33:18]

Description was changed from

==========
Revert of Remove unnecessary page loading in ContentShellActivity (patchset #2
id:20001 of https://codereview.chromium.org/2199223002/ )

Reason for revert:
ContentViewZoomingTest.testJoystickZoomOut is now very flaky in almost all
configurations, including on the main waterfall. crbug.com/635940

Original issue's description:
> Remove unnecessary page loading in ContentShellActivity
> 
> In ContentShellActivity#onCreate, we load the page twice: one is the
> default page and the other is the target page.  The second loading can
> be avoided if we load the target page in the first loading. Thus, we can
> be sure that the target page will be ready once the page stops loading.
> 
> assertWaitForPageScaleFactorMatch() used to be a workaround for the
> flaky bug crbug.com/179511 . The tests are not flaky anymore after
> applying this patch. Thus, we can remove it.
> 
> BUG=580419,179511
> 
> Committed: https://crrev.com/797581eba513a69c80fbcbff43817b6fc6c6a9c0
> Cr-Commit-Position: refs/heads/master@{#410519}

TBR=changwan@chromium.org,aelias@chromium.org,sievers@chromium.org,yabinh@chr...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=580419,179511
==========

to

==========
Revert of Remove unnecessary page loading in ContentShellActivity (patchset #2
id:20001 of https://codereview.chromium.org/2199223002/ )

Reason for revert:
ContentViewZoomingTest.testJoystickZoomOut is now very flaky in almost all
configurations, including on the main waterfall. crbug.com/635940

Original issue's description:
> Remove unnecessary page loading in ContentShellActivity
>
> In ContentShellActivity#onCreate, we load the page twice: one is the
> default page and the other is the target page.  The second loading can
> be avoided if we load the target page in the first loading. Thus, we can
> be sure that the target page will be ready once the page stops loading.
>
> assertWaitForPageScaleFactorMatch() used to be a workaround for the
> flaky bug crbug.com/179511 . The tests are not flaky anymore after
> applying this patch. Thus, we can remove it.
>
> BUG=580419,179511
>
> Committed: https://crrev.com/797581eba513a69c80fbcbff43817b6fc6c6a9c0
> Cr-Commit-Position: refs/heads/master@{#410519}

TBR=changwan@chromium.org,aelias@chromium.org,sievers@chromium.org,yabinh@chr...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=580419,179511

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

[commit-bot: I haz the power, 2016-08-09 15:33:19]

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