Remove unneeded bool from RenderView::selectionBounds

Remove unneeded bool from RenderView::selectionBounds
    
The boolean was added in r21387 and the added test
(fast/overflow/scroll-nested-positioned-layer-in-overflow.html)
is still passing. AFAICT the only adverse potential for this removal
is to invalidate more (as we don't apply the ancestors' clips)
    
The only meaningful test change is
fast/repaint/text-selection-rect-in-overflow-2.html
which had bad invalidations anyway (halfway through the selection)
so I don't think it's meaningful.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=182197

[Julien - ping for review, 2014-09-15 17:56:45]

jchaffraix@chromium.org changed reviewers:
+ eseidel@chromium.org, esprehn@chromium.org, leviw@chromium.org,
yosin@chromium.org

[leviw_travelin_and_unemployed, 2014-09-15 19:31:46]

lgtm!

[yosin_UTC9, 2014-09-16 03:36:36]

LGTM (^_^)b
Thanks for cleanup! 
|bool| parameter makes our life harder...

[Julien - ping for review, 2014-09-16 17:13:29]

The CQ bit was checked by jchaffraix@chromium.org

[commit-bot: I haz the power, 2014-09-16 17:13:42]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/570763003/1

[commit-bot: I haz the power, 2014-09-16 17:13:55]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-16 17:13:56]

Failed to apply patch for Source/core/rendering/RenderSelectionInfo.h:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file Source/core/rendering/RenderSelectionInfo.h
  Hunk #1 FAILED at 66.
  1 out of 1 hunk FAILED -- saving rejects to file
Source/core/rendering/RenderSelectionInfo.h.rej

Patch:       Source/core/rendering/RenderSelectionInfo.h
Index: Source/core/rendering/RenderSelectionInfo.h
diff --git a/Source/core/rendering/RenderSelectionInfo.h
b/Source/core/rendering/RenderSelectionInfo.h
index
79ca4ba5049c4a6ab0b885d9c7e86e4714d1ab18..f3ad4eb419f82fbc30262b1d59d4bc82e424cc62
100644
--- a/Source/core/rendering/RenderSelectionInfo.h
+++ b/Source/core/rendering/RenderSelectionInfo.h
@@ -66,11 +66,11 @@ protected:
 // This struct is used when the selection changes to cache the old and new
state of the selection for each RenderObject.
 class RenderSelectionInfo FINAL : public RenderSelectionInfoBase {
 public:
-    RenderSelectionInfo(RenderObject* o, bool clipToVisibleContent)
+    RenderSelectionInfo(RenderObject* o)
         : RenderSelectionInfoBase(o)
     {
         if (o->canUpdateSelectionOnRootLineBoxes()) {
-            m_rect =
o->selectionRectForPaintInvalidation(m_paintInvalidationContainer,
clipToVisibleContent);
+            m_rect =
o->selectionRectForPaintInvalidation(m_paintInvalidationContainer);
             // FIXME: groupedMapping() leaks the squashing abstraction. See
RenderBlockSelectionInfo for more details.
             if (m_paintInvalidationContainer &&
m_paintInvalidationContainer->layer()->groupedMapping())
                
RenderLayer::mapRectToPaintBackingCoordinates(m_paintInvalidationContainer,
m_rect);

[Julien - ping for review, 2014-09-16 22:11:47]

The CQ bit was checked by jchaffraix@chromium.org

[commit-bot: I haz the power, 2014-09-16 22:12:07]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/570763003/20001

[commit-bot: I haz the power, 2014-09-16 22:23:34]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-16 22:23:36]

Try jobs failed on following builders:
  blink_presubmit on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/1...)

[Julien - ping for review, 2014-09-17 01:00:54]

Eric, Elliott: Your Source/web/ approval is required for a 1-line change in this
directory.

[esprehn, 2014-09-17 01:15:16]

lgtm

[Julien - ping for review, 2014-09-17 17:37:52]

The CQ bit was checked by jchaffraix@chromium.org

[commit-bot: I haz the power, 2014-09-17 17:38:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/570763003/20001

[commit-bot: I haz the power, 2014-09-17 17:47:48]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-17 17:47:49]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/27506)

[Julien - ping for review, 2014-09-17 19:31:09]

The CQ bit was checked by jchaffraix@chromium.org

[commit-bot: I haz the power, 2014-09-17 19:31:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/570763003/40001

[commit-bot: I haz the power, 2014-09-17 19:36:25]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-17 19:36:26]

Try jobs failed on following builders:
  linux_chromium_gn_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_chromium_gn_rel/bu...)
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/57323)

[Julien - ping for review, 2014-09-17 21:50:24]

The CQ bit was checked by jchaffraix@chromium.org

[commit-bot: I haz the power, 2014-09-17 21:51:18]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/570763003/60001

[commit-bot: I haz the power, 2014-09-17 22:57:55]

Committed patchset #4 (id:60001) as 182197