Massive MozMill Failure in Calendar on 2018-11-21

RESOLVED FIXED in 6.7

Status

defect
RESOLVED FIXED
7 months ago
4 months ago

People

(Reporter: jorgk, Assigned: darktrojan)

Tracking

(Depends on 1 bug)

unspecified
Dependency tree / graph

Details

Attachments

(1 attachment)

Reporter

Description

7 months ago
M-C last good: 95a0973a373520f582880ac258e3d2ee7b
M-C first bad: fbd97100c83cc07705244725a3245e6d14
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=95a0973a373520f582880ac258e3d2ee7b&tochange=fbd97100c83cc07705244725a3245e6d14

https://taskcluster-artifacts.net/MV0upWggQeSlTnUy04aiNg/0/public/logs/live_backing.log says:
EXCEPTION: Timeout exceeded for waitForElement Lookup: /id("messengerWindow")/id("tabmail-container")/id("tabmail")/id("tabmail-tabbox")/id("tabpanelcontainer")/id("calendarTabPanel")/id("calendarContent")/id("calendarDisplayDeck")/id("calendar-view-box")/id("view-deck")/id("day-view")/anon({"anonid":"mainbox"})/anon({"anonid":"headerbox"})/anon({"anonid":"headerdaybox"})/[0]/
and more.

Geoff, can you please take a look (I'm afk now).
Flags: needinfo?(geoff)
Assignee

Comment 2

7 months ago
Probably https://hg.mozilla.org/mozilla-central/rev/b9ccf121cfea. We use that binding for the different calendar views, and I think the mozmill tests click on the tab to switch view.
Assignee

Updated

7 months ago
Keywords: leave-open

Comment 3

7 months ago
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/4ddaece5b962
Temporarily stop using view-tab binding to make tests work again; rs=bustage-fix
Assignee

Comment 4

7 months ago
Not using a binding that extends the changed one makes things work again. This has a downside in that the (day, week, multiweek, month) tabs change width slightly when selected as the label is bold. Perhaps we can find some better CSS trickery to make that work, but I think we'll still need to figure out the mechanics of what broke here.
Blocks: 1509139
Assignee

Comment 5

7 months ago
Huh. That worked when I tried it, and now it doesn't.

Comment 6

7 months ago
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/601a4bad47a1
Backed out changeset 4ddaece5b962 ; rs=backout
https://hg.mozilla.org/comm-central/rev/b5146b7bfd61
Convert calendar view selection tabs to radios; rs=bustage-fix

Comment 7

7 months ago
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/781b3615a3ed
Remove accidentally committed change; rs=me
Assignee

Comment 8

7 months ago
If this doesn't work (apart from looking wrong), I'm giving up on this bug.
Reporter

Comment 9

7 months ago
It appears that you didn't have a good time with this bug :-( - I've been working in "bustage fix" for more than two years now.

So where does this leave us? Is https://hg.mozilla.org/comm-central/rev/b5146b7bfd61 meant to be permanent? Then flag is for review?

And "stop using view-tab binding" appeared to be working and then didn't after all?
Assignee

Comment 10

7 months ago
That leaves us with passing tests, and me unhappy with the solution. I would r- it myself, so I've put it to one side for a few hours while I think about a better way.
Assignee

Comment 11

7 months ago
This is what landed, plus what I think needs to land to finish it off. Richard, please check this looks okay on Mac and Windows.

I still don't understand what broke the tabs.
Assignee: nobody → geoff
Status: NEW → ASSIGNED
Attachment #9026909 - Flags: review?(philipp)
Attachment #9026909 - Flags: feedback?(richard.marti)
Reporter

Comment 12

7 months ago
I tried rev 4ddaece5b962 (stop using view-tab binding) but it doesn't let me switch those "view tabs" (Day/Week/Multiweek/Month). Rev b5146b7bfd61/781b3615a3ed work but are a little on the ugly side.

Given that we're busted completely now due to bug 1508056, I'll just go and hibernate for a while.

Comment 13

7 months ago
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/95cd4e14ec48
Fix calendar view selection radios styling; rs=me DONTBUILD
Reporter

Comment 14

7 months ago
Looks good on Windows and works :-)
Comment on attachment 9026909 [details] [diff] [review]
1509194-view-selection-tabs-1.diff

Looks good on Mac and Windows.
Attachment #9026909 - Flags: feedback?(richard.marti) → feedback+
Attachment #9026909 - Flags: review?(philipp) → review+
Assignee

Updated

7 months ago
Status: ASSIGNED → RESOLVED
Closed: 7 months ago
Keywords: leave-open
Resolution: --- → FIXED
Target Milestone: --- → 6.7

Updated

4 months ago
Depends on: 1526674
You need to log in before you can comment on or make changes to this bug.