Bug 22788
| Summary: | [Transforms] Server-side image maps not transform-aware | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Simon Fraser (smfr) <simon.fraser> |
| Component: | Layout and Rendering | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | CC: | ahmad.saleem792, frances_c, karlcow, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | 528+ (Nightly build) | ||
| Hardware: | Mac | ||
| OS: | OS X 10.5 | ||
| Bug Depends on: | |||
| Bug Blocks: | 15670 | ||
Simon Fraser (smfr)
There is code in HTMLAnchorElement::defaultEventHandler():
HTMLImageElement* img = static_cast<HTMLImageElement*>(evt->target()->toNode());
if (img && img->isServerMap()) {
RenderImage* r = static_cast<RenderImage*>(img->renderer());
if (r && e) {
// FIXME: broken with transforms
FloatPoint absPos = r->localToAbsolute();
int x = e->pageX() - absPos.x();
int y = e->pageY() - absPos.y();
url += "?";
url += String::number(x);
url += ",";
url += String::number(y);
that is not transform-aware
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Frances Cornwall
The code that was not transform-aware was removed from HTMLAnchorElement.cpp.
Karl Dubost
Maybe @Simon should close this one.
Ahmad Saleem
Related function (?) `appendServerMapMousePosition` has following FIXME:
// FIXME: This should probably pass UseTransforms in the OptionSet<MapCoordinatesMode>.
https://searchfox.org/wubkat/rev/1ade9b72a383daa2aee2e964d15c42877b9fb999/Source/WebCore/html/HTMLAnchorElement.cpp#157
It might still be needed? (@Simon)
Radar WebKit Bug Importer
<rdar://problem/127402573>