Consolidate how ImageDocument sets image styling

third_party/WebKit/Source/core/html/ImageDocument.h

Index: third_party/WebKit/Source/core/html/ImageDocument.h
diff --git a/third_party/WebKit/Source/core/html/ImageDocument.h b/third_party/WebKit/Source/core/html/ImageDocument.h
index 3f379819a160d57e665cf29f0bd57f51e580e3e0..5065c4649356834b7dc79b31ed23ca1f7eeaa218 100644
--- a/third_party/WebKit/Source/core/html/ImageDocument.h
+++ b/third_party/WebKit/Source/core/html/ImageDocument.h
@@ -87,7 +87,11 @@ class CORE_EXPORT ImageDocument final : public HTMLDocument {
   bool m_imageIsLoaded;
 
   // Size of the checkerboard background tiles
-  int m_checkerSize;
+  int m_styleCheckerSize;
+
+  // Desktop: State of the mouse cursor in the image style
+  enum MouseCursorMode { Uninitialized, Default, ZoomIn, ZoomOut };

[pdr., 2016/11/22 04:15:12]

Is the uninitialized case needed?

[gone, 2016/11/22 18:51:29]

Yeah, it's needed to get past the difference check in ImageDocument.cpp:420 for
initialization.  Alternatively, we could store an initialization boolean, or
check if the style string is empty.  Not sure what's the most intuitive.

+  MouseCursorMode m_styleMouseCursorMode;
 
   enum ShrinkToFitMode { Viewport, Desktop };
   ShrinkToFitMode m_shrinkToFitMode;