Eliminate use of StringPiece(nullptr, n) with non zero n. This constructor is unsupported in C++17'…

net/quic/core/quic_stream_sequencer_test.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/quic/core/quic_stream_sequencer.h"
 
 #include <algorithm>
 #include <cstdint>
 #include <memory>
 #include <utility>
@@ skipping 644 matching lines @@
   EXPECT_EQ(3u, sequencer_->NumBytesBuffered());
 
   EXPECT_TRUE(sequencer_->GetReadableRegion(iovecs, &timestamp));
   EXPECT_EQ(timestamp, t1);
   QuicStreamSequencerTest::ConsumeData(3);
   EXPECT_EQ(0u, NumBufferedBytes());
   EXPECT_EQ(6u, sequencer_->NumBytesConsumed());
   EXPECT_EQ(0u, sequencer_->NumBytesBuffered());
 }
 
+// TODO(danzh): Figure out the way to implement this test case without the use
+// of unsupported StringPiece constructor.
+#if 0
 TEST_F(QuicStreamSequencerTest, OnStreamFrameWithNullSource) {
   // Pass in a frame with data pointing to null address, expect to close
   // connection with error.
   StringPiece source(nullptr, 5u);
   QuicStreamFrame frame(kClientDataStreamId1, false, 1, source);
   EXPECT_CALL(stream_, CloseConnectionWithDetails(
                            QUIC_STREAM_SEQUENCER_INVALID_STATE, _));
   sequencer_->OnStreamFrame(frame);
 }
+#endif
 
 TEST_F(QuicStreamSequencerTest, ReadvError) {
   EXPECT_CALL(stream_, OnDataAvailable());
   string source(100, 'a');
   OnFrame(0u, source.data());
   EXPECT_EQ(source.length(), sequencer_->NumBytesBuffered());
   // Pass in a null iovec, expect to tear down connection.
   EXPECT_CALL(stream_, CloseConnectionWithDetails(
                            QUIC_STREAM_SEQUENCER_INVALID_STATE, _));
   iovec iov{nullptr, 512};
   sequencer_->Readv(&iov, 1u);
 }
 
 }  // namespace
 }  // namespace test
 }  // namespace net