Invalidate Custom Scrollbars irrespective of focused frame.

Source/core/editing/FrameSelection.cpp

 /*
  * Copyright (C) 2004, 2008, 2009, 2010 Apple 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:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 1447 matching lines @@
 {
     m_frame->eventHandler().notifySelectionChanged();
 }
 
 void FrameSelection::focusedOrActiveStateChanged()
 {
     bool activeAndFocused = isFocusedAndActive();
 
     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();
 
     // Because RenderObject::selectionBackgroundColor() and
     // RenderObject::selectionForegroundColor() check if the frame is active,
     // we have to update places those colors were painted.
     if (RenderView* view = document->renderView())
         view->invalidatePaintForSelection();
 
@@ skipping 440 matching lines @@
     sel.showTreeForThis();
 }
 
 void showTree(const blink::FrameSelection* sel)
 {
     if (sel)
         sel->showTreeForThis();
 }
 
 #endif