konrad/qtdeclarative.git
12 years agoReuse QQuickView in ListView table driven tests
Martin Jones [Tue, 13 Mar 2012 05:54:19 +0000]
Reuse QQuickView in ListView table driven tests

Makes the tests run faster since we're not waiting for a
window to appear all the time.

Change-Id: Ibff3706873dbad2f92d2135eae260368f4f4bb10
Reviewed-by: Bea Lam <bea.lam@nokia.com>

12 years agoQmlProfiler: updated standalone app
Christiaan Janssen [Thu, 8 Mar 2012 15:50:14 +0000]
QmlProfiler: updated standalone app

Using a stripped-down version of the profiler data structure.

Change-Id: I93a0b12462edea0ca8a1d0db42aa892aa2afc919
Reviewed-by: Kai Koehne <kai.koehne@nokia.com>

12 years agoDefault image scaling behaviour has changed; fix autotest.
Toby Tomkins [Tue, 13 Mar 2012 01:17:54 +0000]
Default image scaling behaviour has changed; fix autotest.

The default image scaling behaviour has changed in qtdeclarative. This
change fixes the autotest to conform to fixed ratio scaling.

Change-Id: I9c901825295fb04cab74c46f61696c5274ddd943
Reviewed-by: Martin Jones <martin.jones@nokia.com>

12 years agoFix crash when writing property after engine deletion
Chris Adams [Mon, 12 Mar 2012 01:54:11 +0000]
Fix crash when writing property after engine deletion

Previously, the engine pointer stored in a QQmlProperty could be
stale due to engine deletion.  This commit ensures we guard that
pointer.

Also reverts cbb7f8b10e99fad675839d7625d3236ed67b3e01

Task-number: QTBUG-24734
Change-Id: I5349c51fbd19fa46a8710280173c1d224358b96e
Reviewed-by: Michael Brasser <michael.brasser@nokia.com>

12 years agoOnly attempt to load QML test cases from source directory if it exists.
Aaron McCarthy [Mon, 27 Feb 2012 01:11:10 +0000]
Only attempt to load QML test cases from source directory if it exists.

When running installed unit tests the QML test cases are likely not
in the source directory, but in the same directory as the installed
test binary.  If the source directory does not exist, fall back to
searching for QML test cases in the current directory.

Change-Id: I231b9cd49b12d88dd6cfbcc8dfb8b6c2c69d5aad
Reviewed-by: Yunqiao Yin <charles.yin@nokia.com>

12 years agoRefactor XMLHttpRequest example.
Charles Yin [Tue, 6 Mar 2012 02:13:44 +0000]
Refactor XMLHttpRequest example.

Change-Id: I4fd4fa4fb190814e1e32ce847d05ab0015ad4ca0
Reviewed-by: Alan Alpert <alan.alpert@nokia.com>

12 years agoRefactor localstorage example
Charles Yin [Tue, 6 Mar 2012 02:03:36 +0000]
Refactor localstorage example

Change-Id: I0c742c35ba8d5a2812dc1665905e45f5ffb745b5
Reviewed-by: Alan Alpert <alan.alpert@nokia.com>

12 years agoMove binding and expression classes to separate files
Aaron Kennedy [Thu, 8 Mar 2012 13:25:50 +0000]
Move binding and expression classes to separate files

Change-Id: Ia9c6996a606e140f31681ecd26d93b1b0fdedf02
Reviewed-by: Roberto Raggi <roberto.raggi@nokia.com>

12 years agoSpeed up view flick tests.
Martin Jones [Tue, 13 Mar 2012 00:32:48 +0000]
Speed up view flick tests.

Change-Id: I98b0a45b5afd0150bfacef2c1c1e814f70ae042f
Reviewed-by: Bea Lam <bea.lam@nokia.com>

12 years agotestlib: Improve verbose and XPASS output
Jason McDonald [Wed, 29 Feb 2012 05:36:21 +0000]
testlib: Improve verbose and XPASS output

Call only the eight-argument version of QTestResult::compare(), as the
four-argument version will soon be removed from the API.

Change-Id: I9e7b95a30b01fb5e084f2954aeb380b492760484
Reviewed-by: Rohan McGovern <rohan.mcgovern@nokia.com>

12 years agoListView.contentHeight does not include header and footer item heights
Martin Jones [Mon, 12 Mar 2012 07:55:38 +0000]
ListView.contentHeight does not include header and footer item heights

Task-number: QTBUG-24664

Change-Id: I2a2fe144835e61665b8693d58ca4fb9ba506e8a8
Reviewed-by: Bea Lam <bea.lam@nokia.com>

12 years agoFix leak in tst_qquicklistmodel unit test
Chris Adams [Mon, 12 Mar 2012 03:39:45 +0000]
Fix leak in tst_qquicklistmodel unit test

Change-Id: I6883f263648d425240503f6abbd358b744d68506
Reviewed-by: Glenn Watson <glenn.watson@nokia.com>

12 years agoDisplaced items were moving unnecessarily
Bea Lam [Mon, 5 Mar 2012 08:06:33 +0000]
Displaced items were moving unnecessarily

They should only move if they actually change from the last set
position, and not if they are simply changing from their current item
position, as that is wrong during an animation.

This also cleans up some code for resetting the transition data.

Task-number: QTBUG-24586
Change-Id: I0a6635903975ebc40d5cf8398b943a9de92d4493
Reviewed-by: Martin Jones <martin.jones@nokia.com>

12 years agoImprove interaction of nested Flickables and PathView
Martin Jones [Fri, 9 Mar 2012 05:40:29 +0000]
Improve interaction of nested Flickables and PathView

Don't require a flick to come to a complete stop before
allowing another flickable element to begin its
gesture.

