Upstreaming InPageSearchManager.cpp and InPageSearchManager.h for BlackBerry port
Created attachment 127009 [details] Initial upstreaming for find-in-page for BlackBerry
Comment on attachment 127009 [details] Initial upstreaming for find-in-page for BlackBerry View in context: https://bugs.webkit.org/attachment.cgi?id=127009&action=review Some stuff still needs to be fixed. > Source/WebKit/ChangeLog:5 > + Maybe add an explanation why this change is made. > Source/WebKit/blackberry/WebKitSupport/InPageSearchManager.h:41 > + bool findNextString(const WTF::String& text, bool forward); I dont think you need the WTF:: prefix.
Created attachment 127022 [details] Initial upstreaming for find-in-page for BlackBerry
Comment on attachment 127022 [details] Initial upstreaming for find-in-page for BlackBerry Looks good.
Comment on attachment 127022 [details] Initial upstreaming for find-in-page for BlackBerry Clearing flags on attachment: 127022 Committed r107736: <http://trac.webkit.org/changeset/107736>
All reviewed patches have been landed. Closing bug.