Bug 55641 - REGRESSION(r70170) - google images does not work as expected with iphone user agent.
Summary: REGRESSION(r70170) - google images does not work as expected with iphone user...
Status: RESOLVED WORKSFORME
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar, NeedsReduction, Regression
Depends on:
Blocks:
 
Reported: 2011-03-02 18:35 PST by Luiz Agostini
Modified: 2015-03-20 21:13 PDT (History)
15 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Luiz Agostini 2011-03-02 18:35:48 PST 
When using iphone user agent, google images does not work as expected. It is a lot more wide that it is supposed to be.
It seems to be happening since https://trac.webkit.org/changeset/70170.

The tests have been made with Qt and the used user agents were:
    Mozilla/5.0 (iPhone; U; CPU OS 3_2 like Mac OS X; en-us) AppleWebKit/531.21.10 (KHTML, like Gecko) Version/4.0.4 Mobile/7B334b Safari/531.21.10
    Mozilla/5.0 (iPhone; U; CPU iPhone OS 4_0 like Mac OS X; en-us) AppleWebKit/532.9 (KHTML, like Gecko) Version/4.0.5 Mobile/8A293 Safari/6531.22.7
Comment 1 Kenneth Rohde Christiansen 2011-03-03 03:52:23 PST 
Info from Andreas Kling who did some bisecting of this:

r64439 works, where as 65682 does not.
Comment 2 Kenneth Rohde Christiansen 2011-03-03 03:53:33 PST 
Kling you bisecting seems to not match with Luiz findings.
Comment 3 Andreas Kling 2011-03-03 04:14:29 PST 
(In reply to comment #2)
> Kling you bisecting seems to not match with Luiz findings.

I have the bug on r65268 here FWIW
Comment 4 Kenneth Rohde Christiansen 2011-03-03 05:12:42 PST 
I should probably make it clean that it doesn't break the google images front page, but the page shown after you have searched for something
Comment 5 Luiz Agostini 2011-03-03 07:06:16 PST 
I tried again and seems to be on r70170 indeed. We may be testing different issues. What I do is: 

 1 - select user agent: Mozilla/5.0 (iPhone; U; CPU OS 3_2 like Mac OS X; en-us) AppleWebKit/531.21.10 (KHTML, like Gecko) Version/4.0.4 Mobile/7B334b Safari/531.21.10
 2 - go to www.google.com
 3 - click on 'imagens' (portuguese)
 4 - type 'cars'
 5 - enter

On r70169 no horizontal scrollbar is shown. On r70170 a scrollbar can be seen, the content width seems to be twice larger then on r70169 and if you scroll to the right you see the images that are expected to be in the second page.

I am testing on Linux.
Comment 6 David Kilzer (:ddkilzer) 2011-03-03 18:17:40 PST 
<rdar://problem/9085209>
Comment 7 Kenneth Rohde Christiansen 2011-04-11 02:35:44 PDT 
Hyatt, do you have any idea where to look in order to fix this? like what might have broken it?
Comment 8 Simon Fraser (smfr) 2011-04-11 08:57:23 PDT 
Sounds like taking transformed into account for overflow broke this. Google should fix their site.
Comment 9 Kenneth Rohde Christiansen 2011-04-11 10:25:53 PDT 
(In reply to comment #8)
> Sounds like taking transformed into account for overflow broke this. Google should fix their site.

Eric, do you know who at Google can look into this?
Comment 10 Eric Seidel (no email) 2011-04-11 10:28:28 PDT 
Donno.  Other googlers might know someone on the images team.
Comment 11 Matthew Delaney 2011-05-24 12:09:31 PDT 
Adding a few more googlers (probably all the wrong ones, oh well). Do you guys have any internal mechanism for alerting the Google Images team? Or at least a quick way to look up one of their emails to get them on this? Thanks!
Comment 12 Emil A Eklund 2011-05-24 12:14:41 PDT 
FYI: I've forwarded this bug to the google images team.
Comment 13 David Kilzer (:ddkilzer) 2015-03-20 21:13:23 PDT 
Works for me now.  I think his can be closed.