Change-Id: I74c1998e01e04c70c76253cd09edc02f593123d0
Reviewed-by: Michael Brasser <michael.brasser@nokia.com>

12 years agoDeclare change set and compositor structs as movable primative types.
Andrew den Exter [Fri, 9 Mar 2012 01:02:03 +0000]
Declare change set and compositor structs as movable primative types.

Allows QVector to avoid invoking contructors for these types
unnecessarily.

Change-Id: Icc8670e4a033c943666f907c7e1b006a3d205597
Reviewed-by: Bea Lam <bea.lam@nokia.com>

12 years agoChange repeater item to handle model being deleted.
Glenn Watson [Mon, 12 Mar 2012 03:20:22 +0000]
Change repeater item to handle model being deleted.

The repeater item previously stored a raw QObject pointer in
a variant. When this pointer was a dynamic list model element
that was deleted, the variant would continue to hold a stale
pointer. Change repeater to use a guard object to hold the model
when it is a QObject. Continue to use a variant to hold models
that are not based on QObject to maintain same semantics.

Change-Id: Ie100947132923803263c725e86efa68206382f12
Reviewed-by: Martin Jones <martin.jones@nokia.com>

12 years agoFix SignalSpy disconnect issue
Charles Yin [Wed, 7 Mar 2012 13:12:26 +0000]
Fix SignalSpy disconnect issue

Task-number:QTBUG-21083
Change-Id: Ic61472d886bf7c46eb37dc44b5e5ceb401917087
Reviewed-by: Michael Brasser <michael.brasser@nokia.com>

12 years agoRemove the rest of qtquick 1.0 code from qmltest
Charles Yin [Thu, 8 Mar 2012 04:04:25 +0000]
Remove the rest of qtquick 1.0 code from qmltest

Change-Id: I0d20583faf1a75f287dcd866b11457254d250354
Reviewed-by: Matthew Vogt <matthew.vogt@nokia.com>
Reviewed-by: Michael Brasser <michael.brasser@nokia.com>

12 years agoinclude header correctly
Charles Yin [Wed, 7 Mar 2012 13:12:19 +0000]
include header correctly

Change-Id: I462fbcdd2954979fbff92965cc813551773b4118
Reviewed-by: Michael Brasser <michael.brasser@nokia.com>

12 years agoFix for QTBUG-22777. Removed unnecessary check.
Gatis Paeglis [Tue, 14 Feb 2012 18:04:12 +0000]
Fix for QTBUG-22777. Removed unnecessary check.

Change-Id: I82de9e1d6eb4b8bdb8c90c69e686b700a02f40aa
Reviewed-by: Martin Jones <martin.jones@nokia.com>

12 years agoFix v4 nested object access bug.
Michael Brasser [Mon, 12 Mar 2012 05:27:09 +0000]
Fix v4 nested object access bug.

Use the correct type to look up the sub property.

Task-number: QTBUG-24606
Change-Id: Iab372ca7440249241bf855d2875a947ad8fe4527
Reviewed-by: Chris Adams <christopher.adams@nokia.com>
Reviewed-by: Roberto Raggi <roberto.raggi@nokia.com>

12 years agoAdd additional v4 tests.
Michael Brasser [Fri, 9 Mar 2012 03:09:07 +0000]
Add additional v4 tests.

Change-Id: Id33eb50e4a01f3450515f6e666e18cb018f1706e
Reviewed-by: Roberto Raggi <roberto.raggi@nokia.com>

12 years agoAdd two argument Math.min and Math.max support to v4.
Michael Brasser [Thu, 8 Mar 2012 00:36:51 +0000]
Add two argument Math.min and Math.max support to v4.

Change-Id: I26a4812b76571bf0ae6a0b4bd5d1e2afd64051df
Reviewed-by: Roberto Raggi <roberto.raggi@nokia.com>

12 years agoAdd Math.ceil and Math.abs support to v4.
Michael Brasser [Wed, 7 Mar 2012 23:16:07 +0000]
Add Math.ceil and Math.abs support to v4.

Change-Id: I23e7c8294abf80914d4529740af6e9124f66c5bf
Reviewed-by: Roberto Raggi <roberto.raggi@nokia.com>

12 years agoCheck that canvas is valid.
Frederik Gladhorn [Tue, 6 Mar 2012 10:25:59 +0000]
Check that canvas is valid.

During creation the canvas might not have been set.

Change-Id: I7ea8922413e3b6fb4ffd2ed9758c57eea1de02db
Reviewed-by: Jan-Arve Sæther <jan-arve.saether@nokia.com>

12 years agoPrefer \b word boundary markers
Matthew Vogt [Mon, 12 Mar 2012 06:53:24 +0000]
Prefer \b word boundary markers

/usr/bin/sed on mac does not process '\<' and '\>' boundary markers
correctly.

Change-Id: I750f7c69579ddcbe39a6246a3d3321cab714fc27
Reviewed-by: Rohan McGovern <rohan.mcgovern@nokia.com>

12 years agotests: enable headersclean also for cross_compile builds
Rohan McGovern [Mon, 12 Mar 2012 00:01:56 +0000]
tests: enable headersclean also for cross_compile builds

This test is no longer a host-only test.
Enable it everywhere.

Change-Id: Id7e65ad4d6f468b6b65d0f6035934fbd20fcb5e0
Reviewed-by: Toby Tomkins <toby.tomkins@nokia.com>

12 years agoAccessibility: childAt_helper descends too far.
Morten Johan Sorvig [Thu, 19 Jan 2012 07:02:09 +0000]
Accessibility: childAt_helper descends too far.

Recursive hit-testing using childAt_helper must stop
when it hits a leaf accessibility item. This leaf item
might not be a leaf item in Qt Quick - a button contains
text and mouse area items not visible to the accessibility
system for example.

Get the accessible interface and check if it has
children before recursing to the child items.

