Adds type information to the NetworkStateNotifier. Also allows for registration of observers.

Source/core/page/NetworkStateNotifier.h

 /*
  * Copyright (C) 2008 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
  *    documentation and/or other materials provided with the distribution.
  *
  * THIS SOFTWARE IS PROVIDED BY APPLE INC. ``AS IS'' AND ANY
  * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
  * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
  * PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL APPLE INC. OR
  * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
  * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
  * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
  * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
  * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #ifndef NetworkStateNotifier_h
 #define NetworkStateNotifier_h
 
+#include "public/platform/WebNetworkConnection.h"
 #include "wtf/FastAllocBase.h"
+#include "wtf/HashMap.h"
 #include "wtf/Noncopyable.h"
 #include "wtf/ThreadingPrimitives.h"
+#include "wtf/Vector.h"
 
 namespace WebCore {
 
+class ExecutionContext;
+
 class NetworkStateNotifier {
     WTF_MAKE_NONCOPYABLE(NetworkStateNotifier); WTF_MAKE_FAST_ALLOCATED;
 public:
+    class NetworkStateObserver {
+    public:
+        // Will be called on the thread of the context passed in addObserver.
+        virtual void connectionTypeChange(
+            blink::WebNetworkConnection::ConnectionType) = 0;

[adamk, 2014/05/20 15:00:36]

Nit: Don't break lines like this in Blink; there's no line length limit, so
there's no need to break unless a line gets crazy long.

[jkarlin, 2014/05/20 16:20:23]

Done.

+    };
+
+    typedef NetworkStateObserver* ObserverType;

[adamk, 2014/05/20 15:00:36]

I found this typedef a bit confusing while reading the code, perhaps remove it?

[jkarlin, 2014/05/20 16:20:23]

Done.

+
     NetworkStateNotifier()
-        : m_isOnLine(true) { }
+        : m_isOnLine(true)
+        , m_type(blink::WebNetworkConnection::Other)
+    {
+    }
 
     bool onLine() const
     {
         MutexLocker locker(m_mutex);
         return m_isOnLine;
     }
 
     void setOnLine(bool);
 
+    blink::WebNetworkConnection::ConnectionType connectionType() const
+    {
+        MutexLocker locker(m_mutex);
+        return m_type;
+    }
+
+    void setWebConnectionType(blink::WebNetworkConnection::ConnectionType);
+
+    // Must be called on the context's thread. Do not call add/removeObserver
+    // while handling a connectionTypeChange notification as it mutates the
+    // container while it's being iterated over.
+    void addObserver(ObserverType, ExecutionContext*);
+    void removeObserver(ObserverType, ExecutionContext*);
+
 private:
+    struct ObserverList {
+        ObserverList()
+            : iterating(false)
+        {
+        }
+        bool iterating;
+        Vector<ObserverType> observers;
+    };
+
+    typedef HashMap<ExecutionContext*, ObserverList> ObserverListMap;
+
+    void notifyObserversOnContext(ExecutionContext*, blink::WebNetworkConnection::ConnectionType);
+
     mutable Mutex m_mutex;
     bool m_isOnLine;
+    blink::WebNetworkConnection::ConnectionType m_type;
+    ObserverListMap m_observers;
 };
 
 NetworkStateNotifier& networkStateNotifier();
-

[adamk, 2014/05/20 15:00:36]

Nit: I'd leave this blank line.

-};
+}

[adamk, 2014/05/20 15:00:36]

and maybe add a "// namespace WebCore" comment

[jkarlin, 2014/05/20 16:20:23]

Done.

 
 #endif // NetworkStateNotifier_h