MD Settings: Remove unnecessary importHtml() calls in site-settings tests.

MD Settings: Remove unnecessary importHtml() calls in site-settings tests.

Load privacy_page.html as browserPreload in site settings tests to bring in
all necessary dependencies. Previously site settings loaded prefs.html which
does not bring in any of the dependencies needed for the test to run.
Then it programmatically loaded test deps, which is unnecessary.

BUG=673825
Review-Url: https://codereview.chromium.org/2673563002
Cr-Commit-Position: refs/heads/master@{#447817}
Committed: https://chromium.googlesource.com/chromium/src/+/68c7a441710ebec8b3143eca64f4eb92da482737

[dpapad, 2017-02-02 02:05:58]

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

[commit-bot: I haz the power, 2017-02-02 02:06:23]

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

[dpapad, 2017-02-02 02:18:12]

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

[commit-bot: I haz the power, 2017-02-02 02:18:26]

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

[dpapad, 2017-02-02 02:19:01]

Patchset #2 (id:20001) has been deleted

[dpapad, 2017-02-02 02:19:05]

Patchset #1 (id:1) has been deleted

[dpapad, 2017-02-02 02:19:52]

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

[dpapad, 2017-02-02 02:20:02]

Patchset #1 (id:40001) has been deleted

[commit-bot: I haz the power, 2017-02-02 02:20:22]

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

[dpapad, 2017-02-02 02:22:15]

Description was changed from

==========
MD Settings: Remove unnecessary importHtml() calls in site-settings tests.

BUG=673825
==========

to

==========
MD Settings: Remove unnecessary importHtml() calls in site-settings tests.

Load privacy_page.html as browserPreload in site settings tests to bring in
all necessary dependencies. Previously site settings loaded prefs.html which
does not bring in any of the dependencies needed for the test to run.
Then it programmatically loaded test deps, which is unnecessary.

BUG=673825
==========

[dpapad, 2017-02-02 02:22:25]

dpapad@chromium.org changed reviewers:
+ dbeam@chromium.org

[Dan Beam, 2017-02-02 03:57:17]

lgtm!

[commit-bot: I haz the power, 2017-02-02 04:57:34]

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

[commit-bot: I haz the power, 2017-02-02 04:57:35]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[dpapad, 2017-02-02 18:10:04]

The CQ bit was checked by dpapad@chromium.org

[commit-bot: I haz the power, 2017-02-02 18:10:22]

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-02-02 19:31:17]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1486059004349850,
"parent_rev": "d46123678f9f8e22c6210134bdbcfdb3e49b3bf2", "commit_rev":
"68c7a441710ebec8b3143eca64f4eb92da482737"}

[commit-bot: I haz the power, 2017-02-02 19:32:45]

Description was changed from

==========
MD Settings: Remove unnecessary importHtml() calls in site-settings tests.

Load privacy_page.html as browserPreload in site settings tests to bring in
all necessary dependencies. Previously site settings loaded prefs.html which
does not bring in any of the dependencies needed for the test to run.
Then it programmatically loaded test deps, which is unnecessary.

BUG=673825
==========

to

==========
MD Settings: Remove unnecessary importHtml() calls in site-settings tests.

Load privacy_page.html as browserPreload in site settings tests to bring in
all necessary dependencies. Previously site settings loaded prefs.html which
does not bring in any of the dependencies needed for the test to run.
Then it programmatically loaded test deps, which is unnecessary.

BUG=673825

Review-Url: https://codereview.chromium.org/2673563002
Cr-Commit-Position: refs/heads/master@{#447817}
Committed:
https://chromium.googlesource.com/chromium/src/+/68c7a441710ebec8b3143eca64f4...
==========

[commit-bot: I haz the power, 2017-02-02 19:32:46]

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