Change-Id: I2cc286cde4bce6b4ca340e8a1819edc5f84006a5
Reviewed-by: Jan-Arve Sæther <jan-arve.saether@nokia.com>
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@nokia.com>

12 years agoRemove unneeded dependencies to QtWidgets and QtOpenGL
Lars Knoll [Wed, 29 Feb 2012 19:43:09 +0000]
Remove unneeded dependencies to QtWidgets and QtOpenGL

Change-Id: I43bb54524f5786a838073df8812107dda7b0d56e
Reviewed-by: Kent Hansen <kent.hansen@nokia.com>

12 years agoFix MouseArea autotest.
Laszlo Agocs [Mon, 12 Mar 2012 06:40:52 +0000]
Fix MouseArea autotest.

The hover test seems to be simply wrong. The double click tests worked
before due to the obscure handling of double click but were somewhat
incorrect in any case.

Change-Id: Ie5690e26eb5921fe149e128d24d331f52a5a9075
Reviewed-by: Kent Hansen <kent.hansen@nokia.com>

12 years agoFix tst_qquicktextedit.
Michael Brasser [Mon, 12 Mar 2012 00:55:25 +0000]
Fix tst_qquicktextedit.

Recent changes to mouse double-click handling mean that two
calls to QTest::mouseClick with the same position will result in a
double click, which was causing this test to fail.

Change-Id: I21e6aa7ab2221b6ea82b1afd13e36b7ba0aef8dc
Reviewed-by: Kent Hansen <kent.hansen@nokia.com>

12 years agoFix double click handler in QQuickItem.
Laszlo Agocs [Fri, 9 Mar 2012 10:00:29 +0000]
Fix double click handler in QQuickItem.

The legacy, desktop-oriented way of calling mousePressed from the
default handlers for mouseDoubleClick is not working properly with
QWindow and QGuiApplication: In Qt 5 a double click will never
suppress any lower level events (like press or release) and therefore
simulating a mouse press in the double click handlers is wrong.

Change-Id: Ic0ad97a5efbcd3deb1717229010dcb84681d7ed4
Reviewed-by: Kent Hansen <kent.hansen@nokia.com>

12 years agoRemove pin of qtbase for qtdeclarative.
Sergio Ahumada [Tue, 6 Mar 2012 12:17:16 +0000]
Remove pin of qtbase for qtdeclarative.

Change-Id: Ib70a3438d085adbed3a1ca7df81d593c7a512ae0
Reviewed-by: Kent Hansen <kent.hansen@nokia.com>

12 years agoSkip test that accesses deleted QML engine
Kent Hansen [Mon, 12 Mar 2012 12:13:48 +0000]
Skip test that accesses deleted QML engine

The test is blocking the CI because it crashes on linux/x64.
By "luck" it doesn't crash on the other test machines -- valgrind
still complains, though.

Task-number: QTBUG-24734
Change-Id: I6bc12ca72f8900339bc9139b40cf828aff8e47ba
Reviewed-by: Lars Knoll <lars.knoll@nokia.com>

12 years agoFix flakiness in qquicklistmodel autotest
Kent Hansen [Mon, 12 Mar 2012 15:02:43 +0000]
Fix flakiness in qquicklistmodel autotest

The worker_remove_element test calls processEvents() before
calling waitForWorker(). It's possible that the worker actually
finishes during the processEvents() call. In such a situation,
waitForWorker() should return right away; otherwise it would
wait for 10000ms for a signal that had already emitted, and the
test would fail.

Change-Id: I8e98a3297cf5f360c1c405b1baa7524cc6593d81
Reviewed-by: Lars Knoll <lars.knoll@nokia.com>

12 years agoMark presumed unstable test as insignificant.
Lars Knoll [Mon, 12 Mar 2012 16:01:16 +0000]
Mark presumed unstable test as insignificant.

Change-Id: Iea22ae6596c04ba95d59bcb3957bdc3cf9263ae2
Reviewed-by: Kent Hansen <kent.hansen@nokia.com>

12 years agoReturn early if animationjob was deleted in all callback methods.
Charles Yin [Fri, 9 Mar 2012 01:34:18 +0000]
Return early if animationjob was deleted in all callback methods.

Change-Id: I703c8a8904ffd760683de4ee74544dff9eb53144
Reviewed-by: Michael Brasser <michael.brasser@nokia.com>

12 years agoRemove undefined method.
Stephen Kelly [Wed, 7 Mar 2012 20:17:02 +0000]
Remove undefined method.

Change-Id: Id95aebcfc5beac0154db4b1137fc08d5172f3428
Reviewed-by: Aaron Kennedy <aaron.kennedy@nokia.com>

12 years agoMake headersclean in qtdeclarative enforce correct header conventions.
Rohan McGovern [Mon, 5 Mar 2012 05:55:31 +0000]
Make headersclean in qtdeclarative enforce correct header conventions.

This test wasn't testing much before.
Refactor it to reuse headersclean from qtbase.

It will now fail to compile if the headers from this module don't follow
the rules.

Change-Id: I2816b85543da74f66b993b7ee705cdb9e1c0f1c8
Reviewed-by: Jason McDonald <jason.mcdonald@nokia.com>

12 years agorefilled items should be moved immediately
Bea Lam [Mon, 5 Mar 2012 08:05:40 +0000]
refilled items should be moved immediately

refill() functionality should reposition items immediately, else
removeNonVisibleItems() sees different positions from those added in
addVisibleItems() if an item is animating.

Change-Id: Ib9904e08bf92b18fd4b712270c0ab69e9a113e04
Reviewed-by: Martin Jones <martin.jones@nokia.com>

12 years agoClean up transition objects for positioners
Bea Lam [Wed, 7 Mar 2012 08:20:06 +0000]
Clean up transition objects for positioners

