Plumb selection bounds as a single unit

public/web/WebSelection.h

Index: public/web/WebSelection.h
diff --git a/public/web/WebSelection.h b/public/web/WebSelection.h
new file mode 100644
index 0000000000000000000000000000000000000000..1c334770580b70eece5ba1345b66ff6a363365fd
--- /dev/null
+++ b/public/web/WebSelection.h
@@ -0,0 +1,54 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef WebSelection_h
+#define WebSelection_h
+
+#include "../platform/WebCommon.h"
+#include "../platform/WebSelectionBound.h"

[Rick Byers, 2015/04/07 21:15:45]

Any reason why this file is in public/web instead of public/platform along with
WebSelectionBound?  I expected they'd be next to each other and it looks like
you don't need the different DEPS rules of public/web here (but I must admit I
don't fully understand the distinction between public/web and public/platform).

[jdduke (slow), 2015/04/07 21:31:15]

I think I have some logic in the WebSelection.cpp file that made it impossible
to put this in platform. However, I can move that logic just about anywhere, so
I'll go ahead and do that if you prefer (also moving this in to platform).

+
+namespace blink {
+
+struct CompositedSelection;
+
+// The active selection region, containing compositing data for the selection
+// end points as well as metadata for the selection region.
+class BLINK_EXPORT WebSelection {
+public:
+    enum SelectionType { NoSelection, CaretSelection, RangeSelection };
+
+#if INSIDE_BLINK
+    explicit WebSelection(const CompositedSelection&);
+#endif
+    WebSelection(const WebSelection&);
+
+    const WebSelectionBound& start() const { return m_start; }
+    const WebSelectionBound& end() const { return m_end; }
+
+    bool isNone() const { return selectionType() == NoSelection; }
+    bool isCaret() const { return selectionType() == CaretSelection; }
+    bool isRange() const { return selectionType() == RangeSelection; }
+
+    bool isEditable() const { return m_isEditable; }
+    bool isEmptyTextFormControl() const { return m_isEmptyTextFormControl; }
+
+private:
+    SelectionType selectionType() const { return m_selectionType; }
+
+    SelectionType m_selectionType;
+
+    WebSelectionBound m_start;
+    WebSelectionBound m_end;
+
+    // Whether the selection region consists of editable text.
+    bool m_isEditable;
+
+    // Whether the selection resides in an empty text form control. Note that
+    // this only applies to caret-type selections.
+    bool m_isEmptyTextFormControl;
+};
+
+} // namespace blink
+
+#endif // WebSelection_h