stylo: missing labels on add an attachment on BMO

VERIFIED FIXED in Firefox 56

Status

()

Core
CSS Parsing and Computation
P1
normal
VERIFIED FIXED
6 months ago
5 months ago

People

(Reporter: Usul, Assigned: xidorn)

Tracking

(Blocks: 1 bug)

48 Branch
mozilla56
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox56 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(5 attachments)

(Reporter)

Description

6 months ago
Created attachment 8880896 [details]
Screenshot-2017-6-23 Create New Attachment for Bug #1375902.png

with Stylo enabled go to a bugzilla issue click the add an attachment link.
Pages load but doesn't display any text as per the attached screenshot.
Blocks: 1375906
I'm pretty sure this is a very recent regression. I've used a stylo build for filing bugs not long ago.
My build was with the patches on bug 1371130, so something between that an nightly is what have regressed it.
(Reporter)

Updated

6 months ago
Duplicate of this bug: 1375935
So I haven't been able to repro with current autoland tip so far (without webrender). This is on a debug build (waiting on a release build as we speak).

The page has:

#header {
    color: transparent;
}

and also a similar selector for a, #header a, etc... with the correct color.

However, they're all on the same stylesheet, and the |a| selector will make us restyle the whole document when the stylesheet is loaded, so I'm discarding bug 1357583 as the culprit... Also, the other sections that appear blank don't seem to have any selector like that.

The Ctrl + F5 vs. F5 issue may highlight a stylesheet reusing bug or something like that?