These must be cleaned up whenever items are removed from the internal
PositionedItem lists.

Change-Id: I56f64a29bd98c603b00faaead514fd43780c18de
Reviewed-by: Alan Alpert <alan.alpert@nokia.com>

12 years agoAdd mouse wheel events handler to MouseArea
Luis Gabriel Lima [Thu, 5 Jan 2012 18:19:26 +0000]
Add mouse wheel events handler to MouseArea

This patch was based on the attached patch in QTBUG-7369. It basically
exposes the wheel events to MouseArea via the onWheel signal.

The current API is based on the new QWheelEvent API introduced by this
patch: http://codereview.qt-project.org/#change,12532

Task-number: QTBUG-7369
Change-Id: Id58513715c2d0ae81e3a69e9e1ed400bbae07507
Reviewed-by: Michael Brasser <michael.brasser@nokia.com>

12 years agoAvoid using QQuickViewItem as base class
Bea Lam [Wed, 7 Mar 2012 03:29:20 +0000]
Avoid using QQuickViewItem as base class

This patch renames it to QQuickItemViewTransitionableItem, and
FxViewItem and PositionedItem now create instances of this instead of
subclassing it, to reduce the memory used when transitions are not
required.

Change-Id: Ie050cda5a121bff9542c7ba7356f6eacf37d241a
Reviewed-by: Alan Alpert <alan.alpert@nokia.com>

12 years agoDon't grow container when desired size is known
João Abecasis [Wed, 7 Mar 2012 22:17:54 +0000]
Don't grow container when desired size is known

QList<Type>::reserve() is used upfront to allocate necessary memory in a one
go. This tells us straight away whether allocation is possible at all and
reduces re-allocations and consequent memory copies.

This also has the side effect that no spare memory is allocated, also allowing
up to (and including) INT_MAX elements to actually be stored in the underlying
QList, as long as enough memory is available to satisfy the allocation request
and subsequent fill.

The qqmlecmascript::sequenceConversionIndexes was changed to not attempt
INT_MAX allocations as, given enough memory and virtual address space, that
might succeed but take a really long time.

Change-Id: I4b0c965e9c23be78874343a70d7c155933c80903
Reviewed-by: Chris Adams <christopher.adams@nokia.com>
Reviewed-by: Kent Hansen <kent.hansen@nokia.com>

12 years agoFix typo.
Michael Brasser [Wed, 7 Mar 2012 22:25:45 +0000]
Fix typo.

Change-Id: Ibbfcc787e463a3a9d1a154d1986c3969f11e60d8
Reviewed-by: Chris Adams <christopher.adams@nokia.com>

12 years agoAdd autotest for v4 nested object bug.
Michael Brasser [Fri, 2 Mar 2012 01:33:46 +0000]
Add autotest for v4 nested object bug.

Task-number: QTBUG-24606
Change-Id: I0909d60316f7213b21fc315e8a3d0a6124e84409
Reviewed-by: Chris Adams <christopher.adams@nokia.com>

12 years agoReset transitioner pointer when transition job finishes
Bea Lam [Thu, 8 Mar 2012 07:05:07 +0000]
Reset transitioner pointer when transition job finishes

A transition job should only keep a reference to the transitioner
for the lifetime of the transition.

Change-Id: I22564d7b6587fed58ef53fd50ad4a700340dc7a4
Reviewed-by: Martin Jones
Reviewed-by: Martin Jones <martin.jones@nokia.com>

12 years agoUpdate righttoleft examples to new form factor
Alan Alpert [Wed, 7 Mar 2012 07:10:14 +0000]
Update righttoleft examples to new form factor

Change-Id: I2591f45a91da8bad90c5f3b9c963199c57bcf74d
Reviewed-by: Yann Bodson <yann.bodson@nokia.com>

12 years agoUpdate text examples to new guidelines
Alan Alpert [Fri, 2 Mar 2012 04:30:51 +0000]
Update text examples to new guidelines

Change-Id: I6bbf414f3734bc2f29a7182746c5df158a5924da
Reviewed-by: Yann Bodson <yann.bodson@nokia.com>

12 years agoUpdate shadereffects example to new guidelines
Alan Alpert [Fri, 2 Mar 2012 04:16:55 +0000]
Update shadereffects example to new guidelines

Change-Id: Ifa4f06cbf990a2f3fd1c46a3c4fda02b5c42d15b
Reviewed-by: Yann Bodson <yann.bodson@nokia.com>

12 years agoUpdate positioners examples to new guidelines
Alan Alpert [Fri, 2 Mar 2012 01:05:10 +0000]
Update positioners examples to new guidelines

Change-Id: I197a94bca26adbb79f1822b693fa73109db5a4ac
Reviewed-by: Yann Bodson <yann.bodson@nokia.com>

12 years agoFix crash in listmodel when data is assigned incorrectly.
Glenn Watson [Thu, 8 Mar 2012 23:19:35 +0000]
Fix crash in listmodel when data is assigned incorrectly.

If a listmodel with static role types is created, it would crash
if a role was assigned a value type such as string, and then
subsequently assigned an array (sub list) value from a dynamic
meta object (created when using get() from JS).

Change-Id: Ibfd0b0b40be13b04103b49462cfae42a5c9f9fb9
Reviewed-by: Martin Jones <martin.jones@nokia.com>

12 years agoHandle QEvent::TouchCancel in QML_TRANSLATE_TOUCH_TO_MOUSE mode
Martin Jones [Thu, 8 Mar 2012 03:41:19 +0000]
Handle QEvent::TouchCancel in QML_TRANSLATE_TOUCH_TO_MOUSE mode

Just remove the mouse grab.  The elements will react to this and
correct their state.

