The default bug view has changed. See this FAQ.

Lines don't show up in some Google Maps directions

RESOLVED FIXED in Firefox 16

Status

()

Core
SVG
RESOLVED FIXED
5 years ago
3 years ago

People

(Reporter: Margaret, Assigned: jwatt)

Tracking

({regression})

Trunk
mozilla17
regression
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox16+ verified, firefox17 verified)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
This happens for some sets of directions, and seems to only happen with transit direction (not walking or driving). Here's a testcase: http://goo.gl/maps/YLPH.

Tested on latest nightly (7/12). I can't reproduce on Aurora.
Keywords: regressionwindow-wanted

Comment 1

5 years ago
I've been experiencing this on Mac OS X in the latest nightly builds for a bunch of directions on Google Maps - including driving, walking, etc. Sometimes, it is ameliorated by zooming in. I'm going to try to figure out a regression range, although it's going to be difficult to do an STR, since the lines seem to sporadically work.

Comment 2

5 years ago
Last good nightly: 2012-07-09
First bad nightly: 2012-07-10

Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=b6fb3d9bd1b9&tochange=02b26fb307b4

Was able to reproduce consistently with the default zoom level at this link: http://goo.gl/maps/hSI0
Keywords: regressionwindow-wanted
Summary: Lines don't show up in Google Maps transit directions → Lines don't show up in some Google Maps directions

Updated

5 years ago

Comment 3

5 years ago
This is occurring for me in the latest nightly build on Windows as well, so I am marking the OS as All.
OS: Mac OS X → All
The first bad revision is:
changeset:   98666:6e0f0a9228ce
user:        Jonathan Watt <jwatt@jwatt.org>
date:        Mon Jul 09 02:04:56 2012 +0100
summary:     Bug 769742 - Account for nsSVGOuterSVGFrames' border/padding offset by giving nsSVGOuterSVGFrame an anonymous child to wrap its real children. r=roc.
Blocks: 769742
Component: General → SVG
OS: All → Mac OS X
Thanks for the nightly regression range Logan - made finding a narrower regression range much easier!
OS: Mac OS X → All
Hardware: x86 → All
tracking-firefox16: --- → +
Assignee: nobody → jwatt
I'm seeing this as well, though only on Win32 -- couldn't reproduce it on OSX.

Comment 7

5 years ago
Definitely still able to reproduce on OS X with http://goo.gl/maps/hSI0 in the latest nightly build.
(Assignee)

Comment 8

5 years ago
Created attachment 643081 [details] [diff] [review]
patch

The problem occurs when nsDisplayOuterSVG::Paint() passes the nsSVGOuterSVGFrame to nsSVGUtils::PaintFrameWithEffects. PaintFrameWithEffects checks the dirty area against the visual overflow rect of the element, but to do that it takes the transform returned by nsSVGUtils::GetCanvasTM with the children-only component of that transform canceled out. To get that it calls HasChildrenOnlyTransform, which after bug 769742 returns false for nsSVGOuterSVGFrame. Unfortunately bug 769742 didn't also change nsSVGUtils::GetCanvasTM to stop returning the nsSVGSVGElement's transforms for nsSVGOuterSVGFrame.
Attachment #643081 - Flags: review?(roc)
Attachment #643081 - Flags: review?(roc) → review+
(Assignee)

Comment 9

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/f805b2c3b3b4
Target Milestone: --- → mozilla17
https://hg.mozilla.org/mozilla-central/rev/f805b2c3b3b4
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
(Assignee)

Comment 11

5 years ago
Comment on attachment 643081 [details] [diff] [review]
patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 769742
User impact if declined: Directions on Goggle Maps are broken
Testing completed (on m-c, etc.): Passed mozilla-inbound and not on mozilla-central
Risk to taking this patch (and alternatives if risky): very low
String or UUID changes made by this patch: none
Attachment #643081 - Flags: approval-mozilla-aurora?
Comment on attachment 643081 [details] [diff] [review]
patch

[Triage Comment]
Google Maps regression in FF16. Approved for Aurora 16.
Attachment #643081 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
(Assignee)

Comment 13

5 years ago
Pushed https://hg.mozilla.org/releases/mozilla-aurora/rev/b13cc8063a77
status-firefox16: --- → fixed
status-firefox17: --- → fixed
Duplicate of this bug: 779432

Updated

5 years ago
Depends on: 789122
Keywords: verifyme
Verified using Firefox 16 beta 3 on Windows 7, Ubuntu 12.04 and Mac OS X 10.7 - that all the lines are properly displayed on Google maps directories (verified on the provided maps from this bug, bug 779432 and some random ones).

Mozilla/5.0 (Windows NT 6.1; rv:16.0) Gecko/20100101 Firefox/16.0
Mozilla/5.0 (X11; Linux i686; rv:16.0) Gecko/20100101 Firefox/16.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:16.0) Gecko/20100101 Firefox/16.0
status-firefox16: fixed → verified
QA Contact: simona.marcu
Verified on Firefox 17 beta 1 that all the lines are properly displayed on Google maps directory. 

Verified on Windows 7, Ubuntu 12.04 and Mac OS X 10.7:
Mozilla/5.0 (Windows NT 6.1; rv:17.0) Gecko/17.0 Firefox/17.0
Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/17.0 Firefox/17.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:17.0) Gecko/17.0 Firefox/17.0
status-firefox17: fixed → verified
mass remove verifyme requests greater than 4 months old
Keywords: verifyme
You need to log in before you can comment on or make changes to this bug.