356066 - nsIBoxObject get from document.getBoxObjectFor(element) has wrong location(x,y)

Status: RESOLVED WONTFIX

(Core :: General, defect)

Description

[Topper]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; zh-TW; rv:1.8.0.7) Gecko/20060909 Firefox/1.5.0.7
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; zh-TW; rv:1.8.0.7) Gecko/20060909 Firefox/1.5.0.7

Javascript: var bo = document.getBoxObjectFor(element);

Values of bo.x and bo.y are wrong.
Case 1: HTMLHtmlElement do not has margin or border
If element has border width, then the values of (bo.x,bo.y) is the position (x,y) plus extra border widths.
Case 2: HTMLHtmlElement do has margin or border
The values of (bo.x,bo.y) is the position (x,y) plus extra border widths and margins of HTMLHtmlElement.

Reproducible: Always

Steps to Reproduce:
1.Create a div element with border.
2.Use javascript: var bo = document.getBoxObjectFor(element);
3.Check bo.x and bo.y

Actual Results:  
The values of bo.x and bo.y are wrong.

Comment 1

[Topper]

Attachment: HTML and js files

Comment 2

[Nickolay_Ponomarev]

Thanks for the report.

Could you elaborate what exactly is wrong please (i.e. expected: values calculated in <such> way; actual results: values are miscalulated <so>)? It's not immediately obvious from your description of the bug or the testcase.

Also, for future reference, single-page testcases (e.g. HTML with inline CSS+JS) are preferred to zip attachments.

Comment 3

[Topper]

Attachment: test case 1

Comment 4

[Topper]

Attachment: test case 2

Comment 5

[Topper]

case1:
This bug will make all elements with border have wrong offsetLeft, offsetTop values.
case 2:
This bug will make all elements have wrong offsetLeft, offsetTop values, if HTMLHtmlElement has margin or border.

Comment 6

[Nickolay_Ponomarev]

Yeah, thanks for the updated testcase.

Comment 7

[Boris Zbarsky [:bzbarsky]]

offset* does not use nsIBoxObject.  The two are completely unrelated.  So comment  5 is wrong.

What boxObject _should_ do is not very well defined, in fact. So no point in working on this till we decide that.

Comment 8

[Topper]

Attachment: test case 3

@Boris:
Can you explain test case 3?

Comment 9

[Topper]

Comment on attachment 242459 [details]
test case 3

Sorry!

Comment 10

[Boris Zbarsky [:bzbarsky]]

I have no idea what testcase 3 is testing, so no.

Comment 11

[Topper]

Attachment: test case 4

uptodate test case 3

Comment 12

[Hsin-Yi Tsai (she/her)[:hsinyi]]

"The non-standard getBoxObjectFor() DOM method has been removed. You should be using getBoundingClientRect() instead." quoted in https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/3.5