Change-Id: I8b91295606eaface54aa95b1500e7b138d9bc82e
Reviewed-by: Michael Brasser <michael.brasser@nokia.com>

12 years agoKeep high-level intermediate representation for IR::BINOPs.
Roberto Raggi [Thu, 1 Mar 2012 14:33:23 +0000]
Keep high-level intermediate representation for IR::BINOPs.

That is, keep binary expressions in HIR, this will make constant
folding more effective. Also, the register allocator for
expression-trees used in the V4 compiler will do a better job.

Change-Id: I2d5eea08ecd9c8d6f3aa21f1fd674a691ef9e1b0
Reviewed-by: Michael Brasser <michael.brasser@nokia.com>

12 years agoRemove non explicitly typed IR::CONST() nodes.
Roberto Raggi [Thu, 1 Mar 2012 14:18:54 +0000]
Remove non explicitly typed IR::CONST() nodes.

This change ensures that IR::CONST expressions
created by BasicBlock::BINOP(op,left,right) have
the correct types.

Change-Id: Iabac3f4ee1b897cc0d0bdf7e7385d7ae6dc513e4
Reviewed-by: Michael Brasser <michael.brasser@nokia.com>

12 years agoQmlDebugging: Remove QQmlDebugClient
Aurindam Jana [Tue, 6 Mar 2012 13:59:29 +0000]
QmlDebugging: Remove QQmlDebugClient

Remove QQmlDebugClient and relevant classes from the
library and move to client code.

Change-Id: I6f526b3f0c92970dcad5e5abd8585bb9b406349e
Reviewed-by: Kai Koehne <kai.koehne@nokia.com>

12 years agoQmlProfiler: Avoid overhead if profiling is not enabled
Kai Koehne [Thu, 8 Mar 2012 10:47:38 +0000]
QmlProfiler: Avoid overhead if profiling is not enabled

Change-Id: I7ecf32fca53fdb2672760a5b0c5ae88d6bd8fe5a
Reviewed-by: Aaron Kennedy <aaron.kennedy@nokia.com>

12 years agoRemove unused QTextCursor code from rewriter.
Matthew Vogt [Thu, 8 Mar 2012 04:11:46 +0000]
Remove unused QTextCursor code from rewriter.

The rewriter previously supported rewriting operations on either
a QString or a QTextCursor.  In order to remove the dependency on
QtGui, remove the unused QTextCursor support.

Task-number: QTBUG-24559
Change-Id: I7a4acceff8097a8bd8c022db23b6b89d356e305a
Reviewed-by: Roberto Raggi <roberto.raggi@nokia.com>

12 years agoPin qtbase at known-good change ID.
Matthew Vogt [Fri, 9 Mar 2012 00:44:21 +0000]
Pin qtbase at known-good change ID.

Change-Id: I889a427bf1bddbcc70255f4efb47b803f18dd89e
Reviewed-by: Rohan McGovern <rohan.mcgovern@nokia.com>

12 years agoFix selected text highlight in TextEdit.
Andrew den Exter [Wed, 7 Mar 2012 07:42:49 +0000]
Fix selected text highlight in TextEdit.

The call to QQuickTextNode::addTextDocument() omitted the anchor
color argument meaning every subsquent argument was off by one and
the selection end was always the default of -1.

Change-Id: Iac3165e7aa95a3db935a19e54144d3967a3e6e38
Reviewed-by: Yann Bodson <yann.bodson@nokia.com>

12 years agoImprove accessibility action support for Qt Quick
Morten Johan Sorvig [Tue, 10 Jan 2012 11:12:51 +0000]
Improve accessibility action support for Qt Quick

Add interface_cast for the action interface.
Implement actions for the following roles:
Button                           : Press
CheckBox, RadioButton            : Press, Check, Uncheck
Slider, Spinbox, Dial, ScrollBar : Increment, Decrement

Change-Id: Ic8e0d17c709ba51655f3f4b699092baf603b6f18
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@nokia.com>

12 years agoQML Debugger: Change prefix of warnings
Kai Koehne [Wed, 7 Mar 2012 13:53:24 +0000]
QML Debugger: Change prefix of warnings

Since we've been changing from QtDeclarativeDebugServer to
QQmlDebugServer anyway, we might as well change it to the more
readable "QML Debugger:" prefix.

Change-Id: I852577233d7cdb1f57adc43ec1b85a14d212574d
Reviewed-by: Aurindam Jana <aurindam.jana@nokia.com>

12 years agoQQuickView with the default constructor doesn't show any window decorations.
Alexis Menard [Mon, 5 Mar 2012 14:32:52 +0000]
QQuickView with the default constructor doesn't show any window decorations.

The default parameter takes a 0 which is propagated to the window platform
which sets it to 0. QWidget have a method called adjustFlags which takes
care of adding QWindow when necessary. There is no such check in QWindow
or the platform plugin for QPA. The default flag of QWindow is Qt::Window
but in that case it is overiden as QQuickView constructor calls
setWindowFlags(0). Qt::Window as a default parameter is a solution.

Change-Id: Iccbebc5b7e4f094c7fc6dc61b37d9f9622be0237
Reviewed-by: Samuel Rødal <samuel.rodal@nokia.com>
Reviewed-by: Gunnar Sletta <gunnar.sletta@nokia.com>

12 years agoInstall compat into the correct dir
Jørgen Lind [Wed, 7 Mar 2012 12:59:54 +0000]
Install compat into the correct dir

Change-Id: I699a943862d6388add2e4e7c24ed416ec1bd1444
Reviewed-by: Gunnar Sletta <gunnar.sletta@nokia.com>

12 years agoQmlDebugging: Remove deprecated classes
Aurindam Jana [Wed, 7 Mar 2012 08:00:22 +0000]
QmlDebugging: Remove deprecated classes

Remove the deprecated QQmlDebugHelper class.

