[BlackBerry] Implement Web Inspector methods in LayoutTestController
Created attachment 136242 [details] Patch
Comment on attachment 136242 [details] Patch This patch looks straightforward to me. r=me.
Comment on attachment 136242 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=136242&action=review > Tools/DumpRenderTree/blackberry/LayoutTestControllerBlackBerry.cpp:507 > + WTF::String scriptStr = jsStringRefToWebCoreString(script); Nit: I don't see the value of abbreviating the word "string" here. I suggest naming this variable scriptString.
Attachment 136242 [details] was posted by a committer and has review+, assigning to Konrad Piascik for commit.
Comment on attachment 136242 [details] Patch This change doesn't do anything and is simply a setup for future work to get inspector tests to work. I'll upload another patch which will have this work with other changes later rather than splitting it into two separate commits.
Comment on attachment 136242 [details] Patch Cleared Daniel Bates's review+ from obsolete attachment 136242 [details] so that this bug does not appear in http://webkit.org/pending-commit.