Bug 43209

Summary: [Qt] QtTestBrowser: -resizes-to-contents command line parameter is broken
Product: WebKit Reporter: Antonio Gomes <tonikitoo>
Component: Tools / TestsAssignee: Antonio Gomes <tonikitoo>
Status: RESOLVED FIXED    
Severity: Normal CC: kenneth
Priority: P2 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: PC   
OS: All   
Attachments:
Description Flags
patch v1 none

Description Antonio Gomes 2010-07-29 13:53:52 PDT
try running the launcher with the following command line:

$ run-launcher --qt -graphicsbased -resizes-to-contents

I'll launcher in GraphicsView mode and ResizesToContents (Developers->GraphicsView->Toggle Resizes To Contents Mode) menu item toggle ON, however the property is not set in fact.

Browser through sites that require scrollbars, and they will show up.

It applies to other supported command line parameters too.
Comment 1 Antonio Gomes 2010-07-29 18:46:11 PDT
Lets fix -resizes-to-contents for now
Comment 2 Antonio Gomes 2010-07-29 18:54:05 PDT
Created attachment 63021 [details]
patch v1

When -resizes-to-content was being passed as a command line parameter it was simply not being set (although the corresponding menu item was marked as ON). User had to toggle the menu OFF and then ON again for it to take place.

Reason: LauncherWindow::applyPrefs method sets many user options passed in from the command line, but not resizesToContents. Patch addresses that.
Comment 3 Antonio Gomes 2010-07-30 06:34:40 PDT
Comment on attachment 63021 [details]
patch v1

Clearing flags on attachment: 63021

Committed r64354: <http://trac.webkit.org/changeset/64354>