Change-Id: I56823774f67ee06ccf7e0a797e250178f4a141f7
Reviewed-by: Kai Koehne <kai.koehne@nokia.com>

12 years agoQmlDebugging: Revert the names of services
Aurindam Jana [Wed, 7 Mar 2012 08:34:12 +0000]
QmlDebugging: Revert the names of services

Revert the names of the services changed in
b855240b782395f94315f43ea3e7e182299fac48 to
maintain BC.

Change-Id: I79826d92fd09c41e0020541b0c7bb77b5b2ecb1f
Reviewed-by: Kai Koehne <kai.koehne@nokia.com>

12 years agoCompile after API changes in QCoreApplication
Gunnar Sletta [Wed, 7 Mar 2012 09:21:08 +0000]
Compile after API changes in QCoreApplication

Change-Id: I6bc6978a5c68503facd8085bd35e66f86af58a90
Reviewed-by: Lars Knoll <lars.knoll@nokia.com>

12 years agoUse StringRef::load() to initialize DefaultProperty string
Kent Hansen [Mon, 5 Mar 2012 11:58:50 +0000]
Use StringRef::load() to initialize DefaultProperty string

Micro-cleanup; don't access the string data directly. load() is
inline. It additionally asserts that the string length matches.

Change-Id: Ie24df83ea68b96179792995d062db7872933b1a8
Reviewed-by: Aaron Kennedy <aaron.kennedy@nokia.com>

12 years agoDon't needlessly create alias property names in the meta-object
Kent Hansen [Mon, 5 Mar 2012 11:23:19 +0000]
Don't needlessly create alias property names in the meta-object

When aliases should not be resolved, there is no need to create the
alias name StringRef, since it would occupy space in the meta-data
even though the string was never actually initialized.

Similarly, when aliases should be resolved, it's enough to create the
StringRef once.

Change-Id: I44dfe665fe8d7bd5754bc939ff62ad75efe19d5b
Reviewed-by: Aaron Kennedy <aaron.kennedy@nokia.com>

12 years agoMake sure QML type "real" always maps to C++ type "double"
Kent Hansen [Mon, 5 Mar 2012 10:54:01 +0000]
Make sure QML type "real" always maps to C++ type "double"

The type "real" was documented to be a single-precision float, but
that's incorrect. It's always been double.

However, signal parameters of type "real" would be mapped to the C++
type "qreal", which can be either float or double depending on the
platform.

Since JavaScript floating point numbers have double precision, QML
should use the same, to avoid potential loss of precision.

With this change, "real" behaves the same as the QML "double" type
(which already guaranteed double precision). Even though it's
redundant, "double" is kept to preserve compatibility.

Added tests for the "double" type to the QML meta-object autotest,
and a test for the "real" type that ensures there's no loss of
precision.

Change-Id: I1a77f1fd45082ff670684a935e17d1a46ea75d84
Reviewed-by: Aaron Kennedy <aaron.kennedy@nokia.com>

12 years agoCorrectly place '.' when building name for call expressions.
Michael Brasser [Wed, 7 Mar 2012 03:50:41 +0000]
Correctly place '.' when building name for call expressions.

Change-Id: Ic0119a0f4c02387ed19e2905f0b24c4ce86565cb
Reviewed-by: Roberto Raggi <roberto.raggi@nokia.com>

12 years agoDon't link against QtWidgets and QtOpenGL
Lars Knoll [Wed, 29 Feb 2012 19:41:47 +0000]
Don't link against QtWidgets and QtOpenGL

Change-Id: I3a949dff488d200762b18a0c0a16103f8908052e
Reviewed-by: Kent Hansen <kent.hansen@nokia.com>

12 years agoListView can freeze if flicked beyond its bounds.
Martin Jones [Wed, 7 Mar 2012 05:26:33 +0000]
ListView can freeze if flicked beyond its bounds.

If the delegate's size changes in componentComplete and all
items are flicked out of view, an incorrect jump calculation
in addVisibleItems() resulted in a new delegate being created
in the wrong position, and retriggering the jump calculation,
which resulted in a new delegate being created in the wrong
position, and retriggering the jump...

Also fixed currentItem visibility.

Change-Id: Iad5f211c4fc5eed9c009d51a0ce3b58181a7b36e
Reviewed-by: Bea Lam <bea.lam@nokia.com>

12 years agoFix setting of target lists when target Transition is not set
Bea Lam [Mon, 5 Mar 2012 23:39:24 +0000]
Fix setting of target lists when target Transition is not set

Target items are now set from QQuickViewItem::prepareTransition()
instead of QQuickItemView and QQuickPositioner to ensure they are
for a displaced transition even if there is no matching target
transition.

Task-number: QTBUG-24535
Change-Id: I0a6c7e3c6198786527014d421b96fc562c6186dc
Reviewed-by: Martin Jones <martin.jones@nokia.com>

12 years agoChanged particles tests to work from install directory
Kurt Korbatits [Tue, 6 Mar 2012 22:02:44 +0000]
Changed particles tests to work from install directory

- Changed particles tests to use TESTDATA
- Changed particles tests to inherit from QQmlDataTest
  and use testFileUrl() to find test data

Change-Id: Iae7c5111d976ec3efe473c40a3bab2f0a38c0969
Reviewed-by: Michael Brasser <michael.brasser@nokia.com>
Reviewed-by: Jason McDonald <jason.mcdonald@nokia.com>

12 years agoChanged quick tests to work from install directory
Kurt Korbatits [Tue, 6 Mar 2012 04:48:21 +0000]
Changed quick tests to work from install directory

- Changed tests to use TESTDATA
- added check for cross_compile option to skip when sources not available

