Fix potential incorrect sync flags used with screenshare TL stream.

webrtc/modules/video_coding/codecs/vp8/temporal_layers.h

Index: webrtc/modules/video_coding/codecs/vp8/temporal_layers.h
diff --git a/webrtc/modules/video_coding/codecs/vp8/temporal_layers.h b/webrtc/modules/video_coding/codecs/vp8/temporal_layers.h
index 9b122919380684bf651d35fc7753bda93496c976..b3601e69f0d6aa15ba498d55ce8f436649093eb1 100644
--- a/webrtc/modules/video_coding/codecs/vp8/temporal_layers.h
+++ b/webrtc/modules/video_coding/codecs/vp8/temporal_layers.h
@@ -55,6 +55,16 @@ class TemporalLayers {
 
     int pattern_idx;
 
+    bool operator==(const FrameConfig& o) const {

[pbos-webrtc, 2017/07/12 18:05:46]

Where do we need this operator?

[pbos-webrtc, 2017/07/12 18:26:45]

.. if we don't, can we delete it?

[sprang_webrtc, 2017/07/13 10:50:50]

It's used in the test. I could add a comparator in the test suite instead, but
then we risk code duplication and missing changes in this struct, so would
prefer to keep it here.

+      return drop_frame == o.drop_frame &&
+             last_buffer_flags == o.last_buffer_flags &&
+             golden_buffer_flags == o.golden_buffer_flags &&
+             arf_buffer_flags == o.arf_buffer_flags &&
+             layer_sync == o.layer_sync && freeze_entropy == o.freeze_entropy &&
+             pattern_idx == o.pattern_idx;
+    }
+    bool operator!=(const FrameConfig& o) const { return !(*this == o); }
+
    private:
     FrameConfig(BufferFlags last,
                 BufferFlags golden,