Updated POTFILES.skip

icon-theme: Add some preconditions for NULL arguments

Related to this crash: https://bugzilla.gnome.org/show_bug.cgi?id=687967

https://bugzilla.gnome.org/show_bug.cgi?id=687977

Make _gtk_css_lookup_get_missing inline

This is called a lot in the loop in gtk_css_style_provider_lookup which
actually showed up on profiles.

gtk_css_style_provider_lookup: minor loop optimiziation

This is a minor performance improvement but this loop
is run a lot so it actually matters in large state
changes.

Exit early in gtk_style_context_update_cache codepaths

If lookup->missing is empty we don't need to continue looking.
We short circuit in several places as this can happen
after iteratively makeign lookup->missing smaller.

Fix cnp bug in _gtk_bitmask_intersects

We need to use the allocated codepath if *any* argument is
allocated, not if one arg is not allocated.
This bug caused unnecessary calls to _gtk_bitmask_is_allocated,
as well as return completely wrong result if both bitmask are
allocated.

gdk: Add gdk_window_has_alpha helper

This centralizes the current checks for has_alpha_bg, which
lets us extend the check later.

https://bugzilla.gnome.org/show_bug.cgi?id=687842

Add "parent widget" button to test property editor

Without this its hard to access the properties of container
widgets.

https://bugzilla.gnome.org/show_bug.cgi?id=687842

gtkbin: Remove the silliest code on earth

What is this bin doing with all these crazy deltas? Company does:

<Company> that can safely be removed
<Company> in general, code that isn't obvious can either be understood
<Company> with a bit of thinking or it can be removed
<Company> if in doubt, go for the 2nd of those :)

Most GtkBin subclasses override this strange garbage anyway, so it's
not like this code is ever *run*, per se. Just make it proxy directly
to the child, and hope nothing goes wrong.

gtkbin: Complete more of a base implementation for GtkBin subclasses

Implement get_preferred_width, get_preferred_height, and size_allocate.
This allows GtkBin subclasses to be quick and easy, without the
author doing the subclassing to have to do much work.

gtkversion.h.in: Put the documentation inside the header guards

This fixes a warning with gobject-introspecion scanning

reftests: Add test for recent fix

label: Fix another sizing corner case causing segfaults

If the "wider" label is the smaller one, use the wider size for both
cases. This can happen when ellipsizing a single character, which is
often smaller than the ellipsizing glpyph(s).

stylecontext: Fix a memleak

stylecontext: Remove unused function

cssvalue: Convert shadows to GtkCssColorValue

cssimage: Use GtkCssColorValue

cssstyleproperty: Replace symbolic colors with GtkCssColorValue

cssvalue: Add GtkCssColorValue

This is mostly copy/paste from GtkSymbolicColor and is indeed intended
to replace it.

symboliccolor: Change the print output

Functions should not have a space before the opening parenthesis. So
change output like
  alpha (@color, 0.5)
to
  alpha(@color, 0.5)
and do the same for "shade" and "mix".

Tests have been updated accordingly.

symboliccolor: Split out HSLA code

Update a11y test results

These are expected changes, since a11y class name show up in the
dumps.

Nuke GtkBoxAccessible

It didn't contain anything useful.

text-cell-accessible: fix a double unref

We're unreffing the GtkCellRenderer object twice, causing the
application to crash when trying to access the renderer later.

https://bugzilla.gnome.org/show_bug.cgi?id=687872

Updated Traditional Chinese translation(Hong Kong and Taiwan)

gradient: Add color stops to the right gradient

This was causing an infinite loop happily eating up memory.

https://bugzilla.gnome.org/show_bug.cgi?id=687467

tests: Include config.h

Disable deprecation checks for all libraries we depend on on stable branches.

This is so newer versions of those libraries don't cause more warnings
with a stable GTK version.
We don't ever want to turn off deprecation warnings for master however,
because that's where we get rid of deprecated API we use.

Note that only glib allows use to easily do this, so nothing is done for
Pango, gdk-pixbuf or Cairo here.

cssimage: Fix huge memleak

When transitioning gradients, we were leaking the gradient structs. Not
good.

https://bugzilla.gnome.org/show_bug.cgi?id=687467

css: Fix copy/paste error

Benjamin replaced start with end in a bunch of locations, but not all of
them apparently.

Updated POTFILES.skip

reftests: Add a reftest for latest fix

label: Handle crazy corner cases

With ellipsizing, the ellipsized text can have a smaller height than the
non-ellipsized text. So the wider text is also higher. Example:
  .<big>TEXT</big>
will ellipsize to the small text.

Reported-By: Rico Tzschichholz <ricotz@t-online.de>

Fixed Brazilian Portuguese Translation

Bump GLib dependency to 2.35

And drop deprecated g_type_init() calls.

reftests: Add test for recent fix

This test checks the fix in a46368dede684ded093395ac056eaeecf554330d

reftests: Mark windows as popup

