[Webkit-unassigned] [Bug 16433] LOW_BANDWIDTH_DISPLAY build is broken

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=16433





------- Comment #4 from klobag at gmail.com  2007-12-16 15:32 PDT -------
The change in the FrameLoader.h was in the original patch. Android doesn't need
it. But we discovered that we had to add it to avoid crash in the Mac port back
then. I am happy to remove it.

(In reply to comment #3)
> (From update of attachment 17923 [edit])
> The rest of the changes look fine, but the change to FrameLoader doesn't seem
> to have anything to do with LOW_BANDWIDTH_DISPLAY.
> 
> I'm guessing that for some reason Android derives a subclass from FrameLoader.
> That doesn't sound like a good idea to me, but also seems unrelated to
> LOW_BANDWIDTH_DISPLAY so should not be controlled by the same ifdef.
> 


-- 
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.