The default bug view has changed. See this FAQ.

Unregress Talos tests after landing Intl

RESOLVED INVALID

Status

()

Core
JavaScript: Internationalization API
RESOLVED INVALID
4 years ago
3 years ago

People

(Reporter: till, Assigned: Waldo)

Tracking

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

4 years ago
From 853301 bug comment 32:

"Similar to the previous landing, this regressed Ts by ~25ms, tpaint by about 10ms, and increased various memory counters.  Possible clue: "Tp5 No Network Row Major MozAfterPaint (Main Startup File IO Bytes)" regressed by 34% (about 10MB).

https://groups.google.com/d/topic/mozilla.dev.tree-management/o0klAW2kxj8/discussion

If this patch stays in, we'll need follow-up bugs to fix the regressions.  They should be nominated for tracking-firefox25."
(Reporter)

Updated

4 years ago
Depends on: 900292
tracking-firefox25: ? → +

Comment 1

4 years ago
Naveed, how do you plan to address this? Should we worry about this in the FF25 timeframe? If so, do we have alternatives such as disabling part of bug 853301?
Assignee: general → nihsanullah
(Reporter)

Comment 2

4 years ago
(In reply to Alex Keybl [:akeybl] from comment #1)
> Naveed, how do you plan to address this? Should we worry about this in the
> FF25 timeframe? If so, do we have alternatives such as disabling part of bug
> 853301?

This will pretty much be fixed once bug 919872 lands.
Depends on: 919872
Also, if I'm not mistaken, the original regression was backed out before branch uplift, so technically there's nothing in any tree to unregress.  Only once Intl is enabled in desktop builds could there possibly be any regression -- but with bug 919872 landed, I'm fairly sure there'll be no regression any more.
The landing management in bug 853301 has not been great. As I understand it, bug 853301 comment 41 was a trunk disabling, and bug 853301 comment 64 re-enabled building of it, but doesn't expose it (so still no Talos impact). So 25 up to trunk are not "affected", and we don't need to track this for 25.
tracking-firefox25: + → ?
tracking-firefox25: ? → ---
Assignee: nihsanullah → jwalden+bmo
Mass-moving existing Intl-related bugs to the new Core :: JavaScript: Internationalization API component.

If you think this bug has been moved in error, feel free to move it back to Core :: JavaScript Engine.

[Mass change filter: core-js-intl-api-move]
Component: JavaScript Engine → JavaScript: Internationalization API
I don't think this is relevant anymore.
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.