4 ms·

Wow, if you exclude lines changed in the commit due to indenting changing, there are only five new lines of code for this change!

langsoul-com 3y ago
Most of the challenges of a bug isn't the fix, but rather figuring out the behaviour. Especially true if breakpoints don't work :)
- sroussey 3y ago
  Always a bummer when your tools for debugging don’t work. :/
deleted 3y ago
[deleted]
acqq 3y ago
The changes, in sum, in nsXULTooltipListener.cpp - if (tooltipNode->GetComposedDoc() && - nsContentUtils::IsChromeDoc(tooltipNode->GetComposedDoc())) { + // Make sure the document still has focus. + auto* doc = tooltipNode->GetComposedDoc(); + if (!doc || !nsContentUtils::IsChromeDoc(doc) || + !doc->HasFocus(IgnoreErrors())) { + return NS_OK; + } ... - } } return NS_OK; If I see correctly, all the changes are: 1) remembering result of tooltipNode->GetComposedDoc() and adding the test of doc->HasFocus(IgnoreErrors()). Note that writing this one now is maybe easier than it was at the time the initial code was written, it could be the "auto" in this current semantic didn't exist in C++ (or the used compilers/platforms) at that time. 2) Explicit return. Instead of: if (b) X; return OK; now it's: if (!b) return OK; X; return OK; which in this case increases readability as X is in many lines and b is a more complex condition.
- chrisjc 3y ago
  Can any one give a brief synopsis of the state of XUL and gecko in Firefox/Mozilla? Are the XUL (XULRunner) and gecko runtimes still actively worked on? Or has it been absorbed into the Firefox runtime long ago? I recall reading some time ago that the work on XULRunner essentially came to a halt, and that components in Firefox/Mozilla that depend on XUL would slowly be phased out. But does that mean that work on the XUL runtime and components in Firefox also effectively came to a halt? I figured that most of these really old XUL bugs in Firefox were never going to be fixed, instead replacing a whole layer/component dependent on XUL was seen as a better use of time and resources. Edit: anyone have a good diagram of the layers/components in Firefox. Something that can illustrate where Quantum, Gecko, XUL, etc all live in Firefox. It would be really cool (doubt it exists) if there was an animated diagram that would show the changes of this stack overtime.

langsoul-com 3y ago

Most of the challenges of a bug isn't the fix, but rather figuring out the behaviour. Especially true if breakpoints don't work :)

sroussey 3y ago
Always a bummer when your tools for debugging don’t work. :/

deleted 3y ago

[deleted]

acqq 3y ago

The changes, in sum, in nsXULTooltipListener.cpp - if (tooltipNode->GetComposedDoc() && - nsContentUtils::IsChromeDoc(tooltipNode->GetComposedDoc())) { + // Make sure the document still has focus. + auto* doc = tooltipNode->GetComposedDoc(); + if (!doc || !nsContentUtils::IsChromeDoc(doc) || + !doc->HasFocus(IgnoreErrors())) { + return NS_OK; + } ... - } } return NS_OK; If I see correctly, all the changes are: 1) remembering result of tooltipNode->GetComposedDoc() and adding the test of doc->HasFocus(IgnoreErrors()). Note that writing this one now is maybe easier than it was at the time the initial code was written, it could be the "auto" in this current semantic didn't exist in C++ (or the used compilers/platforms) at that time. 2) Explicit return. Instead of: if (b) X; return OK; now it's: if (!b) return OK; X; return OK; which in this case increases readability as X is in many lines and b is a more complex condition.

chrisjc 3y ago
Can any one give a brief synopsis of the state of XUL and gecko in Firefox/Mozilla? Are the XUL (XULRunner) and gecko runtimes still actively worked on? Or has it been absorbed into the Firefox runtime long ago? I recall reading some time ago that the work on XULRunner essentially came to a halt, and that components in Firefox/Mozilla that depend on XUL would slowly be phased out. But does that mean that work on the XUL runtime and components in Firefox also effectively came to a halt? I figured that most of these really old XUL bugs in Firefox were never going to be fixed, instead replacing a whole layer/component dependent on XUL was seen as a better use of time and resources. Edit: anyone have a good diagram of the layers/components in Firefox. Something that can illustrate where Quantum, Gecko, XUL, etc all live in Firefox. It would be really cool (doubt it exists) if there was an animated diagram that would show the changes of this stack overtime.