[scheduler] Plumb websocket information to scheduler

third_party/WebKit/Source/platform/WebFrameScheduler.h

Index: third_party/WebKit/Source/platform/WebFrameScheduler.h
diff --git a/third_party/WebKit/Source/platform/WebFrameScheduler.h b/third_party/WebKit/Source/platform/WebFrameScheduler.h
index 069fedf6b2bbc6dd2eac8d1e6ed62e3edc2c383f..8ef832481e05884d553278cd8bd2e21aba96f9cf 100644
--- a/third_party/WebKit/Source/platform/WebFrameScheduler.h
+++ b/third_party/WebKit/Source/platform/WebFrameScheduler.h
@@ -59,6 +59,11 @@ class WebFrameScheduler {
   // policy decisions based on this.
   virtual void didStopLoading(unsigned long identifier) {}
 
+  // Tells the scheduler that websocket connections was established or
+  // destroyed.
+  virtual void didOpenWebSocket() {}
+  virtual void didCloseWebSocket() {}

[alex clarke (OOO till 29th), 2017/01/24 18:15:33]

I wonder if these should really all be pure virtual (possibly not in this
patch),  Sami / Haraken WDYT?

[haraken, 2017/01/24 21:49:08]

Yeah, pure virtual sounds nicer.

[altimin, 2017/01/26 11:26:15]

I prefer keeping EmptyFrameScheduler free of unneeded overrides.

[Sami, 2017/01/26 15:53:34]

I think everything here should be pure virtual. The reason the existing methods
aren't is because the header and the implementation used to live in different
repositories, but we can't use that excuse any longer :)

[alex clarke (OOO till 29th), 2017/01/26 16:02:36]

Fell free to add TODO(alexclarke): Make this API pure virtual

+
   // Tells the scheduler if we are parsing a document on another thread. This
   // tells the scheduler not to advance virtual time if it's using the
   // DETERMINISTIC_LOADING policy.