Plumb selection bounds as a single unit

Source/core/layout/compositing/CompositedSelectionBounds.h

 /*
  * Copyright (C) 2015 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 10 matching lines @@
  * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
  * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
  * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef WebImageGenerator_h
-#define WebImageGenerator_h
+#ifndef CompositedSelectionBounds_h
+#define CompositedSelectionBounds_h
 
-#include "WebCommon.h"
-
-class SkData;
-class SkImageGenerator;
+#include "core/layout/compositing/CompositedSelectionBound.h"
+#include "platform/geometry/FloatPoint.h"
+#include "platform/graphics/GraphicsLayer.h"
 
 namespace blink {
 
-class BLINK_EXPORT WebImageGenerator {
-public:
-    static SkImageGenerator* create(SkData*);
+struct CompositedSelectionBounds {

[Rick Byers, 2015/03/12 19:27:24]

can you please add a short comment describing the purpose of this class?  Maybe
say to see WebSelectionBounds for details?

[jdduke (slow), 2015/03/16 18:15:41]

Done.

+
+    CompositedSelectionBounds()
+        : isEditable(false)
+        , isEditableRegionEmpty(false)
+    {
+    }
+
+    CompositedSelectionBound start;
+    CompositedSelectionBound end;
+    bool isEditable;
+    bool isEditableRegionEmpty;

[yosin_UTC9, 2015/03/13 02:06:37]

Q: Could you tell me what "Region" is?

[jdduke (slow), 2015/03/16 18:15:41]

Sure.

In short, we want to know whether we can 1) enter text into the region (edit
it), and 2) if we can enter text, whether the area we will enter text into is
empty. We use this metadata to conditionally show the insertion handles in the
browser, aligning with native platform defaults. If you can think of a better
name, by all means do so!

 };
 
 } // namespace blink
 
 #endif