The CI favicon was missing in the jobs page.
This commit fixes it.

Closes #53888

When a branch is deleted, the push commit IDs is an empty array.
The previous change would attempt to call `[].exists?`, which is
invalid. Fix this by returning `MergeRequestDiffCommit.none` instead.

Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/53853

Previously it was using name which could make it hard
to filter for the file you want in a certain folder.

Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/53781

Extract code to make it easier reusable
- introduce AttributesRewriter and ContentRewriter
  - support group entites when rewriting content
- make Uploader copy_to working for Namespaces

In Ruby 2.4, `URI.join("http://test//", "a").to_s` will
remove the double slash, however it's not the case in
Ruby 2.5. Using chomp should work better for the intention,
as we're not trying to allow things like ../ or / paths
resolution.

Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/53180

Updates clipboard button component to use
the new glTooltipDirective and the new glButton
component from gitlab-ui

Fixed duplicate TaskList requests being made

I have also extracted an issue in
https://gitlab.com/gitlab-org/gitlab-ce/issues/53738 to surface these
errors to users but now I am just fixing the bug where
kubernetes_namespace is undefined while handling this error

There is no reason to specially handle 404 here. We only handle 404
specifically if we are fetching something and want to return `nil` if
404.  This does not apply here.

This improves the initial rendering performance of notes
by only creating a new TaskList for the whole app,
rather than on a per-note component basis.

This reverts commit 8c126525.

Previously the string was spanning multiple lines and included a
needless `\n` character in the resulting error message.

This change also reduces duplication by assigning two variables.

However, we only need to raise for project_type clusters to maintain
previous behaviour.

In all probablity this requirement to have actual_namespace came from
KubernetesService and will no longer be required soon.

With multiple discussion requests we loose the reactivity of the
previously stored discussions state.
This means that certain features, toggling line discussions for example,
are no longer reactive.

Use presenters instead otherwise we will have x2 overides in EE

Refacotr project or group specific `if` logic to be contained within
each presenter instead.

Also add :override to ensure super method exists.

Use model method as single source of truth instead of splitting between
presenter and Kubernetes model