Does it repro without WR?
(The WR question is because the screenshot at bug 1375935 mentioned it, btw)
(In reply to Emilio Cobos Álvarez [:emilio] from comment #4)
> Does it repro without WR?

Yes, bug 1375935 is reproducible with webrender+webrendest and without them.
(In reply to Darkspirit from comment #6)
> (In reply to Emilio Cobos Álvarez [:emilio] from comment #4)
> > Does it repro without WR?
> 
> Yes, bug 1375935 is reproducible with webrender+webrendest and without them.

Actually, this could also be a dupe of bug 1375674 too. My build did have those patches, and the symptom could be related. Marking a dependency since that should be close to landing, before continuing to investigate.
Depends on: 1375674
Before speculatively doing that... May I ask which addons do you have? I've tried to repro it in a bunch of ways on a current profile and I just couldn't :(
No longer depends on: 1375674
(In reply to Emilio Cobos Álvarez [:emilio] from comment #8)
> Before speculatively doing that... May I ask which addons do you have? I've
> tried to repro it in a bunch of ways on a current profile and I just
> couldn't :(

With a clean profile, I mean...
Created attachment 8881062 [details]
2017-06-24_18-44-09.mp4

(In reply to Emilio Cobos Álvarez [:emilio] from comment #8)
> Before speculatively doing that... May I ask which addons do you have? I've tried to repro it in a bunch of ways on a current profile and I just couldn't :(

As bug 1375935 is a duplicate of this, I upload this proof here: A fresh profile on Debian testing x64.
(Reporter)

Comment 11

6 months ago
Grammalecte
Mastodon share
Pinboard
test pilot
wayback machine
vidyo replay download
Youtube high definition

it broke for me on flipping the pref and flipping it back.

I also have a theme -> frisian flag
Created attachment 8881339 [details]
text-no-show.png

Unfortunately, I can't file stylo bugs via stylo nightly as this bug seriously affects readability of BMO.
Priority: -- → P1
(In reply to Astley Chen [:astley] (UTC-7) from comment #12)
> Created attachment 8881339 [details]
> text-no-show.png
> 
> Unfortunately, I can't file stylo bugs via stylo nightly as this bug
> seriously affects readability of BMO.

Do you have any addons installed? If not, you and Emilio should sit down tomorrow morning and figure out how to reproduce it.
(In reply to Bobby Holley (:bholley) (busy with Stylo) from comment #13)
> (In reply to Astley Chen [:astley] (UTC-7) from comment #12)
> > Created attachment 8881339 [details]
> > text-no-show.png
> > 
> > Unfortunately, I can't file stylo bugs via stylo nightly as this bug
> > seriously affects readability of BMO.
> 
> Do you have any addons installed? If not, you and Emilio should sit down
> tomorrow morning and figure out how to reproduce it.

Ok, so we debugged this today, and I can't repro it because I had the Fira Sans font installed locally, so it's probably a font-face issue.
Xidorn is going to investigate this.
Assignee: nobody → xidorn+moz
Created attachment 8881588 [details]
simplified testcase from bugzilla page

This is the simplified testcase. The first load of this page may behave normally, but later refresh would trigger this issue.

It happens whenever there is a reference of ex unit anywhere in the page.
Invoking IncrementGeneration() in ContinueLoad fixes this issue. Not sure whether this is the correct fix, though. I need to analyze a bit further to figure out a real fix.
Comment hidden (mozreview-request)

Comment 19

6 months ago
hg error in cmd: hg push -r . -f try: pushing to ssh://hg.mozilla.org/try
searching for changes
remote: waiting for lock on working directory of /repo/hg/mozilla/try held by process '29479' on host 'hgssh4.dmz.scl3.mozilla.com'
remote: got lock after 5 seconds
remote: adding changesets
remote: adding manifests
remote: adding file changes
remote: added 3 changesets with 5 changes to 6 files (+1 heads)
remote: autoland push detected
remote: recorded push in pushlog
remote: 
remote: View your changes here:
remote:   https://hg.mozilla.org/try/rev/467684eafd512d89c5b8ee04e39cd94925e7982e
remote:   https://hg.mozilla.org/try/rev/c89590c461db463b9da14c753ebb1624db8eae7c
remote:   https://hg.mozilla.org/try/rev/b77e1e16ea3bf97a6556415eb5f4206b3661c688
remote: 
remote: Follow the progress of your build on Treeherder:
remote:   https://treeherder.mozilla.org/#/jobs?repo=try&revision=b77e1e16ea3bf97a6556415eb5f4206b3661c688
remote: recorded changegroup in replication log in 0.133s
remote: ** unknown exception encountered, please report by visiting
remote: ** https://mercurial-scm.org/wiki/BugTracker
remote: ** Python 2.7.5 (default, Nov  6 2016, 00:28:07) [GCC 4.8.5 20150623 (Red Hat 4.8.5-11)]
remote: ** Mercurial Distributed SCM (version 4.1.2)
remote: ** Extensions loaded: blackbox, clonebundles, obsolescencehacks, pushlog, serverlog, readonly, vcsreplicator
remote: Traceback (most recent call last):
remote:   File "/var/hg/venv_pash/bin/hg", line 45, in <module>
remote:     mercurial.dispatch.run()
remote:   File "/var/hg/venv_pash/lib/python2.7/site-packages/mercurial/dispatch.py", line 63, in run
remote:     sys.exit((dispatch(request(pycompat.sysargv[1:])) or 0) & 255)
remote:   File "/var/hg/venv_pash/lib/python2.7/site-packages/mercurial/dispatch.py", line 129, in dispatch
remote:     ret = _runcatch(req)
remote:   File "/var/hg/venv_pash/lib/python2.7/site-packages/mercurial/dispatch.py", line 219, in _runcatch
remote:     return callcatch(ui, _runcatchfunc)
remote:   File "/var/hg/venv_pash/lib/python2.7/site-packages/mercurial/dispatch.py", line 227, in callcatch
remote:     return scmutil.callcatch(ui, func)
remote:   File "/var/hg/venv_pash/lib/python2.7/site-packages/mercurial/scmutil.py", line 152, in callcatch
remote:     return func()
remote:   File "/var/hg/venv_pash/lib/python2.7/site-packages/mercurial/dispatch.py", line 208, in _runcatchfunc
remote:     return _dispatch(req)
remote:   File "/var/hg/venv_pash/lib/python2.7/site-packages/mercurial/dispatch.py", line 811, in _dispatch
remote:     cmdpats, cmdoptions)
remote:   File "/var/hg/venv_pash/lib/python2.7/site-packages/mercurial/dispatch.py", line 563, in runcommand
remote:     ret = _runcommand(ui, options, cmd, d)
remote:   File "/var/hg/venv_pash/lib/python2.7/site-packages/mercurial/dispatch.py", line 819, in _runcommand
remote:     return cmdfunc()
remote:   File "/var/hg/venv_pash/lib/python2.7/site-packages/mercurial/dispatch.py", line 808, in <lambda>
remote:     d = lambda: util.checksignature(func)(ui, *args, **strcmdopt)
remote:   File "/var/hg/venv_pash/lib/python2.7/site-packages/mercurial/util.py", line 1051, in check
remote:     return func(*args, **kwargs)
remote:   File "/var/hg/venv_pash/lib/python2.7/site-packages/mercurial/commands.py", line 5824, in serve
remote:     s.serve_forever()
remote:   File "/var/hg/version-control-tools/hgext/serverlog/__init__.py", line 320, in serve_forever
remote:     return super(sshserverwrapped, self).serve_forever()
remote:   File "/var/hg/venv_pash/lib/python2.7/site-packages/mercurial/sshserver.py", line 104, in serve_forever
remote:     while self.serve_one():
remote:   File "/var/hg/version-control-tools/hgext/serverlog/__init__.py", line 351, in serve_one
remote:     return super(sshserverwrapped, self).serve_one()
remote:   File "/var/hg/venv_pash/lib/python2.7/site-packages/mercurial/sshserver.py", line 122, in serve_one
remote:     rsp = wireproto.dispatch(self.repo, self, cmd)
remote:   File "/var/hg/version-control-tools/hgext/serverlog/__init__.py", line 343, in dispatch
remote:     return origdispatch(repo, proto, cmd)
remote:   File "/var/hg/venv_pash/lib/python2.7/site-packages/mercurial/extensions.py", line 223, in closure
remote:     return func(*(args + a), **kw)
remote:   File "/var/hg/version-control-tools/pylib/vcsreplicator/vcsreplicator/hgext.py", line 359, in wireprotodispatch
remote:     return orig(repo, proto, command)
remote:   File "/var/hg/venv_pash/lib/python2.7/site-packages/mercurial/wireproto.py", line 569, in dispatch
remote:     return func(repo, proto, *args)
remote:   File "/var/hg/venv_pash/lib/python2.7/site-packages/mercurial/wireproto.py", line 982, in unbundle
remote:     proto._client())
remote:   File "/var/hg/venv_pash/lib/python2.7/site-packages/mercurial/exchange.py", line 1771, in unbundle
remote:     lockandtr[2].close()
remote:   File "/var/hg/venv_pash/lib/python2.7/site-packages/mercurial/transaction.py", line 43, in _active
remote:     return func(self, *args, **kwds)
remote:   File "/var/hg/venv_pash/lib/python2.7/site-packages/mercurial/transaction.py", line 490, in close
remote:     self._postclosecallback[cat](self)
remote:   File "/var/hg/version-control-tools/hgext/pushlog/__init__.py", line 239, in onpostclose
remote:     conn.commit()
remote: sqlite3.OperationalError: database is locked
abort: stream ended unexpectedly (got 0 bytes, expected 4)
Comment on attachment 8881858 [details]
Bug 1375944 - Increment font set generation when font-face is synchronously loaded in post-traversal.

https://reviewboard.mozilla.org/r/152920/#review158112

::: gfx/thebes/gfxUserFontSet.cpp:463
(Diff revision 1)
>  gfxUserFontEntry::ContinueLoad()
>  {
>      MOZ_ASSERT(mUserFontLoadState == STATUS_LOAD_PENDING);
>      MOZ_ASSERT(mSrcList[mSrcIndex].mSourceType == gfxFontFaceSrc::eSourceType_URL);
>  
> +    SetLoadState(STATUS_LOADING);

Hmm, I wonder if not doing this was causing bugs in the FontFaceSet API...
Attachment #8881858 - Flags: review?(cam) → review+
(Assignee)

Comment 21

6 months ago
mozreview-review-reply
Comment on attachment 8881858 [details]
Bug 1375944 - Increment font set generation when font-face is synchronously loaded in post-traversal.

https://reviewboard.mozilla.org/r/152920/#review158112

> Hmm, I wonder if not doing this was causing bugs in the FontFaceSet API...

Not sure. I just want to be able to check the load state after calling `DoLoadNextSrc`.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=ff601e70bcafeeab12c8ad2ef7be7ffa8ddfec86

Looks like there is an unexpected pass in reftest :)
Comment hidden (mozreview-request)
Blocks: 1376964

Comment 24

6 months ago
Pushed by xquan@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/21f6ba7bb580
Increment font set generation when font-face is synchronously loaded in post-traversal. r=heycam

Comment 25

6 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/21f6ba7bb580
Status: NEW → RESOLVED
Last Resolved: 6 months ago
status-firefox56: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
(In reply to Darkspirit from comment #10)
> 2017-06-24_18-44-09.mp4

Can't reproduce anymore.
Seems to be fixed in today's 20170630100234 @ Debian Testing x64 (Radeon RX480).
Status: RESOLVED → VERIFIED

Updated

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