diff options
| author | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-05 16:31:31 +0000 |
|---|---|---|
| committer | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-05 16:31:31 +0000 |
| commit | 882daa9e07369d55b9e7647e0dd3b3316d3a74a4 (patch) | |
| tree | 63bea18fc9b174b91728fe8fc86ea1cd6628c6f4 /chrome/renderer/render_view_unittest.cc | |
| parent | 6898bbedb15bfddf91ea125e3ffde4b1789d9bf3 (diff) | |
| download | chromium_src-882daa9e07369d55b9e7647e0dd3b3316d3a74a4.zip chromium_src-882daa9e07369d55b9e7647e0dd3b3316d3a74a4.tar.gz chromium_src-882daa9e07369d55b9e7647e0dd3b3316d3a74a4.tar.bz2 | |
Makes it so that we sync changes in content state more often for selected tabs. This is necesitated by the fact that when closing a tab we don't grab the most recent content state, leaving session restore with slightly stale data. To really fix this requires sending a message on tab close and waiting for it, but that induces tab close jank, something we're not going to do.
BUG=17220
TEST=none
Review URL: http://codereview.chromium.org/342100
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@31097 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/renderer/render_view_unittest.cc')
| -rw-r--r-- | chrome/renderer/render_view_unittest.cc | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/chrome/renderer/render_view_unittest.cc b/chrome/renderer/render_view_unittest.cc index 2626096..8128c59 100644 --- a/chrome/renderer/render_view_unittest.cc +++ b/chrome/renderer/render_view_unittest.cc @@ -59,7 +59,7 @@ TEST_F(RenderViewTest, OnNavStateChanged) { // Don't want any delay for form state sync changes. This will still post a // message so updates will get coalesced, but as soon as we spin the message // loop, it will generate an update. - view_->set_delay_seconds_for_form_state_sync(0); + view_->set_send_content_state_immediately(true); LoadHTML("<input type=\"text\" id=\"elt_text\"></input>"); @@ -83,7 +83,7 @@ TEST_F(RenderViewTest, OnImeStateChanged) { view_->OnImeSetInputMode(true); // Load an HTML page consisting of two input fields. - view_->set_delay_seconds_for_form_state_sync(0); + view_->set_send_content_state_immediately(true); LoadHTML("<html>" "<head>" "</head>" @@ -211,7 +211,7 @@ TEST_F(RenderViewTest, ImeComposition) { // and move the input focus to the <div> element, where we can use // IMEs. view_->OnImeSetInputMode(ime_message->enable); - view_->set_delay_seconds_for_form_state_sync(0); + view_->set_send_content_state_immediately(true); LoadHTML("<html>" "<head>" "</head>" @@ -266,7 +266,7 @@ TEST_F(RenderViewTest, OnSetTextDirection) { // This test changes the text direction of the <textarea> element, and // writes the values of its 'dir' attribute and its 'direction' property to // verify that the text direction is changed. - view_->set_delay_seconds_for_form_state_sync(0); + view_->set_send_content_state_immediately(true); LoadHTML("<html>" "<head>" "</head>" @@ -559,7 +559,7 @@ TEST_F(RenderViewTest, OnHandleKeyboardEvent) { // TODO(hbono): <http://crbug.com/2215> Our WebKit port set |ev.metaKey| to // true when pressing an alt key, i.e. the |ev.metaKey| value is not // trustworthy. We will check the |ev.metaKey| value when this issue is fixed. - view_->set_delay_seconds_for_form_state_sync(0); + view_->set_send_content_state_immediately(true); LoadHTML("<html>" "<head>" "<title></title>" @@ -818,7 +818,7 @@ TEST_F(RenderViewTest, InsertCharacters) { // This <div> element is used by the EditorClientImpl class to insert // characters received through the RenderWidget::OnHandleInputEvent() // function. - view_->set_delay_seconds_for_form_state_sync(0); + view_->set_send_content_state_immediately(true); LoadHTML("<html>" "<head>" "<title></title>" |