GitLab

When toplevel window uses a native frame, the server-side decoration is
set and when toplevel window uses a custom frame, the client-side
decoration is set. UseCustomFrame() decides on which frame (native or
custom) to be used for a window.

Devtool chooses a native frame but also draws frame header
(client-side decoration) causing a duplicate decoration for a devtool
window.

BrowserFrame::ShouldDrawFrameHeader always returns `true` assuming
that custom frame is used but instead, this function should check frame
type before drawing a frame header.

Bug: 1155551
Change-Id: I903a194ecf875f51d5ade726b57651c57bd7b028
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2573711Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Commit-Queue: Abhijeet Kandalkar <abhijeet@igalia.com>
Cr-Commit-Position: refs/heads/master@{#836680}