Closed
Bug 126704
Opened 22 years ago
Closed 22 years ago
Loading page from Tasks | Tools | History window fails
Categories
(Core Graveyard :: History: Global, defect)
Core Graveyard
History: Global
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: tracy, Assigned: bugs)
Details
(Keywords: smoketest)
seen on commercail builds: windows 2002-02-20-06-trunk linux 2002-02-20-06-trunk mac osx 2002-02-20-03-trunk -Browse a few random pages -Go to Task | Tools | History -From the window, double click a site the page doesn't load into the browser window as expected. blocker
Comment 1•22 years ago
|
||
-> ben. I assume this is yours. I told you to back that stuff out. Now suffer the consequences! <insert evil laugh>
Assignee: blaker → ben
Comment 2•22 years ago
|
||
Looking at the code, I'm fairly certain the problem is just that ben removed the declaration of builder in history.js' OpenURL. I don't know if Ben is around right now, it doesn't look like it, so let's not hold for this. If someone can try readding that line, that would be good. Ben's changes to history should be backed out anyway (except for the addition of the stylesheet).
Severity: blocker → critical
Comment 3•22 years ago
|
||
sheriff's note: restored original severity to "blocker". I'd like to see more resolution here before we downgrade this. After reading blaker's last commment, we need to do one of three things: 1) try the one line fix, or otherwise fix it. 2) back out Ben's code or 3) come up with a convincing argument as to why this isn't a blocker.
Severity: critical → blocker
Comment 4•22 years ago
|
||
I am home now and will try the one line fix. However, backing out the history patch is the right thing to do anyway. I advised doing that last night. The main reason to hold the tree for a blocker is to make sure that everyone is looking into it. Given that we basically had a fix, there was no reason to make everyone wait. That's why I downgraded it.
Assignee | ||
Comment 5•22 years ago
|
||
I've checked in a fix.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•