62860 2011-06-17 06:35:00 -0700 Web Inspector: Highlighting broken while navigating the tabs 2011-06-30 01:31:25 -0700 1 1 1 Unclassified WebKit Web Inspector (Deprecated) 528+ (Nightly build) All All RESOLVED FIXED P2 Normal --- 1 aandrey podivilov apavlov bweinstein joepeck keishi loislo pfeldman pmuellr podivilov rik timothy yurys oldest_to_newest 422588 0 aandrey 2011-06-17 06:35:00 -0700 1. Go to a web page, open the DevTools on the Console tab 2. Hit refresh (Ctrl+F5) on the console tab 3. Go to the scripts tab, and observe the script is not highlighted NOTE: This bug exists on Chrome 14.0.795.0 (Developer Build 4a50d71), but not in Chrome 13.0.782.15 (Official Build 88529) dev. 422594 1 aandrey 2011-06-17 06:58:37 -0700 Updated steps to repro: 1. Go to a web page, open the DevTools on the Scripts tab 2. Open console by pressing the ESC button 3. Go to Console tab 4. Hit refresh (Ctrl+F5) on the Console tab 5. Go to the Scripts tab, and observe that the script is not highlighted 422596 2 aandrey 2011-06-17 07:01:45 -0700 I remember I fixed the very same bug here: http://trac.webkit.org/browser/trunk/Source/WebCore/inspector/front-end/inspector.js#L1195 but somebody added this code snippet before it: if (!panel.canShowAnchorLocation(anchor)) return false; thus I guess the bug appeared again. 422625 3 aandrey 2011-06-17 08:02:54 -0700 And there should be no horizontal scroll in the Scripts panel bar to repro this bug. 422696 4 podivilov 2011-06-17 10:25:20 -0700 (In reply to comment #2) > I remember I fixed the very same bug here: > http://trac.webkit.org/browser/trunk/Source/WebCore/inspector/front-end/inspector.js#L1195 > > but somebody added this code snippet before it: > > if (!panel.canShowAnchorLocation(anchor)) > return false; > > thus I guess the bug appeared again. It's unrelated since you don't click on the link in your scenario, you just switch tabs. I think this bug was there for a very long time. 422715 5 aandrey 2011-06-17 10:45:01 -0700 Yes, you right. Then the this.drawer.immediatelyFinishAnimation() call should be removed from there, and added to the place where you switch the panels. 427809 6 98689 podivilov 2011-06-27 02:22:51 -0700 Created attachment 98689 Patch. 430248 7 podivilov 2011-06-30 01:31:25 -0700 Committed r90096: <http://trac.webkit.org/changeset/90096> 98689 2011-06-27 02:22:51 -0700 2011-06-27 02:34:56 -0700 Patch. patch text/plain 1193 podivilov ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No YW5nZUxvZwppbmRleCBkNjFlMDc0Li4zMjFkODg0IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQg QEAKKzIwMTEtMDYtMjcgIFBhdmVsIFBvZGl2aWxvdiAgPHBvZGl2aWxvdkBjaHJvbWl1bS5vcmc+ CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgV2ViIElu c3BlY3Rvcjogc291cmNlIGNvZGUgaXMgbm90IGhpZ2hsaWdodGVkIGFmdGVyIGRyYXdlciBoZWln aHQgYW5pbWF0aW9ucy4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcu Y2dpP2lkPTYyODYwCisKKyAgICAgICAgKiBpbnNwZWN0b3IvZnJvbnQtZW5kL0RyYXdlci5qczoK KyAgICAgICAgKFdlYkluc3BlY3Rvci5EcmF3ZXIucHJvdG90eXBlLl9hbmltYXRlRHJhd2VySGVp Z2h0LmFuaW1hdGlvbkZpbmlzaGVkKToKKyAgICAgICAgKFdlYkluc3BlY3Rvci5EcmF3ZXIucHJv dG90eXBlLl9hbmltYXRlRHJhd2VySGVpZ2h0KToKKwogMjAxMS0wNi0yNiAgQWRhbSBCYXJ0aCAg PGFiYXJ0aEB3ZWJraXQub3JnPgogCiAgICAgICAgIFJldmlld2VkIGJ5IEtlbnQgVGFtdXJhLgpk aWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvaW5zcGVjdG9yL2Zyb250LWVuZC9EcmF3ZXIuanMg Yi9Tb3VyY2UvV2ViQ29yZS9pbnNwZWN0b3IvZnJvbnQtZW5kL0RyYXdlci5qcwppbmRleCA1NDZi NWQyLi4yNzkyMmY5IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9pbnNwZWN0b3IvZnJvbnQt ZW5kL0RyYXdlci5qcworKysgYi9Tb3VyY2UvV2ViQ29yZS9pbnNwZWN0b3IvZnJvbnQtZW5kL0Ry YXdlci5qcwpAQCAtMjkzLDYgKzI5Myw3IEBAIFdlYkluc3BlY3Rvci5EcmF3ZXIucHJvdG90eXBl ID0gewogCiAgICAgICAgIGZ1bmN0aW9uIGFuaW1hdGlvbkZpbmlzaGVkKCkKICAgICAgICAgewor ICAgICAgICAgICAgV2ViSW5zcGVjdG9yLmN1cnJlbnRQYW5lbC5yZXNpemUoKTsKICAgICAgICAg ICAgIGRlbGV0ZSB0aGlzLl9hbmltYXRpbmc7CiAgICAgICAgICAgICBkZWxldGUgdGhpcy5fY3Vy cmVudEFuaW1hdGlvbjsKICAgICAgICAgICAgIHRoaXMuc3RhdGUgPSBmaW5hbFN0YXRlOwo=