diff options
| author | darin@chromium.org <darin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-05-28 22:52:04 +0000 |
|---|---|---|
| committer | darin@chromium.org <darin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-05-28 22:52:04 +0000 |
| commit | 691aa2f403e454c0590ccdab2cf4a7c3d7e1fae0 (patch) | |
| tree | 86c86e4f5c9168b2efc48415d9a2ccab3946869d /content/shell/renderer/webkit_test_runner.h | |
| parent | 2d174304df035f99c737ff10a1a1be1485b30f70 (diff) | |
| download | chromium_src-691aa2f403e454c0590ccdab2cf4a7c3d7e1fae0.zip chromium_src-691aa2f403e454c0590ccdab2cf4a7c3d7e1fae0.tar.gz chromium_src-691aa2f403e454c0590ccdab2cf4a7c3d7e1fae0.tar.bz2 | |
Introduce content::PageState (again).
This is a concrete class wrapping a string that contains the
data of a serialized WebKit::WebHistoryItem class. Previously,
we've just passed around these as strings, giving them names
like "state", "content_state" or "history_state". It has been
hard to identify all of the places in the code where these
strings get passed around. A concrete class should make usage
more apparent. Plus, instead of manipulating the strings using
methods from webkit/glue/glue_serialize.h, we can just declare
methods on the PageState class. This makes the code much cleaner.
This first pass just implements PageState in terms of glue_serialize.
It also adds content/public/renderer/history_item_serialization.h
as the home for PageState to WebKit::WebHistoryItem conversion,
which should ideally only be usable from the renderer process.
(This bit is a step toward resolving bug 237243.)
page_state.h declares operator==() to support DCHECK_EQ, which
seems consistent with the idea of PageState being a replacement
for std::string. I didn't want to litter tests with calls to
PageState::ToEncodedData(). That would get cumbersome.
Originally reviewed at:
https://codereview.chromium.org/14985014
The only difference is that page_state.cc is now split into two
pieces: page_state.cc and page_state_webkit.cc. The second holds
the definition of all methods that depend on webkit/glue. That way
code like Chrome Frame and the iOS port of Chromium can use PageState
without pulling in a dependency on webkit/glue at link time.
BUG=240426
R=brettw@chromium.org, grt@chromium.org, joth@chromium.org, tsepez@chromium.org
Review URL: https://codereview.chromium.org/16162003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@202678 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/shell/renderer/webkit_test_runner.h')
| -rw-r--r-- | content/shell/renderer/webkit_test_runner.h | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/content/shell/renderer/webkit_test_runner.h b/content/shell/renderer/webkit_test_runner.h index f3ee030..216e1e8 100644 --- a/content/shell/renderer/webkit_test_runner.h +++ b/content/shell/renderer/webkit_test_runner.h @@ -9,6 +9,7 @@ #include "base/files/file_path.h" #include "base/memory/scoped_ptr.h" +#include "content/public/common/page_state.h" #include "content/public/renderer/render_view_observer.h" #include "content/public/renderer/render_view_observer_tracker.h" #include "content/shell/common/shell_test_configuration.h" @@ -114,7 +115,7 @@ class WebKitTestRunner : public RenderViewObserver, void OnSetTestConfiguration(const ShellTestConfiguration& params); void OnSessionHistory( const std::vector<int>& routing_ids, - const std::vector<std::vector<std::string> >& session_histories, + const std::vector<std::vector<PageState> >& session_histories, const std::vector<unsigned>& current_entry_indexes); void OnReset(); void OnNotifyDone(); @@ -132,7 +133,7 @@ class WebKitTestRunner : public RenderViewObserver, ShellTestConfiguration test_config_; std::vector<int> routing_ids_; - std::vector<std::vector<std::string> > session_histories_; + std::vector<std::vector<PageState> > session_histories_; std::vector<unsigned> current_entry_indexes_; bool is_main_window_; |