Move SharedImageBackingGL functions to appropriate places

This file has gotten jumbled by keeping diffs small. This does nothing
but move code around.

TBR=geofflang

Bug: 1092155
Change-Id: I6951d61ce1ae96cde61cd879c482bd8a20641f80
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2251160Reviewed-by: ccameron <ccameron@chromium.org>
Commit-Queue: ccameron <ccameron@chromium.org>
Cr-Commit-Position: refs/heads/master@{#780084}

gpu/command_buffer/service/shared_image_backing_factory_gl_texture_internal.h

@@ -144,16 +144,10 @@ class SharedImageBackingGLTexture : public SharedImageBacking {
 
  private:
   // SharedImageBacking:
-  void Update(std::unique_ptr<gfx::GpuFence> in_fence) override;
   void OnMemoryDump(const std::string& dump_name,
                     base::trace_event::MemoryAllocatorDump* dump,
                     base::trace_event::ProcessMemoryDump* pmd,
                     uint64_t client_tracing_id) override;
-  std::unique_ptr<SharedImageRepresentationSkia> ProduceSkia(
-      SharedImageManager* manager,
-      MemoryTypeTracker* tracker,
-      scoped_refptr<SharedContextState> context_state) override;
-
   gfx::Rect ClearedRect() const final;
   void SetClearedRect(const gfx::Rect& cleared_rect) final;
   bool ProduceLegacyMailbox(MailboxManager* mailbox_manager) final;
@@ -167,6 +161,11 @@ class SharedImageBackingGLTexture : public SharedImageBacking {
       SharedImageManager* manager,
       MemoryTypeTracker* tracker,
       WGPUDevice device) final;
+  std::unique_ptr<SharedImageRepresentationSkia> ProduceSkia(
+      SharedImageManager* manager,
+      MemoryTypeTracker* tracker,
+      scoped_refptr<SharedContextState> context_state) override;
+  void Update(std::unique_ptr<gfx::GpuFence> in_fence) override;
 
   bool IsPassthrough() const { return is_passthrough_; }
 
@@ -205,20 +204,11 @@ class SharedImageBackingGLImage : public SharedImageBacking {
 
  private:
   // SharedImageBacking:
-  void Update(std::unique_ptr<gfx::GpuFence> in_fence) override;
+  scoped_refptr<gfx::NativePixmap> GetNativePixmap() override;
   void OnMemoryDump(const std::string& dump_name,
                     base::trace_event::MemoryAllocatorDump* dump,
                     base::trace_event::ProcessMemoryDump* pmd,
                     uint64_t client_tracing_id) override;
-  scoped_refptr<gfx::NativePixmap> GetNativePixmap() override;
-  std::unique_ptr<SharedImageRepresentationSkia> ProduceSkia(
-      SharedImageManager* manager,
-      MemoryTypeTracker* tracker,
-      scoped_refptr<SharedContextState> context_state) override;
-  std::unique_ptr<SharedImageRepresentationGLTexture>
-  ProduceRGBEmulationGLTexture(SharedImageManager* manager,
-                               MemoryTypeTracker* tracker) override;
-
   gfx::Rect ClearedRect() const final;
   void SetClearedRect(const gfx::Rect& cleared_rect) final;
   bool ProduceLegacyMailbox(MailboxManager* mailbox_manager) final;
@@ -232,6 +222,14 @@ class SharedImageBackingGLImage : public SharedImageBacking {
       SharedImageManager* manager,
       MemoryTypeTracker* tracker,
       WGPUDevice device) final;
+  std::unique_ptr<SharedImageRepresentationSkia> ProduceSkia(
+      SharedImageManager* manager,
+      MemoryTypeTracker* tracker,
+      scoped_refptr<SharedContextState> context_state) override;
+  std::unique_ptr<SharedImageRepresentationGLTexture>
+  ProduceRGBEmulationGLTexture(SharedImageManager* manager,
+                               MemoryTypeTracker* tracker) override;
+  void Update(std::unique_ptr<gfx::GpuFence> in_fence) override;
 
   void BeginSkiaReadAccess();
   bool IsPassthrough() const { return is_passthrough_; }