917933 - [MP] Change - Use new Bing search code for Metro

Status: VERIFIED FIXED

(Firefox for Metro Graveyard :: General, defect, P1)

Description

[Brian R. Bondy [:bbondy]]

New code should be: 

> unique id:  MOZW

As per email from Asa.

Comment 1

[Brian R. Bondy [:bbondy]]

Attachment: Patch v1.

As per the forwarded email

Comment 2

[Jim Mathies [:jimm]]

Comment on attachment 806816 [details] [diff] [review]
Patch v1.

Curious if we can include Microsoft's name without any trademark gunk or anything. I really don't know. Might want to run that past Asa.

Comment 3

[Brian R. Bondy [:bbondy]]

Do you mean here?
> <Description>Bing. Search by Microsoft.</Description>

We do that already on the Desktop search plugin

Comment 4

[Brian R. Bondy [:bbondy]]

https://hg.mozilla.org/integration/fx-team/rev/5fccd41244a1

Comment 5

[Tim Taubert [:ttaubert] (inactive)]

https://hg.mozilla.org/mozilla-central/rev/5fccd41244a1

Comment 6

[Manuela Muntean [Away]]

Could anyone please give any guidelines that can help the QA in verifying this issue? Thanks!

Comment 7

[Brian R. Bondy [:bbondy]]

When doing a seach in bing from the autocomplete screen, check the URL.
It should have form=MOZW.
Also check in Desktop firefox on the same build, that when you set the search plugin to bing, that it has form=MOZSBR or form=MOZLBR

Comment 8

[Manuela Muntean [Away]]

> When doing a seach in bing from the autocomplete screen, check the URL.
> It should have form=MOZW.

Verified as fixed, for iteration #15, with latest Nightly (build ID: 20130927030202), on Win 8 64-bit. 
After typing something in the URL bar and clicking on Bing to perform the search, I can see "form=MOZW" in the new opened URL, with the search results.

> Also check in Desktop firefox on the same build, that when you set the
> search plugin to bing, that it has form=MOZSBR or form=MOZLBR

Indeed, I can see "form=MOZLBR" in the URL bar, after setting Bing to be my search engine from the search bar and performing a search from the awesomebar.

Thanks Brian for the info!