Closed
Bug 14801
Opened 25 years ago
Closed 25 years ago
cbs.marketwatch.com images don't load
Categories
(Core :: Networking, defect, P3)
Tracking
()
VERIFIED
FIXED
M14
People
(Reporter: cathleennscp, Assigned: andreas.otte)
References
()
Details
(Whiteboard: [PDT+])
Attachments
(5 files)
(deleted),
text/html
|
Details | |
(deleted),
text/plain
|
Details | |
(deleted),
text/plain
|
Details | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review |
1. maximize your apprunner window
2. go to
http://cbs.marketwatch.com/news/current/snapshot.htx?source=htx/http2_mw
3. see the page gets layed out correctly, and all of the sudden content shifts
to the right...
4. hit reload, same thing happens
5. resize window, content doesn't get reflow
Appears to be a table problem. This looks similar to bug #14489
Assignee: troy → karnaze
Component: Layout → HTMLTables
Comment 2•25 years ago
|
||
It appears to be a problem caused by the alt text is displayed instead of the
sidebar images, on unsuccessful retrieving.
The image name is very long and that's what pushes the table too wide.
Table is fixed width so it shouldn't be resized on window resize.
Updated•25 years ago
|
Assignee: karnaze → kipp
Comment 3•25 years ago
|
||
Yes, the <img
src="http://ad.doubleclick.net/ad/cbsmw.button.com/SIDEBAR_BUTTONS;sz=88x31;kw=D
BCC;tile=4;ord=1864641213378545414" width="88" height="31" vspace="5" border=0>
is causing the column to be too wide. Reassigning to Kipp.
Well, the real problem is that we can't load that image with necko. So I'm
bouncing this one over there to be fixed. The layout after the image stops being
an image is reasonable.
Came we bump up the target milestone for this bug? like, by beta or something?
This link is off of my.netscape page... CBS MarketWatch, and most of their
pages are having this layout problem.
Updated•25 years ago
|
Status: NEW → ASSIGNED
Target Milestone: M17 → M11
Updated•25 years ago
|
Assignee: warren → rpotts
Status: ASSIGNED → NEW
Comment 8•25 years ago
|
||
Rick, can you check out why the socket transport isn't loading the image in the
attachment file that Kipp gave:
http://bugzilla.mozilla.org/showattachment.cgi?attach_id=2289
Also, I'm seeing the text bump over after attempting to load this image,
displaying this text:
SIDEBAR_BUTTONS;sg=88x31;kw=DBCC;tile=2;ord=186443241356908581748
so maybe there are 2 bugs here.
Comment 9•25 years ago
|
||
Comment 10•25 years ago
|
||
Comment 11•25 years ago
|
||
It appears that the image URL gets redirected twice...
first to - Location: http://ad.doubleclick.net/viewad/249085-buygolf_88x31.gif
and then - Location:
http://creatives.doubleclick.net/viewad/249085-buygolf_88x31.gif%3Bsz%3D88x31%3B
kw%3DDBCC%3Btile%3D1%3Bord%3D1865239613507480
However, it appears that the image is actually loaded by HTTP as a 43 byte
image/gif stream...
I'll need to track down where the data goes from there...
Comment 12•25 years ago
|
||
Looking at the HTTP log...
The first redirect told mozilla to download
(1) http://ad.doubleclick.net/viewad/249085-buygolf_88x31.gif
but it attempted to download
(2) http://ad.doubleclick.net/viewad/249085-buygolf_88x31.gif;sz=88x31;kw=DBCC;
tile=1;ord=186523961350748063055
(Looks like an URL parsing error)
When I view (1) in Nav4 I get the correct picture.
When I view (2) in Nav4 I get an 1x1 gif (43 bytes length).
And that is how it happens in Mozilla too!
The difference is that when I load the URL Mozilla tried to get
http://ad.doubleclick.net/ad/cbsmw.button.com/SIDEBAR_BUTTONS;sz=88x31;
kw=DBCC;tile=1;ord=186523961350748063055
that does get an advertisement image on Nav4, but on Mozilla I get
a 404 Not Found error (which I don't copy here because Copy doesn't work...)
If I hit reload I got the error message back, but if I click on the URL bar
and press ENTER I got the 1x1 image...
Updated•25 years ago
|
Assignee: rpotts → gagan
Summary: page not rendering correctly → cbs.marketwatch.com images don't load
Comment 13•25 years ago
|
||
Changing summary from: page not rendering correctly
to: cbs.marketwatch.com images don't load
Gagan can you take a look at this one for Rick? Thanks.
Updated•25 years ago
|
Target Milestone: M11 → M12
Comment 14•25 years ago
|
||
this is our new anti-ad-banner feature for m11.;-)
moving to m12. let me know if I've missed something.
Comment 15•25 years ago
|
||
Moving Assignee from gagan to warren since he is away.
Assignee | ||
Comment 16•25 years ago
|
||
I think there are two bugs here:
1. An urlparser bug
2. Something in layout which damages links, highly unpredictable ...
The urlparser bug happens because the current parser does not account for
multiple params, the last one currently wins, the constructed url is false and
mapped to the grey 1x1 gif. Some time ago Kipp fixed this problem, but the
solution was rejected by Gagan. I have revived that stuff locally in a little
modified version and the page loads successfully.
However sometimes bug 2 still happens ...
Assignee | ||
Comment 17•25 years ago
|
||
Assignee | ||
Comment 18•25 years ago
|
||
Comment 19•25 years ago
|
||
Moving what's not done for M12 to M13.
Comment 20•25 years ago
|
||
As far as I can tell, ALL Doubleclick.com ad redirects use this
";" multi-parameter syntax... probably other ad servers do too.
For reference, the use of multiple parameters separated by ";" is defined in
section 3.3 of RFC 2396, "Uniform Resource Identifiers (URI): Generic Syntax",
<URL:http://www.ietf.org/rfc/rfc2396.txt>. Quoting:
>3.3. Path Component
>
>The path component contains data, specific to the authority (or the
>scheme if there is no authority component), identifying the resource
>within the scope of that scheme and authority.
>
> path = [ abs_path | opaque_part ]
>
> path_segments = segment *( "/" segment )
> segment = *pchar *( ";" param )
> param = *pchar
>
> pchar = unreserved | escaped |
> ":" | "@" | "&" | "=" | "+" | "$" | ","
>
>The path may consist of a sequence of path segments separated by a
>single slash "/" character. Within a path segment, the characters
>"/", ";", "=", and "?" are reserved. Each path segment may include a
>sequence of parameters, indicated by the semicolon ";" character.
>The parameters are not significant to the parsing of relative
>references.
Comment 21•25 years ago
|
||
*** Bug 20026 has been marked as a duplicate of this bug. ***
Comment 22•25 years ago
|
||
*** Bug 23097 has been marked as a duplicate of this bug. ***
Updated•25 years ago
|
Component: HTMLTables → Networking
Comment 23•25 years ago
|
||
Changing Component from HTMLTables to Networking.
Comment 24•25 years ago
|
||
*** Bug 22715 has been marked as a duplicate of this bug. ***
Assignee | ||
Updated•25 years ago
|
Assignee: warren → andreas.otte
Assignee | ||
Comment 25•25 years ago
|
||
assigning this bug to me per Warrens request
Assignee | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
Updated•25 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Comment 26•25 years ago
|
||
Fixed by Andreas' changes that I just checked in.
Assignee | ||
Updated•25 years ago
|
Status: RESOLVED → REOPENED
Assignee | ||
Comment 27•25 years ago
|
||
reopened because of backout
Assignee | ||
Updated•25 years ago
|
Target Milestone: M13 → M14
Assignee | ||
Comment 28•25 years ago
|
||
This will not make it into M13
Comment 29•25 years ago
|
||
Clearing FIXED resolution due to reopen.
Comment 30•25 years ago
|
||
*** Bug 25340 has been marked as a duplicate of this bug. ***
Comment 31•25 years ago
|
||
*** Bug 25355 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 33•25 years ago
|
||
finally fixed
Status: REOPENED → RESOLVED
Closed: 25 years ago → 25 years ago
Resolution: --- → FIXED
Comment 34•25 years ago
|
||
Since component has been changed to networking, I 'm reassigning QA contact to
tever.
QA Contact: petersen → tever
You need to log in
before you can comment on or make changes to this bug.
Description
•