[Screenshot] Fallback:fix close button z order

https://hsv.googleplex.com/6244305050009600

Bug: 1139773
Change-Id: If27b3ff02b867e972c3b1f579e707e643fe7047e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2490848Reviewed-by: Kristi Park <kristipark@chromium.org>
Commit-Queue: Jeffrey Cohen <jeffreycohen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#819645}

chrome/browser/share/android/java/res/layout/screenshot_share_sheet.xml

@@ -12,15 +12,6 @@
         android:layout_width="match_parent"
         android:layout_height="match_parent">
 
-    <!-- TODO (crbug.com/1034733) Fix that this button is being 
-    partially covered by the iamgeview -->
-        <org.chromium.ui.widget.ChromeImageButton
-            android:id="@+id/close_button"
-            style="@style/ToolbarButton"
-            android:src="@drawable/btn_close"
-            android:contentDescription="@string/close"
-            app:tint="@color/default_icon_color_tint_list" />
-
         <LinearLayout
             android:layout_width="match_parent"
             android:layout_height="match_parent"
@@ -99,6 +90,13 @@
 
         </LinearLayout>
 
+        <org.chromium.ui.widget.ChromeImageButton
+            android:id="@+id/close_button"
+            style="@style/ToolbarButton"
+            android:src="@drawable/btn_close"
+            android:contentDescription="@string/close"
+            app:tint="@color/default_icon_color_tint_list" />
+
     </RelativeLayout>
 
 </org.chromium.chrome.browser.share.screenshot.ScreenshotShareSheetView>