Change-Id: I1f382794ff982bbc07fc20438a4e4a8c8b8d565f
Reviewed-by: Michael Brasser <michael.brasser@nokia.com>
Reviewed-by: Jason McDonald <jason.mcdonald@nokia.com>

12 years agoChanged qml tests to work from install directory
Kurt Korbatits [Tue, 6 Mar 2012 04:01:34 +0000]
Changed qml tests to work from install directory

- Changed tests to use TESTDATA
- moved qqmlcontext to private test as it contains private header
- added check for cross_compile option to skip when sources not available

Change-Id: I0f68f58ffcb1b41b8e40a9851e3e003fe72ee2f9
Reviewed-by: Jason McDonald <jason.mcdonald@nokia.com>
Reviewed-by: Michael Brasser <michael.brasser@nokia.com>

12 years agoChange the default renderTarget and renderStrategy
Charles Yin [Tue, 6 Mar 2012 23:28:27 +0000]
Change the default renderTarget and renderStrategy

As we always use OpenGL and scenegraph render thread now, the best default
render target should be FBO and the best render strategy should be
cooperative mode (use the render thread).

Task-number:QTBUG-23956
Change-Id: I64b070dec566e359d27c84680fef1c691d6ce411
Reviewed-by: Michael Brasser <michael.brasser@nokia.com>

12 years agoRepaint the canvas item properly when canvas resized
Charles Yin [Tue, 6 Mar 2012 23:34:18 +0000]
Repaint the canvas item properly when canvas resized

Task-number:QTBUG-23594
Change-Id: I703a5190b5b306666ffd61001ff94f296399465f
Reviewed-by: Michael Brasser <michael.brasser@nokia.com>

12 years agoprepareTransition() must cancel current transition
Bea Lam [Mon, 5 Mar 2012 23:28:56 +0000]
prepareTransition() must cancel current transition

If it does not, itemX() or itemY() will still return the current
transition's 'to' position instead of the nextTransitionTo pos that was
scheduled for the next transition (which the item now has moved to
using the direct setPos() method).

Also refactor prepareTransition() to always move the item directly to
the nextTransitionTo if transition is not going ahead.

Also fix some broken test code.

Task-number: QTBUG-24523
Change-Id: I2e536fbc0da2acbf96fdf2d177190a8968f7fdb1
Reviewed-by: Martin Jones <martin.jones@nokia.com>

12 years agoFix texture leaks in QQuickCanvasItem
Charles Yin [Wed, 7 Mar 2012 00:20:06 +0000]
Fix texture leaks in QQuickCanvasItem

In commit be4c74948371ddf6f1ed260783b43b5a3d9e60a7 we removed the
QQuickCanvasItemNode, but textures are not automatically deleted with
QSGSimpleTextureNode, so add a very thin sub class of QSGSimpleTextureNode
to just cleanup canvas texture to avoid texture leaks.

Change-Id: I31aac1068e8218a75129b52afbbf30fb66e4bbf7
Reviewed-by: Michael Brasser <michael.brasser@nokia.com>

12 years agoCheck for parent item before calling functions on it.
Frederik Gladhorn [Tue, 6 Mar 2012 10:26:55 +0000]
Check for parent item before calling functions on it.

Change-Id: Ic4066d133ef16ce8cf11fcb9fb5ddcc19599e85d
Reviewed-by: Jan-Arve Sæther <jan-arve.saether@nokia.com>

12 years agoFix missing glyphs when using shared distance-field cache
Eskil Abrahamsen Blomfeldt [Mon, 5 Mar 2012 16:25:50 +0000]
Fix missing glyphs when using shared distance-field cache

When using a shared distance field cache, signals such as
itemsMissing() and itemsUpdated() can be triggered in the
server process by external requests. This, in turn, can
lead to unnecessary storeGlyphs() calls (performance hit)
and, even worse, calls to setGlyphPositions() for glyphs
which have not previously been requested through populate().
The latter could cause missing glyphs when the same
characters were requested later, as they would then be
stored in the cache with the bounding rect of a default
constructed QPainterPath (in setGlyphPositions()).

To fix this, we ignore all glyphs which have no been requested
in this process, thus filtering out all events for glyphs which
have only been used externally so far. I've also added an
assert to the setGlyphPositions() to help us catch this case
early if it should return.

Change-Id: Ief333f612e4affea768d9c60409d43ce29fe3f60
Reviewed-by: Jiang Jiang <jiang.jiang@nokia.com>

12 years agoCleanup QQuickCanvas and QQuickView docs.
Martin Jones [Mon, 5 Mar 2012 23:45:58 +0000]
Cleanup QQuickCanvas and QQuickView docs.

Mark reimplemented protected methods.

Change-Id: I544d68e16d82a37f5612fd72cd185a23c70157a7
Reviewed-by: Alan Alpert <alan.alpert@nokia.com>

12 years agoContinue refactor of Canvas rendering.
Justin McPherson [Fri, 2 Mar 2012 06:28:16 +0000]
Continue refactor of Canvas rendering.

- Remove sync() - not necessary
- Queue buffer before flush

Change-Id: If7dea8e56a612d241c67a82eae265febc45059a8
Reviewed-by: Yunqiao Yin <charles.yin@nokia.com>

12 years agoUse NPOT support if available.
Justin McPherson [Tue, 28 Feb 2012 05:08:43 +0000]
Use NPOT support if available.

In QQuickContext2DTexture.

Change-Id: If0311450658a837b7e2665c041ec2846e3c8c2dd
Reviewed-by: Yunqiao Yin <charles.yin@nokia.com>

12 years agoRemove QQuickCanvasItemNode.
Justin McPherson [Fri, 2 Mar 2012 06:24:04 +0000]
Remove QQuickCanvasItemNode.

A custom SGNode is not necessary.

