1371479 - Update search behavior about:newtab tests to work with activity-stream.enabled true and false

Status: RESOLVED FIXED

(Firefox Graveyard :: Activity Streams: General, enhancement)

Description

[Ed Lee :Mardak]

As part of https://github.com/mozilla/activity-stream/issues/2340 we can update the tests to work with both versions of `about:newtab`.

Comment 1

[Ed Lee :Mardak]

Attachment: Bug 1371479 - Update search behavior about:newtab tests to work with activity-stream.enabled true and false.

Update tests to select the search input for either about:newtab with a querySelector instead of specific id. Also removes gSearch checks that are always false anyway with the move to e10s contentSearch.

Review commit: https://reviewboard.mozilla.org/r/147342/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/147342/

Comment 2

[Ursula Sarracini (:ursula)]

Comment on attachment 8875938 [details]
Bug 1371479 - Update search behavior about:newtab tests to work with activity-stream.enabled true and false.

https://reviewboard.mozilla.org/r/147342/#review151830

Looks great, thanks!

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/407132c5b037
Update search behavior about:newtab tests to work with activity-stream.enabled true and false. r=ursula

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/407132c5b037