Update RenderViewTest::LoadHTML documentation, and remove spurious commits.

Update LoadHTML documentation, and remove spurious commits.

All these calls to ProcessPendingMessages() after LoadHTML() were redundant,
since LoadHTML directly forces a commit.

BUG=None


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=138895

[gavinp, 2012-05-24 14:27:37]

jam,

These changes were in my index after my last RenderViewTest work.  WDYT?

[jochen (gone - plz use gerrit), 2012-05-24 14:36:02]

https://chromiumcodereview.appspot.com/10408089/diff/1/content/test/render_vi...
File content/test/render_view_test.h (right):

https://chromiumcodereview.appspot.com/10408089/diff/1/content/test/render_vi...
content/test/render_view_test.h:74: // Loads the given HTML into the main frame
as a data: URL and commits.
what about " and blocks until the navigation is committed?"

[jam, 2012-05-24 16:25:33]

lgtm from my pov, Jochen's comment suggestion also sounds good

[gavinp, 2012-05-24 18:12:25]

jochen, WDYT?

http://codereview.chromium.org/10408089/diff/1/content/test/render_view_test.h
File content/test/render_view_test.h (right):

http://codereview.chromium.org/10408089/diff/1/content/test/render_view_test....
content/test/render_view_test.h:74: // Loads the given HTML into the main frame
as a data: URL and commits.
On 2012/05/24 14:36:03, jochen wrote:
> what about " and blocks until the navigation is committed?"

Done.

[jochen (gone - plz use gerrit), 2012-05-24 18:25:53]

lgtm, thanks

[commit-bot: I haz the power, 2012-05-24 18:29:18]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/gavinp@chromium.org/10408089/13002

[commit-bot: I haz the power, 2012-05-24 21:28:59]

Change committed as 138895