Change-Id: Ibde71dd502bc782503dbd7d98e6ad8b811b7aedb
Reviewed-by: Yunqiao Yin <charles.yin@nokia.com>

12 years agoRemove incorrect line from performance docs
Chris Adams [Tue, 28 Feb 2012 07:38:35 +0000]
Remove incorrect line from performance docs

V8 does not use a single map-transition-graph for all types, and thus
the line about ordering of properties in constructor functions is
incorrect.

Change-Id: Iaf4ac02dcfc53dd7254af1a40a3c66852a78a320
Reviewed-by: Alan Alpert <alan.alpert@nokia.com>

12 years agoFixed QQuickCanvas header with QT_NO_KEYWORDS
Rohan McGovern [Mon, 5 Mar 2012 06:12:48 +0000]
Fixed QQuickCanvas header with QT_NO_KEYWORDS

slots -> Q_SLOTS

Change-Id: I971a0077a2606e197e9542ccb864e57d0b0e9f4f
Reviewed-by: Jason McDonald <jason.mcdonald@nokia.com>

12 years agoFix renaming merge with examples
Alan Alpert [Mon, 5 Mar 2012 05:40:59 +0000]
Fix renaming merge with examples

Some examples were duplicated a few times. Removing the excess.

Change-Id: I6fa59f79de63ac952103421ba1c063f5912b702b
Reviewed-by: Martin Jones <martin.jones@nokia.com>

12 years agoHandle MouseArea.enabled = false after mouse is pressed.
Martin Jones [Mon, 5 Mar 2012 05:07:27 +0000]
Handle MouseArea.enabled = false after mouse is pressed.

Currently this leaves the MouseArea in a broken state,
i.e. still in pressed state, and the next press after
it is re-enabled is ignored.

In this case we now allow subsequent mouse move or
release events to continue. Following the release,
no further press will be accepted.

Change-Id: I65a890da90e2166ad568505fffdbd3db6c97165b
Reviewed-by: Andrew den Exter <andrew.den-exter@nokia.com>

12 years agoDo not store an explicit QImage in the declarative pixmap cache.
Gunnar Sletta [Mon, 5 Mar 2012 09:11:16 +0000]
Do not store an explicit QImage in the declarative pixmap cache.

This opens up for the scene graph adaptation only storing textures which
effectively halves the memory usage for textures and images.

Change-Id: I3452a15844ceeb4fc3a465865f99ee16e3e05607
Reviewed-by: Aaron Kennedy <aaron.kennedy@nokia.com>

12 years agoMade QQuickWindowManager pluggable via QSGContext.
Gunnar Sletta [Mon, 5 Mar 2012 08:28:10 +0000]
Made QQuickWindowManager pluggable via QSGContext.

Also rename paint() to exposureChanged() as that is what the function
actually means. The implementation of paint() has been removed in both
trivial and threaded version as it is superflous as show() already
triggers rendering.

Change-Id: I7e53f42786efaf961921f10a39977de511965d71
Reviewed-by: Jørgen Lind <jorgen.lind@nokia.com>

12 years agoFix crash with AnimatedImage caused by race condition.
Gunnar Sletta [Mon, 5 Mar 2012 08:44:46 +0000]
Fix crash with AnimatedImage caused by race condition.

In QDeclarativePixmap::setImage() we deleted a QDeclPixmapData
and recreated a new one in a very short timespan and the new
texture factory was the same pointer as the deleted one, yet a
queued destroyed signal was still emitted. Depending on when the
queued connection was handled in the rendering thread, this would
cause problems with the value returned from textureForFactory.

Change-Id: Ibd785ca12667c99efb88b92689ae7ac4fa87c7ee
Reviewed-by: Kim M. Kalland <kim.kalland@nokia.com>

12 years agoUse premultiplied format because it's faster
Alan Alpert [Mon, 5 Mar 2012 03:31:56 +0000]
Use premultiplied format because it's faster

Change-Id: I4d78830d2cf03f2deba70c29c2fc221eeec9d63c
Reviewed-by: Gunnar Sletta <gunnar.sletta@nokia.com>

12 years agoChange bugreports.qt.nokia.com -> bugreports.qt-project.org
Sergio Ahumada [Fri, 2 Mar 2012 12:34:21 +0000]
Change bugreports.qt.nokia.com -> bugreports.qt-project.org

Change-Id: Icafc7b49e7c42368fe10939b8a134632276c0671
Reviewed-by: Damian Jansen <damian.jansen@nokia.com>

12 years agoMerge remote branch 'gerrit/api_changes' into merge-master
Matthew Vogt [Mon, 5 Mar 2012 03:20:58 +0000]
Merge remote branch 'gerrit/api_changes' into merge-master

Change-Id: I9c54ff2a8aadb2bf95667eb3cf0a48f8291ce927

12 years agoMerge master <-> api_changes
Matthew Vogt [Mon, 5 Mar 2012 01:39:24 +0000]
Merge master <-> api_changes

Change-Id: Iad2f07b989b25349fd2d4fff010e24dcd5a1688f

12 years agoMore documentation about importing LocalStorage module from Javascript
Charles Yin [Wed, 29 Feb 2012 00:10:11 +0000]
More documentation about importing LocalStorage module from Javascript

Task-number:QTBUG-24478
Change-Id: Iad0fe15e8b2f1fcfa82b1654c3fd606d08563457
Reviewed-by: Martin Jones <martin.jones@nokia.com>

12 years agoProfiler: Use RAII helper structs for ranges
Kai Koehne [Wed, 29 Feb 2012 13:59:49 +0000]
Profiler: Use RAII helper structs for ranges

Exclusively use RAII helper structs for ranges.

Change-Id: Ief9ab25a9e49e1b2c3c091e5d9de6479e36eaa50
Reviewed-by: Christiaan Janssen <christiaan.janssen@nokia.com>