[Streams] Pass WebDataConsumerHandle when the response arrives.

Source/core/fileapi/FileReaderLoader.cpp

 /*
  * Copyright (C) 2010 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 153 matching lines @@
 
     // If we get any error, we do not need to keep a buffer around.
     if (m_errorCode) {
         m_rawData.clear();
         m_stringResult = "";
         m_isRawDataConverted = true;
         m_decoder.clear();
     }
 }
 
-void FileReaderLoader::didReceiveResponse(unsigned long, const ResourceResponse& response)
+void FileReaderLoader::didReceiveResponse(unsigned long, const ResourceResponse& response, PassOwnPtr<WebDataConsumerHandle> handle)
 {
+    ASSERT_UNUSED(handle, !handle);

[Mike West, 2014/11/04 15:09:31]

Why this ASSERT?

[yhirano, 2014/11/05 04:38:40]

I'm planning to use this parameter only for XHR and fetch API (at least, at
first). FileReaderLoaderClient has no interface for WebDataConsumerHandle and
having non-null handle here means something wrong happens. Hence I added this
assertion.

     if (response.httpStatusCode() != 200) {
         failed(httpStatusCodeToErrorCode(response.httpStatusCode()));
         return;
     }
 
     // A negative value means that the content length wasn't specified.
     m_totalBytes = response.expectedContentLength();
 
     long long initialBufferLength = -1;
 
@@ skipping 213 matching lines @@
     m_stringResult = builder.toString();
 }
 
 void FileReaderLoader::setEncoding(const String& encoding)
 {
     if (!encoding.isEmpty())
         m_encoding = WTF::TextEncoding(encoding);
 }
 
 } // namespace blink