Previously, all of the data members in FrameLoadState were public and accessed directly from outside the class. But these should all be private as noted in FrameLoadState.h for better encapsulation.
Created attachment 252297 [details] Patch
Comment on attachment 252297 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=252297&action=review > Source/WebKit2/UIProcess/FrameLoadState.h:54 > + const State& state() const { return m_state; } I think the type here should just be State since itβs a small scalar.
Created attachment 252348 [details] Patch
Addressed darin's comments.
Comment on attachment 252348 [details] Patch Clearing flags on attachment: 252348 Committed r183813: <http://trac.webkit.org/changeset/183813>
All reviewed patches have been landed. Closing bug.