[webkit-reviews] review denied: [Bug 16433] LOW_BANDWIDTH_DISPLAY build is broken : [Attachment 17923] Make LOW_BANDWIDTH_DISPLAY compile

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Dec 16 10:19:16 PST 2007

Darin Adler <darin at apple.com> has denied Grace Kloba <klobag at gmail.com>'s
request for review:
Bug 16433: LOW_BANDWIDTH_DISPLAY build is broken

Attachment 17923: Make LOW_BANDWIDTH_DISPLAY compile

------- Additional Comments from Darin Adler <darin at apple.com>
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.

More information about the webkit-reviews mailing list