Bug 46347

Summary:

[Qt] Remove setDeviceSize methods

Product:

WebKit

Reporter:

Kenneth Rohde Christiansen <kenneth>

Component:

WebKit Qt

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Major

CC:

ademar, hausmann, kling

Priority:

Keywords:

Qt, QtTriaged

Version:

528+ (Nightly build)

Hardware:

OS:

All

Bug Depends on:

Bug Blocks:

31552

Attachments:

Description	Flags
Patch	tonikitoo: review+

Description Kenneth Rohde Christiansen 2010-09-23 06:37:51 PDT

These methods are not in 2.0 and we do not want them in 2.1. Instead the viewport meta tag stuff should use QDesktopServices to get the device size.

device-width is the width of the device when in portrait and the device-height is the width of the device when in landscape.

It would be nice to have some environment variables to overwrite the values when testing mobile browsers on the desktop.

Comment 1 Kenneth Rohde Christiansen 2010-09-23 06:55:11 PDT

QDesktopWidget should have what we need. We just need to make sure that the geometry gives the actual screen geometry and doesn't subtract things like the cellphone status area.

Comment 2 Kenneth Rohde Christiansen 2010-09-27 10:39:39 PDT

Created attachment 68928 [details]
Patch

Comment 3 Kenneth Rohde Christiansen 2010-09-27 17:31:26 PDT

Fixed (after conversation on irc) version landed in r68457

Comment 4 Simon Hausmann 2010-10-05 06:29:16 PDT

<cherry-pick-for-backport: r68457>

Comment 5 Simon Hausmann 2010-10-05 06:29:33 PDT

<cherry-pick-for-backport: r69104>

Comment 6 Ademar Reis 2010-10-06 07:38:11 PDT

Revision r68457 cherry-picked into qtwebkit-2.1 with commit b2d63cf <http://gitorious.org/webkit/qtwebkit/commit/b2d63cf>
Revision r69104 cherry-picked into qtwebkit-2.1 with commit 69b269b <http://gitorious.org/webkit/qtwebkit/commit/69b269b>