This makes the tests run faster and should always be done.

sizerequest: do not derefence NULL pointers

gtkmenubutton: remove weak pointer when needed

We must make sure to remove the weak pointer when disposing the widget
or when resetting the align widget otherwise glib will try to nullify
invalid memory.

gtkmenubutton: set menu to NULL in dispose.

Prevent menu to be detached multiple times.

sizerequest: Restructure code

Make the compute_size_request() function take into account size groups
itself instead of doing a weird "bump_requisition" call.

sizerequest: Move sizegroups function to different source file

sizegroup: Construct the set of sizegroup peers as a hash table

This way we don't need a marker on GtkWidgetParivate that needs to be
unset later, so we have all our data in the same place and can avoid
problems with reentrancy and shenanigans like that.

But the main reason I wrote that is cleaner code.

reftests: Add a test from evolution

This was used in a bug report.

https://bugzilla.gnome.org/show_bug.cgi?id=677609

reftests: Add a test for all the recent sizegroup hackery

sizegroup: Use _gtk_widget_compute_size_for_orientation()

With this function now available, we can do size computation in 2
ways:
(1) Compute size with size groups
(2) Compute size without size groups

And have (1) use (2) instead of setting flags on widgets. This patch
does exactly that.

sizerequest: Move optimization

With size groups now doing hfw, doing the optimization for CONSTANT_SIZE
was done too early. Size groups need to know that it's a hfw request, so
the other widgets in the size group get the correct behavior.

label: Redo get_preferred_width/height()

This is important for size groups mostly, but also has some small fixes.
The label-sizing reftest as been updated accordingly.

label: Fix ellipsize and wrap being set

The label code assumed that Pango treats this as "wrap to as much space
as possible and then ellipsize all the lines", but for Pango, ellipsize
takes precedence over wrap. So do the same thing in GtkLabel.

Also updated is the reftest that checked this behavior.

reftests: Improve reftest performance

Get rid of all the event boxes in this test. Event boxes need GDK
windows which cost a lot of performance when running the test and they
clip the label output.

Getting rid of the clipping also shows 2 bugs in this test that weren't
visible before. Those will be fixed in a followup patch.

reftests: Make label-sizing tests use better CSS

ow that labels can have backgrounds, just use label backgrounds

sizerequest: Export _gtk_widget_compute_size_for_orientation()

and add an "ignore_size_groups" flag to it. This way we can use it for
size group shenanigans.

sizegroup: Move GtkSizeGroupMode to gtkenums.h

This is in preparation for the next patch, which would otherwise lead to
conflicts.

sizerequest: Cache sizes without size groups

