GitLab

Also, make sure the category dragger only reacts to vertical mouse
movements.

Change-Id: Ic53db9030536a1f2d35ea02facb920a4d3f0a005
Task-number: QTCREATORBUG-15333
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

Change-Id: I40e4a777c6f485443ab142cc220ae22dab44b752
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

This avoids the "QTransform::translate with NaN called" warnings. They
were triggered by the text item having a negative width due to its
parent having a width of 0 and the item itself subtracting a margin
from that.

Change-Id: I3ee89b11ba2a27e568982a7ff9a9fe297907a37e
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

If zoomer.rangeDuration <= 0 we should set contentWidth and contentX to
0. Otherwise we get a long empty, but scrollable trace.

Change-Id: I1fcb5d75d220fdec609d1a7a2908f68d084d0f49
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

selectedItemChanged is entirely enough to signal that the selection has
changed.

Change-Id: I1b5640c4ad4891ab9fadac83a53b202fdd6871a5
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

Every time a batch root is added or removed the whole scene is rebuilt.
Instead of using the "visible" property we can collapse items to a
height of 0 in order to make them invisible while keeping them in the
scene. This avoids the rebuilds.

Change-Id: Icc6dddb8c8f892a37ef4ed5bced501e5f1658d80
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

Changing an item from alpha blended to solid or vice versa forces a
full rebuild. The difference between solid and 0xfe alpha won't be
visible.

Change-Id: I4da47870fc465449f0354a18fc87717c569be812
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

Using an array of strings as model has the unfortunate effect that all
text nodes resulting from it are rebuilt whenever anything in the
array changes. Using a number as model only triggers a rebuild of
everything when the number changes.

Change-Id: I8a762e78eb75b282f46641a6cba6eff43c8ebfd5
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

The selection details "windows" contain frequently updated text and can
be moved around in the scene. We don't want their contents to be batched
with the rest of the scene as there may be many events in the timeline
and rebuilding the batch they are in is expensive. The clip property has
the side effect of preventing nodes within the clipped area from getting
batched with nodes outside it. Thus, by setting the clip property on
the details we reduce the number of batch rebuilds for the actual
timeline.

Having a transparent rectangle in there doesn't help the renderer,
either, so we replace it with a plain item.

Change-Id: Ie6bac1324d3bc6b8b3d56e658a766f1f90be9046
Task-number: QTCREATORBUG-14983
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

The only thing that needs clipping are the thin vertical lines. We can
manually manage those with the "visible" property. Clipping with the
"clip" property on the parent element can cause costly rebatching for
the whole content while scrolling.

Change-Id: Ia31eeefc4c0a943b00ea47b12c5c9d49b0f004e3
Task-number: QTCREATORBUG-14983
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

This is better done in a central place, where it applies to all views
connected to the relevant models.

Change-Id: I33ce2c48de6ca3cf37b10a9a9cb7ecac15c3d5bd
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

Apparently the default can be something random.

Task-number: QTCREATORBUG-14672
Change-Id: I76fa89074b4eb118810cc584919fec1c917073e3
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

This quietens warnings from libPNG during startup and shrinks the
image sizes.

Change-Id: Ieb4cb5e8ba30b99653896e283c2fb2cc267257f2
Reviewed-by: Eike Ziller <eike.ziller@theqtcompany.com>

If the trace being shown is shorter than 1ns we better don't show any
labels as those are necessarily useless. Trying to "repair" the
duration to show something in that case creates more problems than
it solves, as the re-evaluation of the bindings when the actual values
arrive produces strange transitional states, which surface as large
negative numbers for the time labels.

Change-Id: I16598a2b1c3271dc50bbd3877baf4c0e15cc6411
Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>

The only thing they did was trigger some QML code that set the zoom to
10% of the available range. 10% is somewhat arbitrary, so we shouldn't
have it in the general purpose timeline. Also, we don't really have to
pass the signal through several connections just to do that in QML.

The clearChildren() didn't have any effect there as dataAvailable()
should only happen if the view is already cleared.

Change-Id: I1813ed37a82fc3fd5c3b7fc02b2afb1dcbcadcfb
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

Events with duration == 0 were erroneously filtered out by the items
render pass. Also, we have to give them a very small width in order for
the "stretching" mechanism in the vertex shader to work.

Change-Id: Icb76168f0831547a3ca55ab79df7161736ed4dc4
Task-number: QTCREATORBUG-14446
Reviewed-by: Eike Ziller <eike.ziller@theqtcompany.com>
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

The text next to the checkboxes needs to be elided to fit into the
menu. The filter gets reset every time we clear and reload. The
checkboxes reflect this now.

Change-Id: If34188a4abb0c087a8f585653abcf0a787fc3232
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

The height of the trace is not sufficient to determine this as by
hiding categories it can be reduced to 0 without the trace being empty.

