Invalidate Custom Scrollbars irrespective of focused frame.

Source/core/editing/FrameSelection.cpp

Index: Source/core/editing/FrameSelection.cpp
diff --git a/Source/core/editing/FrameSelection.cpp b/Source/core/editing/FrameSelection.cpp
index f497ad9d995889ab29a3aa8445c5c8450307ce0d..1d2f7432da67e81e8ca7a4d99f5a675d08597f6f 100644
--- a/Source/core/editing/FrameSelection.cpp
+++ b/Source/core/editing/FrameSelection.cpp
@@ -1465,8 +1465,10 @@ void FrameSelection::focusedOrActiveStateChanged()
 
     RefPtrWillBeRawPtr<Document> document = m_frame->document();
     // Invalidate All CustomScrollbars as CustomScrollbars supports
-    // CSS window-active attribute.
-    if (FrameView* view = document->view())
+    // CSS window-active attribute.Irrespective of focus on frame, this
+    // needs to be applied for entire page. So invalidate from root FrameView

[skobes, 2014/12/16 22:19:52]

If the frame's focus state is irrelevant, this seems like the wrong place to
trigger the invalidation.  Maybe it should hook into FocusController::setActive
instead?

[MuVen, 2014/12/17 17:17:15]

yes, i tried earlier with what you have mentioned. But the following API's for
Caret appears in the active frame, Element focusStateChanged will go for toss
which eventually fails lost of test cases. Due to these reasons i have written
it here.

[skobes, 2014/12/17 17:41:31]

I'm sorry, I don't understand what you're trying to say here.  Can you rephrase
this sentence and/or provide more detail?

[MuVen, 2014/12/17 18:06:52]

I have earlier tried calling focusorActiveStateChanged on root Frame from
SetActive;   
    Frame* frame = focusedOrMainFrame();
    if (frame->isLocalFrame())
        toLocalFrame(frame)->selection().pageActivationChanged();
//tried
toLocalFrame(frame)->localFrameRoot()->selection().pageActivationChanged

But many test cases have failed due to this as,
spellchecker/CaretVisiblilty/FocusedElement  api's implementation is based on
focused frame (if available).

[skobes, 2014/12/17 18:51:22]

I didn't mean for FocusController::setActive to call focusorActiveStateChanged
on the root frame.  I meant that FocusController::setActive should invalidate
all of the page's custom scrollbars.

[MuVen, 2014/12/17 18:59:43]

Oh ok ok !!! Got it, Make sense even. I shall update the same by tomo.
I'm sorry for miss-understanding & for my english.

Thanks,

+    // rather than focused frame.
+    if (FrameView* view = m_frame->localFrameRoot()->document()->view())
         view->invalidateAllCustomScrollbarsOnActiveChanged();
 
     document->updateRenderTreeIfNeeded();