73595 2011-12-01 15:26:54 -0800 Focus ring of imagemap's area element does not scale when CSS zoom style is applied 2011-12-01 22:33:15 -0800 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) All All RESOLVED FIXED P2 Normal --- 0 mvujovic webkit-unassigned ap jonlee mvujovic webkit.review.bot oldest_to_newest 513194 0 117503 mvujovic 2011-12-01 15:26:54 -0800 Created attachment 117503 Reproduction In the attached reproduction, the clickable region of the area element scales with the image, but the area element's focus ring does not scale. The image has a CSS zoom property applied to it. 513319 1 117531 mvujovic 2011-12-01 17:24:01 -0800 Created attachment 117531 Patch HTMLAreaElement is only considering the page zoom when computing the path for drawing the focus ring. In the patch, I've updated HTMLAreaElement to use the RenderObject's effectiveZoom instead of the Frame's page zoom. effectiveZoom takes into account both the page zoom and any scaling from a CSS zoom property. 513502 2 117531 webkit.review.bot 2011-12-01 22:33:10 -0800 Comment on attachment 117531 Patch Clearing flags on attachment: 117531 Committed r101743: <http://trac.webkit.org/changeset/101743> 513503 3 webkit.review.bot 2011-12-01 22:33:15 -0800 All reviewed patches have been landed. Closing bug. 117503 2011-12-01 15:26:54 -0800 2011-12-01 15:26:54 -0800 Reproduction imagemap-focus-ring-zoom-style.html text/html 644 mvujovic PGh0bWw+CjxoZWFkPgo8c2NyaXB0Pgp3aW5kb3cub25sb2FkID0gZnVuY3Rpb24oKQp7CiAgICBk b2N1bWVudC5nZXRFbGVtZW50QnlJZCgiYXJlYSIpLmZvY3VzKCk7Cn0KPC9zY3JpcHQ+CjxzdHls ZT4KaW1nIHsgem9vbTogMi4wOyB9Cjwvc3R5bGU+CjwvaGVhZD4KPGJvZHk+CjxwPkFzc3VtaW5n IHRoZSBwb3J0LXNwZWNpZmljIHRoZW1lIGRyYXdzIGZvY3VzIHJpbmdzLCB0aGlzIHRlc3QgZW5z dXJlcyB0aGF0IGEgZm9jdXMgcmluZyBpcyBkcmF3biBjb3JyZWN0bHkgZm9yIGFuIGltYWdlbWFw IHdoZW4gdGhlIHpvb20gc3R5bGUgaXMgYXBwbGllZCB0byB0aGUgaW1hZ2UuIFRoaXMgdGVzdCBQ QVNTRUQgaWYgYSBmb2N1cyByaW5nIGlzIGRyYXduIGFyb3VuZCB0aGUgcGVyaW1ldGVyIG9mIHRo ZSAmbHQ7YXJlYSZndDsgaW4gdGhlIGltYWdlbWFwIChiZWxvdykuPC9wPgo8bWFwIG5hbWU9Imlt YWdlbWFwIj4KICAgIDxhcmVhIGlkPSJhcmVhIiBzaGFwZT0icmVjdCIgY29vcmRzPSIwLDAsMTI4 LDEyOCIgaHJlZj0iI2R1bW15IiAvPgo8L21hcD4KPGltZyBzcmM9ImltYWdlbWFwLmpwZyIgd2lk dGg9IjEyOCIgaGVpZ2h0PSIxMjgiIHVzZW1hcD0iI2ltYWdlbWFwIiBpc21hcCAvPgo8L2JvZHk+ CjwvaGVhZD4KPC9odG1sPgo= 117531 2011-12-01 17:24:01 -0800 2011-12-01 22:33:10 -0800 Patch 73595.patch text/plain 4263 mvujovic SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDEwMTcxOCkKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE5IEBACisyMDExLTEyLTAxICBNYXggVnVq b3ZpYyAgPG12dWpvdmljQGFkb2JlLmNvbT4KKworICAgICAgICBGb2N1cyByaW5nIG9mIGltYWdl bWFwJ3MgYXJlYSBlbGVtZW50IGRvZXMgbm90IHNjYWxlIHdoZW4gQ1NTIHpvb20gc3R5bGUgaXMg YXBwbGllZAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9 NzM1OTUKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBU ZXN0czogZmFzdC9pbWFnZXMvaW1hZ2VtYXAtZm9jdXMtcmluZy16b29tLXN0eWxlLWV4cGVjdGVk Lmh0bWwKKyAgICAgICAgICAgICAgIGZhc3QvaW1hZ2VzL2ltYWdlbWFwLWZvY3VzLXJpbmctem9v bS1zdHlsZS5odG1sCisKKyAgICAgICAgKiBodG1sL0hUTUxBcmVhRWxlbWVudC5jcHA6CisgICAg ICAgIChXZWJDb3JlOjpIVE1MQXJlYUVsZW1lbnQ6OmNvbXB1dGVQYXRoKToKKyAgICAgICAgVGhl IGNvbXB1dGVQYXRoIG1ldGhvZCBub3cgdXNlcyB0aGUgUmVuZGVyT2JqZWN0J3MgZWZmZWN0aXZl Wm9vbSAKKyAgICAgICAgaW5zdGVhZCBvZiB0aGUgRnJhbWUncyBwYWdlIHpvb20gdG8gY29tcHV0 ZSB0aGUgcGF0aCBmb3IgdGhlIGFyZWEKKyAgICAgICAgZWxlbWVudCdzIGZvY3VzIHJpbmcuCisK IDIwMTEtMTItMDEgIE1hcmsgUGlsZ3JpbSAgPHBpbGdyaW1AY2hyb21pdW0ub3JnPgogCiAgICAg ICAgIFtGaWxlU3lzdGVtIEFQSV0gRGlyZWN0b3J5RW50cnkucmVtb3ZlUmVjdXJzaXZlbHkgc3Vj Y2Vzc0NhbGxiYWNrIGlzIHJlcXVpcmVkCkluZGV4OiBTb3VyY2UvV2ViQ29yZS9odG1sL0hUTUxB cmVhRWxlbWVudC5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL1dlYkNvcmUvaHRtbC9IVE1MQXJl YUVsZW1lbnQuY3BwCShyZXZpc2lvbiAxMDEyNjMpCisrKyBTb3VyY2UvV2ViQ29yZS9odG1sL0hU TUxBcmVhRWxlbWVudC5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTEwNyw3ICsxMDcsNyBAQCBQYXRo IEhUTUxBcmVhRWxlbWVudDo6Y29tcHV0ZVBhdGgoUmVuZGVyCiAgICAgICAgIHNpemUgPSBvYmot PmFic29sdXRlT3V0bGluZUJvdW5kcygpLnNpemUoKTsKICAgICAKICAgICBQYXRoIHAgPSBnZXRS ZWdpb24oc2l6ZSk7Ci0gICAgZmxvYXQgem9vbUZhY3RvciA9IGRvY3VtZW50KCktPmZyYW1lKCkt PnBhZ2Vab29tRmFjdG9yKCk7CisgICAgZmxvYXQgem9vbUZhY3RvciA9IG9iai0+c3R5bGUoKS0+ ZWZmZWN0aXZlWm9vbSgpOwogICAgIGlmICh6b29tRmFjdG9yICE9IDEuMGYpIHsKICAgICAgICAg QWZmaW5lVHJhbnNmb3JtIHpvb21UcmFuc2Zvcm07CiAgICAgICAgIHpvb21UcmFuc2Zvcm0uc2Nh bGUoem9vbUZhY3Rvcik7CkluZGV4OiBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cKPT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQot LS0gTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCShyZXZpc2lvbiAxMDE3MTgpCisrKyBMYXlvdXRUZXN0 cy9DaGFuZ2VMb2cJKHdvcmtpbmcgY29weSkKQEAgLTEsMyArMSwxMyBAQAorMjAxMS0xMi0wMSAg TWF4IFZ1am92aWMgIDxtdnVqb3ZpY0BhZG9iZS5jb20+CisKKyAgICAgICAgRm9jdXMgcmluZyBv ZiBpbWFnZW1hcCdzIGFyZWEgZWxlbWVudCBkb2VzIG5vdCBzY2FsZSB3aGVuIENTUyB6b29tIHN0 eWxlIGlzIGFwcGxpZWQKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcu Y2dpP2lkPTczNTk1CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAg ICAgICAgKiBmYXN0L2ltYWdlcy9pbWFnZW1hcC1mb2N1cy1yaW5nLXpvb20tc3R5bGUtZXhwZWN0 ZWQuaHRtbDogQWRkZWQuCisgICAgICAgICogZmFzdC9pbWFnZXMvaW1hZ2VtYXAtZm9jdXMtcmlu Zy16b29tLXN0eWxlLmh0bWw6IEFkZGVkLgorCiAyMDExLTEyLTAxICBNYXJrIFBpbGdyaW0gIDxw aWxncmltQGNocm9taXVtLm9yZz4KIAogICAgICAgICBbRmlsZVN5c3RlbSBBUEldIERpcmVjdG9y eUVudHJ5LnJlbW92ZVJlY3Vyc2l2ZWx5IHN1Y2Nlc3NDYWxsYmFjayBpcyByZXF1aXJlZApJbmRl eDogTGF5b3V0VGVzdHMvZmFzdC9pbWFnZXMvaW1hZ2VtYXAtZm9jdXMtcmluZy16b29tLXN0eWxl LWV4cGVjdGVkLmh0bWwKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvZmFzdC9pbWFnZXMvaW1h Z2VtYXAtZm9jdXMtcmluZy16b29tLXN0eWxlLWV4cGVjdGVkLmh0bWwJKHJldmlzaW9uIDApCisr KyBMYXlvdXRUZXN0cy9mYXN0L2ltYWdlcy9pbWFnZW1hcC1mb2N1cy1yaW5nLXpvb20tc3R5bGUt ZXhwZWN0ZWQuaHRtbAkocmV2aXNpb24gMCkKQEAgLTAsMCArMSwxOCBAQAorPGh0bWw+Cis8aGVh ZD4KKzxzY3JpcHQ+Cit3aW5kb3cub25sb2FkID0gZnVuY3Rpb24oKQoreworICAgIGRvY3VtZW50 LmdldEVsZW1lbnRCeUlkKCJhcmVhIikuZm9jdXMoKTsKK30KKzwvc2NyaXB0PgorPC9oZWFkPgor PGJvZHk+Cis8cD5Bc3N1bWluZyB0aGUgcG9ydC1zcGVjaWZpYyB0aGVtZSBkcmF3cyBmb2N1cyBy aW5ncywgdGhpcyB0ZXN0IGVuc3VyZXMgdGhhdCBhIGZvY3VzIHJpbmcgaXMgZHJhd24gY29ycmVj dGx5IGZvciBhbiBpbWFnZW1hcCB3aGVuIHRoZSB6b29tIHN0eWxlIGlzIGFwcGxpZWQgdG8gdGhl IGltYWdlLiBUaGlzIHRlc3QgUEFTU0VEIGlmIGEgZm9jdXMgcmluZyBpcyBkcmF3biBhcm91bmQg dGhlIHBlcmltZXRlciBvZiB0aGUgJmx0O2FyZWEmZ3Q7IGluIHRoZSBpbWFnZW1hcCAoYmVsb3cp LjwvcD4KKzxtYXAgbmFtZT0iaW1hZ2VtYXAiPgorICAgIDxhcmVhIGlkPSJhcmVhIiBzaGFwZT0i cmVjdCIgY29vcmRzPSIwLDAsMjU2LDI1NiIgaHJlZj0iI2R1bW15IiAvPgorPC9tYXA+Cis8aW1n IHNyYz0iaW1hZ2VtYXAuanBnIiB3aWR0aD0iMjU2IiBoZWlnaHQ9IjI1NiIgdXNlbWFwPSIjaW1h Z2VtYXAiIGlzbWFwIC8+Cis8L2JvZHk+Cis8L2hlYWQ+Cis8L2h0bWw+CkluZGV4OiBMYXlvdXRU ZXN0cy9mYXN0L2ltYWdlcy9pbWFnZW1hcC1mb2N1cy1yaW5nLXpvb20tc3R5bGUuaHRtbAo9PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9mYXN0L2ltYWdlcy9pbWFnZW1hcC1mb2N1cy1yaW5nLXpv b20tc3R5bGUuaHRtbAkocmV2aXNpb24gMCkKKysrIExheW91dFRlc3RzL2Zhc3QvaW1hZ2VzL2lt YWdlbWFwLWZvY3VzLXJpbmctem9vbS1zdHlsZS5odG1sCShyZXZpc2lvbiAwKQpAQCAtMCwwICsx LDIxIEBACis8aHRtbD4KKzxoZWFkPgorPHNjcmlwdD4KK3dpbmRvdy5vbmxvYWQgPSBmdW5jdGlv bigpCit7CisgICAgZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoImFyZWEiKS5mb2N1cygpOworfQor PC9zY3JpcHQ+Cis8c3R5bGU+CitpbWcgeyB6b29tOiAyLjA7IH0KKzwvc3R5bGU+Cis8L2hlYWQ+ Cis8Ym9keT4KKzxwPkFzc3VtaW5nIHRoZSBwb3J0LXNwZWNpZmljIHRoZW1lIGRyYXdzIGZvY3Vz IHJpbmdzLCB0aGlzIHRlc3QgZW5zdXJlcyB0aGF0IGEgZm9jdXMgcmluZyBpcyBkcmF3biBjb3Jy ZWN0bHkgZm9yIGFuIGltYWdlbWFwIHdoZW4gdGhlIHpvb20gc3R5bGUgaXMgYXBwbGllZCB0byB0 aGUgaW1hZ2UuIFRoaXMgdGVzdCBQQVNTRUQgaWYgYSBmb2N1cyByaW5nIGlzIGRyYXduIGFyb3Vu ZCB0aGUgcGVyaW1ldGVyIG9mIHRoZSAmbHQ7YXJlYSZndDsgaW4gdGhlIGltYWdlbWFwIChiZWxv dykuPC9wPgorPG1hcCBuYW1lPSJpbWFnZW1hcCI+CisgICAgPGFyZWEgaWQ9ImFyZWEiIHNoYXBl PSJyZWN0IiBjb29yZHM9IjAsMCwxMjgsMTI4IiBocmVmPSIjZHVtbXkiIC8+Cis8L21hcD4KKzxp bWcgc3JjPSJpbWFnZW1hcC5qcGciIHdpZHRoPSIxMjgiIGhlaWdodD0iMTI4IiB1c2VtYXA9IiNp bWFnZW1hcCIgaXNtYXAgLz4KKzwvYm9keT4KKzwvaGVhZD4KKzwvaHRtbD4K