Change-Id: Ic8100b4da9c93808a6d49cf4ceba12de391ebc0e
Task-number: QTCREATORBUG-14277
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

This way they get elided correctly.

Change-Id: I924679bd0f985f621fc62b152f3a4536e48e4492
Reviewed-by: Kai Koehne <kai.koehne@theqtcompany.com>
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>
Reviewed-by: Eike Ziller <eike.ziller@theqtcompany.com>

We don't want to resize them to absurd dimensions.

Change-Id: I7f824919052c6740f291d2b0266fd009da99fd53
Reviewed-by: Kai Koehne <kai.koehne@theqtcompany.com>
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>
Reviewed-by: Eike Ziller <eike.ziller@theqtcompany.com>

We don't want the labels to be elided and we don't want to align all
windows by the largest possible label.

Change-Id: I062a1eab5b15298a6edf40435925f54723683b93
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

As the selection range is defined by pixel offset from the parent item,
we have to move it when the time the parent item refers to changes.

Change-Id: I6ad0aaab8262ea6a80fbe2e3336e79417d55edba
Task-number: QTCREATORBUG-14170
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

This is where it logically belongs, given that trace, window, and range
are already there. Also, it fixes the types to qint64, making it easier
to reason about type conversions and numerical overflows.

Change-Id: I2f88b2646b9a649d34bdf4fe87c37e7afdeee078
Task-number: QTCREATORBUG-14170
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

In OpenGL ES you have to:
* Specify the precision for most values in fragment shaders
* Add a special declaration to use fwidth()
* Make sure both values passed to max() are of the same type
  (this may be the case for some Desktop OpenGL implementations, too)

Repeating the default precision (highp float) in vertex shaders is useless
and might lead to type casting problems, so we drop it.

Without this change, the timeline doesn't work on Windows with ANGLE.

Change-Id: I69a1976eddb6f4dc6dfe5fe9f270839432088cbb
Reviewed-by: Maurice Kalinowski <maurice.kalinowski@theqtcompany.com>
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>
Reviewed-by: Kai Koehne <kai.koehne@theqtcompany.com>

The details table can become quite large, so we elide the text.
However, as we don't know what kind of information we're hiding and
the user might want to see it it's quite practical to allow resizing.

Also, make sure that the detail window (and the range window) is on
top of everything else so that it can set the cursor shape.

Task-number: QTCREATORBUG-14054
Change-Id: I12850124c327c57c148b69960687b7402bdead90
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

Change-Id: I69a3ea81b1c4e1df51663d8e5c75972f67635a26
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

Trace data is saved in the background, progress is emitted, and
the save operation can be canceled. While data is being saved
the views are disabled and a semitransparent layer is put on top
of the trace view.

Task-number: QTCREATORBUG-11822
Change-Id: I94ec93147fb1788fc85939ddc591961d058050b5
Reviewed-by: Ulf Hermann <ulf.hermann@theqtcompany.com>

Also, remove the obsolete function to enable the buttonsBar.

Change-Id: I92cefe841541d997cddcbc7f20ae6669ed2e3743
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

Added a text field to search for notes within the timeline's events.

Task-number: QTCREATORBUG-13417
Change-Id: Ic121ec8ade42b1ef99d5da13d1f732761d244327
Reviewed-by: Ulf Hermann <ulf.hermann@theqtcompany.com>

This further cuts down the initial loading time by a large margin and
can save some memory.

Change-Id: I8c3240d3137461229fa5576ad33c0d77b28776e0
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

Loading them all at once and keeping them around all the time wastes
memory and time.

Change-Id: I088e0f955a93f94fa460ca17583123eb4d317da9
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

Change-Id: I1f03cba1cc7a8be5d744b4d8f918ce6cd41f2612
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

Like this the labels are also updated initially and when the model
changes. Also, use the more generic label "[unknown]" instead of
"<bytecode>" to replace empty descriptions. Even in the QML case it's
not always bytecode.

Change-Id: Iea0dc36eb9061c928f2cb9706adaaa3b4bab7811
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

Apparently the parent is not always defined and we get warnings about
that.

Change-Id: I574d68c2668fef53af9422049143a0decf05286f
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

Change-Id: I45778c9562ba530a36ddaf201f0c61d380d701cf
Reviewed-by: Leena Miettinen <riitta-leena.miettinen@theqtcompany.com>

If you pass 0 as notes model to the aggregator then all the notes
features are hidden.

Change-Id: I432f2820812c472221a0330de5b8f5d8d0fcad13
Reviewed-by: Kai Koehne <kai.koehne@theqtcompany.com>

Change-Id: I964b2f149e237eb25a08600e8dab8968e8bc0cb9
Reviewed-by: Kai Koehne <kai.koehne@theqtcompany.com>