[Webkit-unassigned] [Bug 8079] REGRESSION: Redraw from page cache does not show visited links

[bugzilla-daemon at opendarwin.org]

http://bugzilla.opendarwin.org/show_bug.cgi?id=8079


ddkilzer at kilzer.net changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|webkit-                     |ddkilzer at kilzer.net
                   |unassigned at opendarwin.org   |
             Status|NEW                         |ASSIGNED
   Attachment #7476|                            |review?
               Flag|                            |




------- Comment #7 from ddkilzer at kilzer.net  2006-04-02 21:53 PDT -------
Created an attachment (id=7476)
 --> (http://bugzilla.opendarwin.org/attachment.cgi?id=7476&action=view)
Patch v1 (no test)

Okay, I found the problem using the debugger.  One line of code (a one-line
'if' statement that really should have been formatted as two lines) was removed
from the end of the reparseConfiguration() method in Frame.cpp from the patch
for Bug 7907:

-  if (d->m_doc) d->m_doc->updateStyleSelector();

Adding this back fixes this bug.  I do not have a test yet.  I think a test
will require an http test with a PNG so that a unique link is rendered as
visited.  I'll need more time to write a test if that is required for this fix
(unless someone else wants to do it).

Also, I did not see a reason for removing this line of code in Bug 7907, so I'm
not sure if adding this back is the appropriate fix.


-- 
Configure bugmail: http://bugzilla.opendarwin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
You are the assignee for the bug, or are watching the assignee.