Name already in use

* Now it's possible to push tag after inspecting results, no need to
  rewrite pushed tag if something went wrong.
* Removed ugly hack with secrets.QTWEBKIT_BUILD_NUMBER - it's not a secret
  but build input.
* Don't hardcode Qt version in workflow file, made it build input as well
* Removed dependency on 3rd-party action

Change-Id: I7bd85b354f38138b7a60fa0a272598b865d8c1b8

… WIP

https://bugs.webkit.org/show_bug.cgi?id=211193

Reviewed by Adrian Perez de Castro.

Fixes the following errors,

Source/WebCore/html/MediaElementSession.cpp:1059:9: error: type 'WebCore::RenderMedia *' cannot be narrowed to 'bool' in initializer list [-Wc++11-narrowing]
m_element.renderer(),
^~~~~~~~~~~~~~~~~~~~

Source/WebCore/style/StyleResolver.cpp:106:55: error: type 'const char [4]' cannot be narrowed to 'bool' in initializer list [-Wc++11-narrowing]
m_mediaQueryEvaluator = MediaQueryEvaluator { "all" };
                                              ^~~~~
Source/WebCore/style/StyleResolver.cpp:106:55: note: insert an explicit cast to silence this issue
m_mediaQueryEvaluator = MediaQueryEvaluator { "all" };
                                              ^~~~~
                                              static_cast<bool>( )

* html/HTMLMediaElement.h:
(WebCore::HTMLMediaElement::hasRenderer const):
MediaElementSession was implicitly casting a pointer to a bool,
which is not allowed with modern Clang checks. Add a helper method
to encapsulate the now required static_cast<bool>.
* html/MediaElementSession.cpp: Use the new helper method to see
if the HTMLMediaElement has an associated renderer.
(WebCore::MediaElementSession::updateMediaUsageIfChanged):
* style/StyleResolver.cpp: This was calling MediaQueryEvaluator {
"all" }; and seemingly expecting to cast a const char[] to a bool,
or maybe String? It's confusing because of the MediaQueryEvaluator
API. If it was implicitly converting to bool then that could be
unintentional. Such casts are not allowed either now. The
MediaQueryEvaluator's default constructor says it returns true for
"all", which appears to be the original intent of this call, so I
replaced it with that.
(WebCore::Style::Resolver::Resolver):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@260951 268f45cc-cd09-0410-ab3c-d52691b4dbfc

"QObject::startTimer: Timers can only be used with threads started with
QThread"

Change-Id: I446f0ed6a94ef1ad896440e0a6d719ba2e596beb

QImage is internally a pointer to data with shared ownership, which makes
its behavior close to RefPtr<cairo_surface_t> and RetainPtr<NSImage>.

OTOH, this change a) breaks source compatibility with nullptr and boolean
conversions; b) can lead to unexpected detaches and writing image changes
to object's copy.

For some reason, this change fixes existing issues with image decoding.

TODO: Implement wrapper object for QImage, check ownership model, make
sure that QPixmap::fromImage is called with QImage&& where appropriate

Change-Id: I7e9f8e69f4fdcb335dd7b055b659b7d8b36a2e89

…mage

See r122720.

Missing fixes:
* ImageGStreamer
* GraphicsContext3DQt
* WK2
* ImageDecoderQt

Change-Id: I1d393fc982623c441bd81bd53ed59b8ddb7fed6c

Change-Id: I533e2a1b52911ee9fe6e976c851229aee9637ab7

Change-Id: I1fe5261563942c1a1395ef4e6ac12f7b5eb571d2

… fly

Change-Id: I2c79b33437c36d77460322fe956ff335b404e830

Change-Id: Ibe1eec5eb6555fca4e19cd0e2efceb9dee8f4c97

After r224267 we need to use asynchronous ResourceHandleClient calls in
WebKit1.

Change-Id: I8d7838fff806affea310182ecea7c4b71e596c62

Change-Id: Ia9f65fec13629acaeb79101974719caac72afd12

Need to deprecate it properly

Change-Id: If966bfeec618493f10cca56cb0381047c855cefd