991111 - Middle clicks on tiles are not counted

Status: VERIFIED FIXED

(Tracking Graveyard :: Firefox Operations, defect)

Description

[Paul Silaghi, QA [:pauly]]

31.0a1 (2014-04-01), win 7 x64
Tiles opened with mouse middle click button (in new tabs) are not counted, NEWTAB_PAGE_DIRECTORY_TYPE_CLICKED doesn't change.

Note: CTRL+left click on tiles (to open them in new tabs) is counted.

Comment 1

[Bryan Clark (DevTools PM) [:clarkbw]]

Ed can you take a look at this?

Comment 2

[Ed Lee :Mardak]

pauly, does "NEWTAB_PAGE_SITE_CLICKED" record that the position was clicked with a middle click?

Comment 3

[Ed Lee :Mardak]

Turns out "click" is not triggered but "mousedown" and "mouseup" do get triggered with event.button == 1. Adding the listener directly to the thumbnail or setting true useCapture or wantsUntrusted don't result in "click" happening on middle click.

Anyone with ideas why? It might be related to the xul/xhtml mix?

Comment 4

[Ed Lee :Mardak]

If I add the "click" event listener to document.documentElement (xul:window) or use window.addEventListener, the middle-click "click" event is correctly triggered. All the node ancestors between the xul:window and span.newtab-thumbnail are xhtml.

Comment 5

[Ed Lee :Mardak]

Attachment: v1

Comment 6

[Paul Silaghi, QA [:pauly]]

(In reply to Ed Lee :Mardak from comment #2)
> pauly, does "NEWTAB_PAGE_SITE_CLICKED" record that the position was clicked
> with a middle click?
Nope

Comment 7

[Paul Silaghi, QA [:pauly]]

I no longer see the tiles on the NTP in nightly 31.0a1 (2014-04-14). What happened?

Comment 8

[Drew Willcoxon :adw]

Comment on attachment 8406583 [details] [diff] [review]
v1

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

Thanks for the test.

Comment 9

[Drew Willcoxon :adw]

(In reply to Paul Silaghi, QA [:pauly] from comment #7)
> I no longer see the tiles on the NTP in nightly 31.0a1 (2014-04-14). What
> happened?

Bug 993581 probably.

Comment 10

[Ed Lee :Mardak]

https://hg.mozilla.org/integration/fx-team/rev/578a88956f7d

Comment 11

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/578a88956f7d

Comment 12

[Paul Silaghi, QA [:pauly]]

(In reply to Drew Willcoxon :adw from comment #9)
> (In reply to Paul Silaghi, QA [:pauly] from comment #7)
> > I no longer see the tiles on the NTP in nightly 31.0a1 (2014-04-14). What
> > happened?
> 
> Bug 993581 probably.
Is there a way to verify this bug, considering the above?

Comment 13

[Ed Lee :Mardak]

You can set "browser.newtabpage.directorySource" to "chrome://global/content/directoryLinks.json"

Or you can just look for NEWTAB_PAGE_SITE_CLICKED per comment 6.

Comment 14

[Paul Silaghi, QA [:pauly]]

Ok, the middle clicks are counted now.
But the right clicks are also counted (even if don't choose anything from the context menu). Is this expected ?

Comment 15

[Ed Lee :Mardak]

(In reply to Paul Silaghi, QA [:pauly] from comment #14)
> But the right clicks are also counted (even if don't choose anything from
> the context menu). Is this expected ?
I don't think so... I filed a related bug 998387, and it'll be fixed by bug 998163. Although there's a deeper bug/question of if we want/should be counting link visits from the context menu.

Comment 16

[Marco Mucci [:MarcoM]]

Hi Paul, confirming if this resolved bug can be verified before the end of the desktop iteration on Monday April 28?

Comment 17

[Paul Silaghi, QA [:pauly]]

(In reply to Ed Lee :Mardak from comment #15)
> (In reply to Paul Silaghi, QA [:pauly] from comment #14)
> > But the right clicks are also counted (even if don't choose anything from
> > the context menu). Is this expected ?
> I don't think so... I filed a related bug 998387, and it'll be fixed by bug
> 998163. Although there's a deeper bug/question of if we want/should be
> counting link visits from the context menu.
31.0a1 (2014-04-21)
Right clicks are no longer counted, middle click on the pin/close tile buttons doesn't have any effect, right click/open link in a new tab is also not counted

(In reply to Marco Mucci [:MarcoM] from comment #16)
> Hi Paul, confirming if this resolved bug can be verified before the end of
> the desktop iteration on Monday April 28?
Done. Verified fixed

Comment 18

[Paul Silaghi, QA [:pauly]]

(In reply to Ed Lee :Mardak from comment #15)
> Although there's a deeper bug/question of if we want/should be
> counting link visits from the context menu.
Please say if you want me to file a new bug for this