[Webkit-unassigned] [Bug 74864] There is additional space not belonged to a table between the table cells

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Aug 7 08:58:59 PDT 2012


https://bugs.webkit.org/show_bug.cgi?id=74864





--- Comment #18 from Allan Sandfeld Jensen <allan.jensen at nokia.com>  2012-08-07 08:58:56 PST ---
(In reply to comment #17)
> (In reply to comment #15)
> > > LayoutTests/fast/dom/hittest-tablecell-edge.html:32
> > > +	y = ele.getBoundingClientRect().bottom/2;
> > > +	debug('Executing for element: '+orgElement);
> > > +	debug('Node at edge-1: '+document.elementFromPoint(x-1,y).id);
> > > +	debug('Node at edge: '+document.elementFromPoint(x,y).id);
> > > +	debug('Node at edge+1: '+document.elementFromPoint(x+1,y).id);
> > 
> > It's prefered to use WebKit style for tests.
> > 
> Changed these to use shouldBe() instead.
> 
I think he meant also coding style. The style-checker doesn't enforce WebKit coding style in javascript, but it is a good idea to use it where it make sense, like adding a space after comma or around binary operators.

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



More information about the webkit-unassigned mailing list