1088540 - [e10s] Email link does not work

Status: VERIFIED FIXED

(Firefox :: Toolbars and Customization, defect)

Description

[martin]

Story
Open some Tabs & browse to some pages
Try to use items that need the current tab context
from the toolbar or "Additional Tools and Features" (in Toolbar)

Function will fail or miss the current displayed tab page context.

Tried built in:
Share This Page-> Result Nothing. Expected: Sharing selector should appear
Email Link -> Result Nothing      Expected: Default email program should start and show email with url

Tried Extension:
Nightly Testers Tool - Insert Build ID into textbox (when textbox is active in current Tab - Result: Dialog offering to copy to clipboard 
Expected Result: BuildID should be added to textbox

I suspect that all these issues are probably caused by the same root cause.

Reproducible: Always on BuildID Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:36.0) Gecko/20100101 Firefox/36.0 ID:20141023030203 CSet: 88adcf8fef83

Comment 2

[martin]

Don't know what changed but share this page works for me now on build ID
Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:36.0) Gecko/20100101 Firefox/36.0 ID:20141111030203 CSet: cbe6afcae26c

Works:
Share this Page (only tried with facebook)

Does not work:
Email Page (fails)
Error in browser console:
>
17:03:11.868 "[CustomizableUI]" TypeError: aWindow is null
Stack trace:
MailIntegration.sendLinkForWindow@chrome://browser/content/browser.js:15294:5
CustomizableWidgets<.onCommand@resource:///modules/CustomizableWidgets.jsm:943:7
CustomizableUIInternal.handleWidgetCommand@resource://app/modules/CustomizableUI.jsm:1362:11

Nightly Testers Tool still can not insert its string into (this) texbox.
<

Comment 3

[Neil Deakin]

This seems to be several bugs.

1. Sharing was fixing by bug 915547.
2. Email link is fairly simple to fix. I will attach a patch.
3. The Tester tools bug is essentially caused by 921935.

Comment 4

[Neil Deakin]

Attachment: Fix Email Link

Comment 5

[Neil Deakin]

Attachment: Fix Email Link

Oops, wrong patch

Comment 6

[:Gijs (he/him)]

Comment on attachment 8544700 [details] [diff] [review]
Fix Email Link

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

This misses CustomizableWidgets' implementation of the email link button (it did say "toolbar items" in the summary...).

It also seems to me that if we're updating this anyway, we should just pass gBrowser.selectedBrowser. The MailIntegration method can take a browser and use a sendMessage with the url being browser.currentURI.spec and the title being browser.contentTitle. That avoids the CPOW and doesn't need any additional message passing, AFAICT. Would that work?

Comment 7

[martin]

Nightly Testers Tool works now on:
Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:37.0) Gecko/20100101 Firefox/37.0 ID:20150106030201 CSet: 2a193b7f395c
Inserted the above string directyl into the text box. 
so one less issue to fix.

Comment 8

[martin]

Email Link Function works also on:
Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:37.0) Gecko/20100101 Firefox/37.0 ID:20150106030201 CSet: 2a193b7f395c

Ie: 
Click email Link Icon, Email Program opens with link pasted into draft email

Comment 9

[Neil Deakin]

Attachment: Fix Email Link, v2

Comment 10

[Marco Mucci [:MarcoM]]

Hi Neil, can you provide a point value.

Comment 11

[:Gijs (he/him)]

Comment on attachment 8545355 [details] [diff] [review]
Fix Email Link, v2

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

Ship it!

Comment 12

[Neil Deakin]

https://hg.mozilla.org/integration/mozilla-inbound/rev/e38e801acf7b

The tester tools issue still exists. I can file a separate bug.

Comment 13

[Carsten Book [:Tomcat]]

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

Comment 15

[martin]

@ comment 22 : Did you file a new bug for the testers tool? If so what bug number is it?

Just tested email link on Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:38.0) Gecko/20100101 Firefox/38.0 ID:20150123093302 CSet: a6bbabebed2f

Works for me now.

Comment 16

[Neil Deakin]

I filed bug 1125897 on the testers tool issue.

Comment 17

[Cornel Ionce [:noni] [Hubs QA]]

Confirming this is fixed using Firefox 37 beta 6 (build ID: 20150316202753) and latest Nightly, build ID: 20150318030202 on Windows 7 32-bit.