MD Settings: Add ImportDataDialog tests.

MD Settings: Add ImportDataDialog tests.

BUG=659259
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/346403640ca91d217eef9108ee5b03c4acfd5839
Cr-Commit-Position: refs/heads/master@{#432941}

[dpapad, 2016-11-16 23:50:46]

Description was changed from

==========
MD Settings: Add ImportDataDialog tests.

BUG=659259
==========

to

==========
MD Settings: Add ImportDataDialog tests.

BUG=659259
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[dpapad, 2016-11-16 23:53:13]

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

[commit-bot: I haz the power, 2016-11-16 23:54:19]

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

[dpapad, 2016-11-16 23:56:31]

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

[dpapad, 2016-11-16 23:56:50]

Patchset #1 (id:1) has been deleted

[dpapad, 2016-11-16 23:57:18]

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

[commit-bot: I haz the power, 2016-11-16 23:57:28]

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

[tommycli, 2016-11-17 17:04:28]

lgtm

https://codereview.chromium.org/2507253003/diff/60001/chrome/test/data/webui/...
File chrome/test/data/webui/settings/import_data_dialog_test.js (right):

https://codereview.chromium.org/2507253003/diff/60001/chrome/test/data/webui/...
chrome/test/data/webui/settings/import_data_dialog_test.js:57: }, {
nit... isn't normally the second { on the next line?

[dpapad, 2016-11-17 18:29:09]

https://codereview.chromium.org/2507253003/diff/60001/chrome/test/data/webui/...
File chrome/test/data/webui/settings/import_data_dialog_test.js (right):

https://codereview.chromium.org/2507253003/diff/60001/chrome/test/data/webui/...
chrome/test/data/webui/settings/import_data_dialog_test.js:57: }, {
On 2016/11/17 at 17:04:28, tommycli wrote:
> nit... isn't normally the second { on the next line?

Done.

[dpapad, 2016-11-17 18:29:12]

The CQ bit was checked by dpapad@chromium.org

[dpapad, 2016-11-17 18:29:12]

The patchset sent to the CQ was uploaded after l-g-t-m from
tommycli@chromium.org
Link to the patchset: https://codereview.chromium.org/2507253003/#ps40002
(title: "Nit.")

[commit-bot: I haz the power, 2016-11-17 18:29:51]

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-11-17 18:41:00]

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

[commit-bot: I haz the power, 2016-11-17 18:41:01]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Dan Beam, 2016-11-17 19:25:15]

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

[Dan Beam, 2016-11-17 19:25:15]

prefs_util.cc lgtm

[Dan Beam, 2016-11-17 19:25:26]

The CQ bit was checked by dbeam@chromium.org

[commit-bot: I haz the power, 2016-11-17 19:26:24]

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

[dpapad, 2016-11-17 19:26:31]

The CQ bit was unchecked by dpapad@chromium.org

[dpapad, 2016-11-17 19:26:36]

The CQ bit was checked by dpapad@chromium.org

[commit-bot: I haz the power, 2016-11-17 19:27:32]

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-11-17 20:13:22]

Committed patchset #4 (id:40002)

[commit-bot: I haz the power, 2016-11-17 20:29:06]

Description was changed from

==========
MD Settings: Add ImportDataDialog tests.

BUG=659259
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Settings: Add ImportDataDialog tests.

BUG=659259
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/346403640ca91d217eef9108ee5b03c4acfd5839
Cr-Commit-Position: refs/heads/master@{#432941}
==========

[commit-bot: I haz the power, 2016-11-17 20:29:08]

Patchset 4 (id:??) landed as
https://crrev.com/346403640ca91d217eef9108ee5b03c4acfd5839
Cr-Commit-Position: refs/heads/master@{#432941}