Notifying the Out of Process Renderer about Visibility Change of a Remote Frame

third_party/WebKit/Source/web/tests/FrameTestHelpers.h

 /*
  * Copyright (C) 2011 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 145 matching lines @@
 
     WebRemoteFrameImpl* frame() const { return m_frame; }
 
     // WebRemoteFrameClient overrides:
     void frameDetached(DetachType) override;
     void postMessageEvent(
         WebLocalFrame* sourceFrame,
         WebRemoteFrame* targetFrame,
         WebSecurityOrigin targetOrigin,
         WebDOMMessageEvent) override { }
+    void visibilityChanged(bool visible) override { m_visible = visible; }

[dcheng, 2016/01/06 18:43:58]

Move these to a subclass of TestWebRemoteFrameClient? I don't think this is a
method that's going to be used by other tests.

[EhsanK, 2016/01/08 22:05:37]

Done.

+
+    bool isVisible() { return m_visible; }

[dcheng, 2016/01/06 18:43:58]

Nit: const

[EhsanK, 2016/01/08 22:05:37]

Done.

 
 private:
     RawPtrWillBePersistent<WebRemoteFrameImpl> const m_frame;
+    bool m_visible;
 };
 
 class TestWebViewClient : public WebViewClient {
 public:
     virtual ~TestWebViewClient() { }
     void initializeLayerTreeView() override;
     WebLayerTreeView* layerTreeView() override { return m_layerTreeView.get(); }
 
 private:
     OwnPtr<WebLayerTreeView> m_layerTreeView;
 };
 
 } // namespace FrameTestHelpers
 } // namespace blink
 
 #endif // FrameTestHelpers_h