482359 - [IDE] Recorder is not able to recognize metadata of mouse clicks (right, middle, Ctrl...)

Status: RESOLVED WONTFIX

(Testing Graveyard :: Mozmill, defect, P2)

Description

[Henrik Skupin [:whimboo][⌚️UTC+2] (away 07/21 - 08/08)]

Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b4pre) Gecko/20090309 Shiretoko/3.1b4pre ID:20090309020654 and latest MozMill trunk

With the latest version of MozMill it is not possible to record a session where mouse clicks other than left clicks are used. The additional meta keys aren't recognized. All clicks end-up in left clicks.

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+2] (away 07/21 - 08/08)]

Damn, hit enter too early...

Here an example. Open http://www.mozilla.org and use the recorder to create a test which opens the properties dialog of the link which points to Mozilla.com in the grey box:

var mozmill = {}; Components.utils.import('resource://mozmill/modules/mozmill.js', mozmill);
var elementslib = {}; Components.utils.import('resource://mozmill/modules/elementslib.js', elementslib);

var setupModule = function(module) {
  controller = mozmill.getBrowserController();
}

var testRecorded = function () {
  controller.click(new elementslib.XPath(controller.tabs.activeTab, "/html/body[@id='www-mozilla-org']/div[@id='container']/div[@id='front-feature']/a"));
  controller.click(new elementslib.ID(controller.window.document, "context-metadata"));
}

Running this test makes a left click on the link and opens the linked URL.

Comment 2

[Mikeal Rogers [:mikeal] (mrogers@)]

I'm a little confused about the blocker addition to this bug.

If that is meant to keep track of what the tbird folks would like have in the next version's feature list then that's fine, but this bug shouldn't "block" any testing as you can still modify recorded clicks to have the right metadata.

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+2] (away 07/21 - 08/08)]

Probably they ment bug 482364 which is really a blocker.

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+2] (away 07/21 - 08/08)]

It's bad that it doesn't work with the recorder (especially for longer tests) but you have a workaround via the inspector for now.

Comment 5

[Adam Christian]

Right click seems to be the most blatant shortcoming in this part of the recorder. I understand the need for the modifiers, but we don't yet have that built into the way click works, which I don't think there is time for in this release, so I am keeping this open and moving it to 1.3 to do the rest of the work.

Added rightClick recording for 1.2, Committed revision 500.

Comment 6

[Henrik Skupin [:whimboo][⌚️UTC+2] (away 07/21 - 08/08)]

So we should wait for a possible fix in bug 497852? I'll add it to the dependency list meanwhile. If that's not the case feel free to remove it again.(In reply to comment #5)

> Added rightClick recording for 1.2, Committed revision 500.

This is called rightclick and not rightClick. Can you please fix this?

Further recording a test against attachment 371829 [details] by right clicking the text box and choosing paste does produce a non-valid test. The right click is not detected. Only a click call has been inserted for me.