diff options
| author | brettw@google.com <brettw@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-09-09 20:53:27 +0000 |
|---|---|---|
| committer | brettw@google.com <brettw@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-09-09 20:53:27 +0000 |
| commit | 0faf0bd93ce937ae915999b694ccf0b59d6c7ce9 (patch) | |
| tree | ec64dab28cfc990a37bcfe63ba626e1f2c6581ea | |
| parent | ab52947d4390b1b59aae47f5f6c16b7d537030fd (diff) | |
| download | chromium_src-0faf0bd93ce937ae915999b694ccf0b59d6c7ce9.zip chromium_src-0faf0bd93ce937ae915999b694ccf0b59d6c7ce9.tar.gz chromium_src-0faf0bd93ce937ae915999b694ccf0b59d6c7ce9.tar.bz2 | |
Don't index https content. This uses the toplevel
Review URL: http://codereview.chromium.org/1646
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@1927 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | chrome/renderer/render_view.cc | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/chrome/renderer/render_view.cc b/chrome/renderer/render_view.cc index 450b1623..9755fa8 100644 --- a/chrome/renderer/render_view.cc +++ b/chrome/renderer/render_view.cc @@ -645,6 +645,15 @@ void RenderView::CaptureText(WebFrame* frame, std::wstring* contents) { if (!frame) return; + // Don't index any https pages. People generally don't want their bank + // accounts, etc. indexed on their computer, especially since some of these + // things are not marked cachable. + // TODO(brettw) we may want to consider more elaborate heuristics such as + // the cachability of the page. We may also want to consider subframes (this + // test will still index subframes if the subframe is SSL). + if (frame->GetURL().SchemeIsSecure()) + return; + #ifdef TIME_TEXT_RETRIEVAL double begin = time_util::GetHighResolutionTimeNow(); #endif |