110004 2013-02-15 23:39:21 -0800 Selection doesn't change when clicked on an image element 2019-02-06 21:04:39 -0800 1 1 1 Unclassified WebKit UI Events 528+ (Nightly build) All All RESOLVED INVALID P2 Normal --- 0 nvineeth webkit-unassigned aneesh.raj.25 ap enrica jesus leviw rniwa shinyak syoichi tony oldest_to_newest 834652 0 nvineeth 2013-02-15 23:39:21 -0800 document.onselectionchange is not fired when we click on an img. I have attached simple html page using which this is reproducible. Clicking anywhere in the <p> tag results in onselectionchange being called, but this not fired when clicked on the img. Verified on Chrome 24 on Windows 7 and OSx 10.7 as well. Thanks. 834654 1 188696 nvineeth 2013-02-15 23:41:39 -0800 Created attachment 188696 test html page to reproduce the problem. 835402 2 aneesh.raj.25 2013-02-18 05:55:40 -0800 Returning 'true' from HTMLImageElement::CanStartSelection() will fix this issue. 835404 3 188866 aneesh.raj.25 2013-02-18 05:57:40 -0800 Created attachment 188866 Patch for HTMLImageElement.cpp Please review. 835621 4 rniwa 2013-02-18 10:48:51 -0800 This isn't about selectionchange event not being fired. It's just that WebKit doesn't move selection when clicked on an image element. Given that this is also the behavior observed on Firefox and native applications like TextEdit, I don't think we should change our behavior. 835622 5 188866 rniwa 2013-02-18 10:50:20 -0800 Comment on attachment 188866 Patch for HTMLImageElement.cpp r- because the current behavior is desirable. 836815 6 tony 2013-02-19 15:05:35 -0800 It looks like this is an invalid bug report. FWIW, I get the same behavior in Firefox as I do in WebKit, which makes me think that the existing behavior is correct. 1503340 7 lforschler 2019-02-06 09:19:14 -0800 Mass move bugs into the DOM component. 1503622 8 rniwa 2019-02-06 21:04:39 -0800 Another bug which belongs to UI Events. 188696 2013-02-15 23:41:39 -0800 2013-02-15 23:41:39 -0800 test html page to reproduce the problem. onSelectionChange.html.zip application/zip 532 nvineeth UEsDBBQACAAIAIVmUEIAAAAAAAAAAAAAAAAWABAAb25TZWxlY3Rpb25DaGFuZ2UuaHRtbFVYDAAu Nx9RojMfUfUBFAB9kk1OwzAQhfc5xTSrViqxygraNBLqhgUSCw5QOc4ksep4LHvSUCHujtOmgIqE N+Of976xn5zPKlJ8cggtd6ZI8mtBWcXCmg0WQBbe0KBiHWeqlbZBYAwMubgokjworx0XScT1HVrO yIarZXJsoe7teWO+gI8E4lBRRQYzQ808jexw06VKF5uz8Cj9eBgZg7YVDVmD/H2l+STSNcxnUbUA j9x7u/nb46fBGtLliMxKGfC1rgNyxHwmubg+JRdTCiVVp1hcATuj1UHbBqQ9DS16BG2BWx3ASS9j 29AbDpdNhNsAlDSmlOqQwTMNeES/BHUF0sWiuwZKNDRARRjAEk/QkXmbzi9gLly84egOXm3Tltmt heidIVllgz7oDistM/KNGFduXAlFXRfDEdz2XSlWYvUodiR5T/X+yXdhrC86HkobreHYiNX9g3u/ +1eTOdukYgxvSk1cftQXUEsHCFopXZlaAQAAaQIAAFBLAQIVAxQACAAIAIVmUEJaKV2ZWgEAAGkC AAAWAAwAAAAAAAAAAECkgQAAAABvblNlbGVjdGlvbkNoYW5nZS5odG1sVVgIAC43H1GiMx9RUEsF BgAAAAABAAEAUAAAAK4BAAAAAA== 188866 2013-02-18 05:57:40 -0800 2013-02-18 10:50:20 -0800 Patch for HTMLImageElement.cpp HTMLImageElement.patch text/plain 389 aneesh.raj.25 SW5kZXg6IFdlYkNvcmUvaHRtbC9IVE1MSW1hZ2VFbGVtZW50LmNwcA0KPT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQ0KLS0t IFdlYkNvcmUvaHRtbC9IVE1MSW1hZ2VFbGVtZW50LmNwcAkocmV2aXNpb24gMTM5NTQ4KQ0KKysr IFdlYkNvcmUvaHRtbC9IVE1MSW1hZ2VFbGVtZW50LmNwcAkod29ya2luZyBjb3B5KQ0KQEAgLTIw OSw3ICsyMDksNyBAQA0KICAgICBpZiAoc2hhZG93KCkpCiAgICAgICAgIHJldHVybiBIVE1MRWxl bWVudDo6Y2FuU3RhcnRTZWxlY3Rpb24oKTsKIAotICAgIHJldHVybiBmYWxzZTsKKyAgICByZXR1 cm4gdHJ1ZTsKIH0KIAogdm9pZCBIVE1MSW1hZ2VFbGVtZW50OjphdHRhY2goKQo=