Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(180)

Side by Side Diff: third_party/WebKit/Source/platform/graphics/ThreadSafeDataTransport.h

Issue 1484853003: Ganesh: images upload to GPU performance fix (skip copying encoded data) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: review update, assert for appending data to m_allDataReceived Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2012 Google Inc. All rights reserved. 2 * Copyright (C) 2012 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * 1. Redistributions of source code must retain the above copyright 7 * 1. Redistributions of source code must retain the above copyright
8 * notice, this list of conditions and the following disclaimer. 8 * notice, this list of conditions and the following disclaimer.
9 * 2. Redistributions in binary form must reproduce the above copyright 9 * 2. Redistributions in binary form must reproduce the above copyright
10 * notice, this list of conditions and the following disclaimer in the 10 * notice, this list of conditions and the following disclaimer in the
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
56 void data(SharedBuffer**, bool* allDataReceived); 56 void data(SharedBuffer**, bool* allDataReceived);
57 57
58 // Return true of there is new data submitted to this class 58 // Return true of there is new data submitted to this class
59 // since last time data() was called. 59 // since last time data() was called.
60 bool hasNewData(); 60 bool hasNewData();
61 61
62 private: 62 private:
63 Mutex m_mutex; 63 Mutex m_mutex;
64 64
65 Vector<RefPtr<SharedBuffer>> m_newBufferQueue; 65 Vector<RefPtr<SharedBuffer>> m_newBufferQueue;
66 bool m_newAllDataReceived;
chrishtr 2015/12/02 16:44:29 How about m_allDataReceivedForNewBufferQueue. Long
scroggo_chromium 2015/12/02 20:15:01 +1. I find m_newAllDataReceived confusing.
66 RefPtr<SharedBuffer> m_readBuffer; 67 RefPtr<SharedBuffer> m_readBuffer;
67 bool m_allDataReceived; 68 bool m_allDataReceived;
68 size_t m_readPosition; 69 size_t m_readPosition;
69 }; 70 };
70 71
71 } // namespace blink 72 } // namespace blink
72 73
73 #endif 74 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698