We compute on-demand for size groups anyway, so we can (in theory, this
patch doesn't do that yet) get around costly cache blowing when
invalidating single widgets of a size group this way.

sizegroups: Use is_visible() instead of get_mapped() for visibility

The current approach of using gtk_widget_get_mapped() is broken:
The usual steps taken when showing a window are:
(1) request the sizes
(2) allocate the sizes
(3) show the window in the allocated size

Showing the window with a random size between steps (1) and (2) would of
course
result in extra work and potential flickering when the widgets get
resized to
their proper sizes.

However, as GtkSizeGroup::ignore-hidden uses gtk_widget_get_mapped() to
determine visibility for a widget, the following will happen:
(1) the widget will request a 0 size
(2) the widget will be allocated a 0 size
(3) the widget will be too small when it is shown

gtk_widget_get_visible() however is set in advance. Note that toggling
visibility also causes a gtk-widget_queue_resize() call already so we
take care of changes in here automatically.

API: Add gtk_widget_is_visible()

This is a recursive gtk_widget_get_visible(): Returns TRUE if the widget
and all its parents are visible.

sizegroup: Handle hfw in size groups

sizegroup: Add a function for clarity

... and restructure code to accomodate that function.

sizegroup: Don't keep groups around everywhere

The code is only interested in the actual widgets that belong together,
not in the groups. So just don't return the groups.

sizegroup: Check ignore_hidden flag when adding groups

Instead of only checking the ignore_hidden flag when getting the
preferred sizes, respect it already when constructing the list of
widgets. This way, widgets don't queue resizes for groups they're
ignored in anyway.

sizegroup: Use for loops

For loops to loop over lists look nicer and actually do the right thing
with "break" and "continue" statements. So they are vastly preferred to
while loops.

sizegroups: Restructure code

This way, we do the checks at the start of the effected function, not
before calling it.

sizegroup: Don't cache the sizes anymore

This simplifies code and because sizes are cached by the widgets
themselves, it's not a large performance problem (unless people use huge
amounts of widgets in a single size group, but who does that?

Merge branch 'bgo687196-filesystemmodel-crash'

bgo#687196 - Fix model corruption during file removal

The main problem is that we were emitting the row-deleted signal for the model in the middle
of the process that actually deletes the row from the model (remove the row from the array,
update the model->file_lookup hash table, etc.).  In the model's caller, one of the row-deleted
callbacks was requesting an iter, which caused the model to revalidate itself - but it did
this while it was in an inconsistent state.  This led to an assertion failure later when the
model resorted itself.

The fix in remove_file() is like this:

* The filteredness/visibility of the deleted node is not updated.  The
  node will simply be gone; we don't need to update those values at
  all.

* We invalidate just the node that is being deleted.

* The model->file_lookup hash table is not completely nuked; instead,
  we carefully adjust its indices.

* The row-deleted signal is only emitted at the very end, when
  deletion is complete and the model is consistent.

Many thanks to William Hua for doing the detective work on this bug!

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Make freeze_updates() and thaw_updates() static functions

They were in the semi-public API of GtkFileSystemModel, but never actually used outside of it.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Comments on how the filtering and sorting processes work

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

sizegroup: Always at least use widget's size

When widgets were hidden, they were otherwise assigned a 0 size.

settings: Reset all styles when the enable-animations settings changes

https://bugzilla.gnome.org/show_bug.cgi?id=686021

Raleigh: Fix spinners with disabled animations

When animations are disabled, active and inactive spinners should look
different.

https://bugzilla.gnome.org/show_bug.cgi?id=686021

cssimage: Implement some equal functions

Remove argument to _gtk_file_system_model_update_file() that should not be part of the internal API

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Rename gtk_tree_path_new_from_node() to tree_path_new_from_node()

This is a function internal to the file system model; let's not pollute the gtk_tree_path namespace.

Also, make the 'i' variable into 'r' as it refers to a row index, not a file-array index (for
consistency with the docs and the rest of the code).

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

icon-theme: support loading symbolic GFileIcons from generic URIs

Right now we support loading and recoloring symbolic GFileIcons, but
only if the underlying GFile has a local path. This breaks when the
GFileIcon is loaded from a GResource, which is a reasonable option for an
application that wants to ship a custom symbolic icon.

This patch changes GtkIconInfo to store a GFile together with the file
path, and changes the symbolic icon lookup code to use the GFile URI,
which transparently makes the code work also for GResources.

https://bugzilla.gnome.org/show_bug.cgi?id=687059

themingbackground: Remove struct members

... and put them in the only function they are used in.

reftests: Add reftest for fractional border sizes

themingengine: Draw fradctional border sizes

reftests: Fix linear-gradient reftest

... and add it to the Makefile

themingbackground: Remove GtkThemingBackgroundLayer

The struct was just the index. So just pass the index around instead of
a full struct.

themingbackground: Use get_box() for background image size

themingbackground: Introduce gtk_theming_background_get_box()

to query the different clip boxes used by the background drawing code.
Use this function to query these boxes when clipping.

themingbackground: Add content_box variable

... to go with border_box and padding_box.

themingbackground: Get rid of flags variable

themingbackground: Move image variable

... from the Layer struct onto the stack of the only function using it.

themingbackground: Restructure code some more

Move variable initialization outside the first code with side effects.
This allows adding some more early returns, including one for code that
used to trigger g_return_if_fail() in certain corner cases.

themingbackground: Restructure code

Make if statements encompassing the whole function into early returns.
The rest of the diff is reindenting.

cssimage: Add a warning for drawing empty images

width and height of an image must be > 0 for the image to get drawn.
It's up to the code further up to ensure that this is not happening.

reftests: Add a reftest for recent commit

Check that a computed background-size of 0 is treated as 0.

cssvalue: Compute "background-size: 0 0" properly

Previously a computed value of 0 was treated as "auto", which is wrong.

stylecontext: Don't use bg image in gtk_style_context_set_background()

Old code tried to use the "background-image" proeprty for setting the
default image background. While this used to work in the early days of
GTK3, today it is grossly misleading as the backgronud image may be
resized, repositioned and semi-translucent which causes very weird
artifacts when rendering.

So we use the background-color only instead.

cssimage: Add an equal vfunc

No implementations for it exist yet.

menuitem: Draw background unconditionally

settings: Use _gtk_css_provider_load_named()

This way we create one provider per settings object instead of stuffing
it into a global unchanging never-deleting hash table.
Also, we now reload the theme when instructed instead of keeping the old
loaded (and possibly stale) data forever.

https://bugzilla.gnome.org/show_bug.cgi?id=683896

cssprovider: Move fallback code into _gtk_css_provider_load_named()

This makes sure the full theme loading logic resides in one function and
isn't scattered around.

As a side-effect, the hash table kept by gtk_css_provider_get_named()
will now be populated with fallback themes. This will not be a problem
after the next commit though.

cssprovider: Export gtk_css_provider_load_named) function

... and document it.
For now, the function is only exported internally.

cssprovider: Split out theme loading function

Split maintaining the global themes hash table and the theme loading
code into two functions.
This also fixes leaking the provider when loading a theme from a builtin
resource.

Updated Galician translations

Updated Marathi Translations

Updated German translation

[l10n] Updated Estonian translation