1287622 - Remove Cortana-related code from mozilla-central

Status: RESOLVED FIXED

(Firefox :: Search, defect, P4)

Description

[Jared Wein [:jaws] (please needinfo? me)]

Due to bug 1286832, Cortana search no longer works in Firefox. We should remove the code from the tree since it's dead.

Comment 1

[Justin Dolske [:Dolske]]

(To expand a bit, lest other bugs start getting duped to this one...)

Microsoft made a change to Windows 10 so that Cortana is now hard-coded to open search results only in Edge, and only with Bing. Previously, they respected your default browser choice, so that Firefox (or whatever your default was) would be launched, which could then use your default search engine. So this isn't a bug in Firefox, it's that Windows 10 will no longer send requests to Firefox in the first place.

Comment 2

[Justin Dolske [:Dolske]]

See also: http://www.pcworld.com/article/3063119/windows/microsoft-locks-down-cortana-blocks-integration-with-other-browsers-search-engines.html

Comment 3

[Justin Dolske [:Dolske]]

I believe Microsoft rolled out this change with the recent Anniversary Update of Windows 10.

Comment 4

[Marco Bonardo [:mak]]

can we make this a mentored bug? Code removal sounds like something a contributor can easily handle.

Comment 5

[Jared Wein [:jaws] (please needinfo? me)]

Good call, yes we can. I'll mentor it.

To fix this bug, we'll want to undo the changes from https://hg.mozilla.org/mozilla-central/rev/5d12635ce6be, https://hg.mozilla.org/mozilla-central/rev/c43d530f61c2, and https://hg.mozilla.org/mozilla-central/rev/7bc32b1953a1

Comment 6

[u579587]

Hi!

I'm at a Mozilla event looking for a first bug. I've a working build, yey :)

Can I work on this?

Thanks a bunch!

Comment 7

[Jared Wein [:jaws] (please needinfo? me)]

Yes, you may work on this. I will wait until you attach a patch before assigning the bug.

Comment 8

[u579587]

Attachment: Remove Cortana-related code from Firefox Search, as it no longer works in Firefox after Microsoft hard-coded search results to Edge

Comment 9

[u579587]

Attachment: Remove Cortana-related code from Firefox Search, as it no longer works in Firefox after Microsoft hard-coded search results to Edge

Comment 10

[u579587]

Thanks Jared! I'm winging this a lot. Can you check the patch?

Comment 11

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 8791737 [details] [diff] [review]
Remove Cortana-related code from Firefox Search, as it no longer works in Firefox after Microsoft hard-coded search results to Edge

Review of attachment 8791737 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good! Can you update the commit message to the following?
"Bug 1287622 - Remove Cortana-related code from Firefox as it no longer works after Microsoft hard-coded search results to Edge."

::: browser/app/profile/firefox.js
@@ -1517,4 @@
>  pref("browser.crashReports.unsubmittedCheck.enabled", true);
>  #endif
>  
> -pref("browser.crashReports.unsubmittedCheck.autoSubmit", false);
\ No newline at end of file

It looks like your patch added an extra newline at the end of this file.

Comment 12

[u579587]

Hey Jared! I'm a tad confused. We should have newlines at EOFs? Or did I add yet another? Because I can only the see the normal 1 newline at EOF. Sorry for being a newbie.

Also, I'm used to git, so basically I'm stuck trying to squash 2 commits and exporting a new patch.

Comment 13

[u579587]

Attachment: 0001-Bug-8791737-Remove-Cortana-related-code-from-Firefox.patch

Comment 14

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 8791961 [details] [diff] [review]
0001-Bug-8791737-Remove-Cortana-related-code-from-Firefox.patch

Review of attachment 8791961 [details] [diff] [review]:
-----------------------------------------------------------------

Yeah, only one newline at EOF. firefox.js already had a newline at the end of the file.

Squashing commits will work until you get a better hang of how to use Mercurial. I'll fix up the previous version of the patch and get it landed for you. I'll also see if I can find another bug for you to work on.

::: browser/app/profile/firefox.js
@@ -18,5 @@
> -#ifdef XP_UNIX
> -#ifndef XP_MACOSX
> -#define UNIX_BUT_NOT_MAC
> -#endif
> -#endif

These lines shouldn't be removed. The previous patch removed the "browser.search.redirectWindowsSearch" preference, which was correct.

Comment 15

[Jared Wein [:jaws] (please needinfo? me)]

(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #14)
> Comment on attachment 8791961 [details] [diff] [review]
> 0001-Bug-8791737-Remove-Cortana-related-code-from-Firefox.patch
> 
> Review of attachment 8791961 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Yeah, only one newline at EOF. firefox.js already had a newline at the end
> of the file.

I was wrong here, it looks like it didn't have a newline at the end of the file. I'm sorry for misleading you, your original change was fine.

Comment 16

[Jared Wein [:jaws] (please needinfo? me)]

Attachment: Patch for check-in

Comment 17

[Jared Wein [:jaws] (please needinfo? me)]

I'm marking this patch as checkin-needed, which means that it should get checked in to mozilla-inbound within the next day or two. After it is checked in, it will get merged to mozilla-central a day or two later. Once it reaches mozilla-central, it will appear in Firefox Nightly the next day.

Congrats on getting your first bug fixed!

Comment 18

[u579587]

(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #17)
> I'm marking this patch as checkin-needed, which means that it should get
> checked in to mozilla-inbound within the next day or two. After it is
> checked in, it will get merged to mozilla-central a day or two later. Once
> it reaches mozilla-central, it will appear in Firefox Nightly the next day.
> 
> Congrats on getting your first bug fixed!

Thanks a million Jared! I'm now working with gecko-dev from GitHub + exporting the patch with git. It's a more comfortable setup for now. Thanks for your mentoring!

Comment 19

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/fx-team/rev/fd5453b8f5b8
Remove Cortana-related code from Firefox as it no longer works after Microsoft hard-coded search results to Edge. r=jaws

Comment 20

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/fd5453b8f5b8