[Webkit-unassigned] [Bug 12687] REGRESSION: Google Calendar cell highlight misplaced

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=12687


koivisto at iki.fi changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #13091|                            |review?
               Flag|                            |




------- Comment #12 from koivisto at iki.fi  2007-02-09 13:26 PDT -------
Created an attachment (id=13091)
 --> (http://bugs.webkit.org/attachment.cgi?id=13091&action=view)
proposed patch

Ok, so here is proposed patch. It does three things
1) calculate offsets from padding edge (inner edge of border), not from border
edge
2) revert this 

!(!style()->htmlHacks() && skipTables ? curr->isRoot() : curr->isBody())

in http://trac.webkit.org/projects/webkit/changeset/11484 back to this

!curr->isBody()

which seems to match FF behavior in both strict and compat modes. Why was this
change done? Hard to say since there was no test case nor explanation.

3) Check for tableness from dom tree instead of rendering tree. That's just
what other browser seem to do based on Anne's tests.

These changes make all Anne's tests pass except for 3,8,12,29,43,45,49,50. In
all those cases we match FF behavior (except 12, where we fail a bit less).
Failures are of course all debatable since there is no authorative spec about
offset behaviors. No layout test failures either.

(some of Anne's test cases may randomly produce wrong offsetTop results when
reloaded. that is some layout timing problem not directly related to offset
calculations)

None of this actually fixes Google Calendar, it needs ToT fixes in the other
end. Best we can do is to make this stuff consistent.


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