android_webview : Class member variable WeakPtrFactory in proper order.

android_webview/browser/net/android_stream_reader_url_request_job.h

Index: android_webview/browser/net/android_stream_reader_url_request_job.h
diff --git a/android_webview/browser/net/android_stream_reader_url_request_job.h b/android_webview/browser/net/android_stream_reader_url_request_job.h
index b7d6a621c8144b6c249a9ffe3b21a011ebe83fa2..8b7cafb557d8e46687006b86eed063d065f7c987 100644
--- a/android_webview/browser/net/android_stream_reader_url_request_job.h
+++ b/android_webview/browser/net/android_stream_reader_url_request_job.h
@@ -116,9 +116,10 @@ class AndroidStreamReaderURLRequestJob : public net::URLRequestJob {
   scoped_ptr<net::HttpResponseInfo> response_info_;
   scoped_ptr<Delegate> delegate_;
   scoped_refptr<InputStreamReaderWrapper> input_stream_reader_wrapper_;
-  base::WeakPtrFactory<AndroidStreamReaderURLRequestJob> weak_factory_;
   base::ThreadChecker thread_checker_;
 
+  base::WeakPtrFactory<AndroidStreamReaderURLRequestJob> weak_factory_;

[mkosiba (inactive), 2014/09/01 08:59:37]

maybe add a commend saying that the WeakPtrFactory needs to be last in order to
prevent the next person adding something below it?

[mkosiba (inactive), 2014/09/01 09:00:24]

ah, just looked at the bug - are you going to add a clang checker?

+
   DISALLOW_COPY_AND_ASSIGN(AndroidStreamReaderURLRequestJob);
 };