Port "Bug 1521280 - Remove search-textbox binding" to Thunderbird

RESOLVED FIXED in Thunderbird 68.0

Status

enhancement
RESOLVED FIXED
4 months ago
2 months ago

People

(Reporter: mkmelin, Assigned: mkmelin)

Tracking

Trunk
Thunderbird 68.0
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [Thunderbird-testfailure: Z all][Thunderbird-disabled-test][XBL:2])

Attachments

(2 attachments, 1 obsolete attachment)

+++ This bug was initially created as a clone of Bug #1521280 +++

Bug 1521280 will remove the support for <textbox type="search">.

This should be it. It's apparently not working completely, but I think bug 1534930 could solve it. The m-c patch is also not 100% done yet so perhaps some moving parts from there too.

Just for reference, in the preferences(?) I got

console.error: (new TypeError("setting getter-only property "inputField"", "chrome://global/content/elements/search-textbox.js", 62))
console.error: (new TypeError("setting getter-only property "inputField"", "chrome://global/content/elements/search-textbox.js", 62))
prpl-irc: Unused mode parameters: #test
JavaScript error: chrome://global/content/bindings/textbox.xml, line 239: TypeError: this.setSelectionRange is not a function
JavaScript error: chrome://global/content/bindings/textbox.xml, line 247: TypeError: this._maybeSelectAll is not a function
JavaScript error: chrome://global/content/bindings/textbox.xml, line 247: TypeError: this._maybeSelectAll is not a function
JavaScript error: chrome://global/content/bindings/textbox.xml, line 239: TypeError: this.setSelectionRange is not a function
JavaScript error: chrome://global/content/bindings/textbox.xml, line 247: TypeError: this._maybeSelectAll is not a function

Bug 1521280 will land in the next days.
This seems to work fine.

I do get an error about gApplicationsPane.rebuildView "is not a function", but that happens on trunk too (for the applications search)
https://searchfox.org/comm-central/rev/b2c998eb41d696d31a87384487514e9b386f201c/mail/components/preferences/applications.inc.xul#49

Attachment #9050584 - Attachment is obsolete: true
Attachment #9058957 - Flags: review?(jorgk)
Status: NEW → ASSIGNED
Comment on attachment 9058957 [details] [diff] [review]
Bug_1534913___Port__Bug_1521280___Remove_search_textbox_binding__to_Thunderbird__r_.patch

Looks OK.
Attachment #9058957 - Flags: review?(jorgk) → review+

When bug 1521280 gets merged.

Keywords: checkin-needed

Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/55a635377bed
Port bug 1521280 [Remove search-textbox binding] to Thunderbird. r=jorgk

Status: ASSIGNED → RESOLVED
Closed: 2 months ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 68.0

Look like you need to fix two tests now:
TEST-UNEXPECTED-FAIL | Z:\task_1557525750\build\tests\mozmill\quick-filter-bar\test-keyboard-interface.js | test-keyboard-interface.js::test_control_shift_k_shows_quick_filter_bar
EXCEPTION: second control-shift-k did not select text in filter textbox

TEST-UNEXPECTED-FAIL | Z:\task_1557525750\build\tests\mozmill\testLocalICS.js | testLocalICS.js::testLocalICS
EXCEPTION: Expression "{"anonid":"moz-input-box"}" returned null. Anonymous == true
EXCEPTION: menuitem is null

Maybe the Calendar failure is unrelated?

Flags: needinfo?(mkmelin+mozilla)
Flags: needinfo?(geoff)
Flags: needinfo?(acelists)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---

The calendar failure appears to be related. The element in question is a <textbox type="search">.

Flags: needinfo?(geoff)

You should have told me which one, since it seems that Magnus replaced the three I can see:

calendar/base/content/calendar-task-view.xul
48 type="search"
calendar/base/content/calendar-unifinder.xul
55 type="search"
calendar/resources/content/calendarCreation.xul
62 type="search"

EDIT: OK, so you're saying that element needs further code adjustments. We might find an example here:
https://hg.mozilla.org/mozilla-central/rev/c5798de806e2

Flags: needinfo?(geoff)
Flags: needinfo?(geoff)

Yes, I got there too looking for "anonid":"moz-input-box". Sadly this magic needs another sorcerer :-(

Keywords: leave-open
Whiteboard: [Thunderbird-testfailure: Z all][Thunderbird-disabled-test][XBL:2]
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/fc51c0e147cc
temporarily disable failining tests. rs=bustage-fix DONTBUILD
Flags: needinfo?(mkmelin+mozilla)
Flags: needinfo?(acelists)
Attachment #9064309 - Flags: review?(jorgk)

Thanks. Linux debug is not the best choice for a try since it has some many oranges. Let's try Mac instead:
https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=271ed605f84624b02a5fed25339f893f6d0c7ac6

I tried adding .inputField like in https://hg.mozilla.org/mozilla-central/rev/c5798de806e2#l26.19, but I added it to the wrong spot. Seems so obvious now :-(

EDIT: I wonder whether -u mozmill will work at all. I think only -u all works these days due to the "1proc" suffixes.

Comment on attachment 9064309 [details] [diff] [review]
bug1534913_input_search_tests.patch

Thanks. My try was successful, yours didn't go anywhere.
Attachment #9064309 - Flags: review?(jorgk) → review+

Yeah typoed, was supposed to be the opt build.

Status: REOPENED → RESOLVED
Closed: 2 months ago2 months ago
Keywords: checkin-needed
Resolution: --- → FIXED

https://hg.mozilla.org/comm-central/rev/937e7c60b40043bbdd456e18aaef5df8c0af91a1
Backed out 3 changesets (bug 1534913) since ported bug 1521280 was backed out. a=backout

Status: RESOLVED → REOPENED
Resolution: FIXED → ---

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/779732045e34
Port bug 1521280 [Remove search-textbox binding] to Thunderbird. r=jorgk

Status: REOPENED → RESOLVED
Closed: 2 months ago2 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.