summaryrefslogtreecommitdiffstats
path: root/chrome/browser/visitedlink/visitedlink_master.cc
Commit message (Collapse)AuthorAgeFilesLines
* Hook up the SequencedWorkerPool to the browser thread.brettw@chromium.org2012-01-191-12/+10
| | | | | | | | | | | [re-land of 116816 http://codereview.chromium.org/9065009] This does some refactoring of the static data in the browser thread so we only have one global object instead of a bunch fo separate arrays. It also hooks up the visited link master's I/O to use this new system as a proof of concept. Review URL: https://chromiumcodereview.appspot.com/9124033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@118236 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 116816 - Hook up the SequencedWorkerPool to the browser thread.msw@chromium.org2012-01-071-10/+12
| | | | | | | | | | | | This does some refactoring of the static data in the browser thread so we only have one global object instead of a bunch fo separate arrays. It also hooks up the visited link master's I/O to use this new system as a proof of concept. Review URL: http://codereview.chromium.org/9065009 TBR=brettw@chromium.org Review URL: http://codereview.chromium.org/9122022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116817 0039d316-1c4b-4281-b951-d872f2087c98
* Hook up the SequencedWorkerPool to the browser thread.brettw@chromium.org2012-01-071-12/+10
| | | | | | | | | This does some refactoring of the static data in the browser thread so we only have one global object instead of a bunch fo separate arrays. It also hooks up the visited link master's I/O to use this new system as a proof of concept. Review URL: http://codereview.chromium.org/9065009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116816 0039d316-1c4b-4281-b951-d872f2087c98
* Eliminated the last few uses of MessageLoop::QuitTask in chrome/dcheng@chromium.org2011-12-101-3/+3
| | | | | | | | | BUG=none TEST=trybots Review URL: http://codereview.chromium.org/8892023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113957 0039d316-1c4b-4281-b951-d872f2087c98
* Remove custom Task implementations from VisitedLinkMaster and web_app.dcheng@chromium.org2011-12-081-80/+30
| | | | | | | | | | | | | | | | | | | Fairly straightforward conversion except for two things: - VisitedLinkMaster tries to be really clever and used StackVector. I converted it to use a plain old std::string since I don't think the cleverness adds anything. - web_app::CreateShortcut() sole caller doesn't even use the success callback, so it was removed. BUG=none TEST=trybots Do not submit. It's entirely possible we can just delete all the CreateShortcut related goop in web_app. Review URL: http://codereview.chromium.org/8885007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113662 0039d316-1c4b-4281-b951-d872f2087c98
* base::Bind migrations in chrome/browsercsilv@chromium.org2011-11-171-1/+2
| | | | | | | BUG=none Review URL: http://codereview.chromium.org/8513020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110561 0039d316-1c4b-4281-b951-d872f2087c98
* Move BrowserThread to content namespace.joi@chromium.org2011-11-021-0/+1
| | | | | | | | | TBR=owners BUG=98716 Review URL: http://codereview.chromium.org/8437002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108270 0039d316-1c4b-4281-b951-d872f2087c98
* Split BrowserThread into public API and private implementation, step 1.joi@chromium.org2011-10-281-1/+1
| | | | | | | | | | | | | | | | | | | | | Only content/ now has the ability to create BrowserThread objects, with the exception that tests can create the content::TestBrowserThread subclass, and (temporarily) code in chrome/ can create the DeprecatedBrowserThread subclass. A follow-up change will make content/ take care of its own thread creation, remove DeprecatedBrowserThread, and move all state and non-trivial constructors from BrowserThread down to BrowserThreadImpl. Also moved BrowserProcessSubThread into content/ namespace. As part of follow-up cleanup, chrome/ will stop using this class. BUG=98716 TEST=existing Review URL: http://codereview.chromium.org/8392042 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107718 0039d316-1c4b-4281-b951-d872f2087c98
* Fix bug in history salt computationthakis@chromium.org2011-06-171-6/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Since the salt is stored on disk, also increase the db version number. This will cause a rebuild of the history database. I added the following warning to clang to make sure we don't make this mistake anywhere else in the codebase: /Volumes/MacintoshHD2/src/chrome-git/src/chrome/browser/visitedlink/visitedlink_master.cc:968:30: error: argument to 'sizeof' in 'memcpy' call is the same expression as the source; did you mean to provide an explicit length? [-Werror,-Wsizeof-pointer-memaccess] memcpy(salt_, salt, sizeof(salt)); ~~~~ ^~~~ /Volumes/MacintoshHD2/src/chrome-git/src/chrome/browser/visitedlink/visitedlink_master.cc:964:17: note: declared here const uint8 salt[LINK_SALT_LENGTH]) ^ /Volumes/MacintoshHD2/src/chrome-git/src/chrome/common/visitedlink_common.cc:84:31: error: sizeof on array function parameter will return size of 'const uint8 *' (aka 'const unsigned char *') instead of 'const uint8 [8]' [-Werror,-Wsizeof-array-argument] MD5Update(&ctx, salt, sizeof(salt)); ^ /Volumes/MacintoshHD2/src/chrome-git/src/chrome/common/visitedlink_common.cc:79:17: note: declared here const uint8 salt[LINK_SALT_LENGTH]) { ^ BUG=25629 TEST=none Review URL: http://codereview.chromium.org/7068017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89443 0039d316-1c4b-4281-b951-d872f2087c98
* Removing references to off the record in comments and log messages.akalin@chromium.org2011-03-081-1/+1
| | | | | | | | | | | Original patch by vipul.bhasin@gmail.com. BUG=3333 TEST=None Review URL: http://codereview.chromium.org/6625076 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77331 0039d316-1c4b-4281-b951-d872f2087c98
* Update a bunch of files to the new location of browser_thread.h jam@chromium.org2011-03-011-1/+1
| | | | | | | TBR=avi Review URL: http://codereview.chromium.org/6591066 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76390 0039d316-1c4b-4281-b951-d872f2087c98
* Coverity: Fix unchecked return values.kmadhusu@chromium.org2011-01-121-1/+2
| | | | | | | | | | CID=13943, 13746, 13749, 13597, 13249, 13145, 12875, 12874, 12871, 12438, 12436, 12431 BUG=none TEST=none Review URL: http://codereview.chromium.org/6076006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71179 0039d316-1c4b-4281-b951-d872f2087c98
* Move base/thread.h to base/threading, fix up callers to use the new location.brettw@chromium.org2011-01-011-1/+1
| | | | | | | | TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/6028009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70345 0039d316-1c4b-4281-b951-d872f2087c98
* Remove unneeded browser_process.h includes.thestig@chromium.org2010-12-051-1/+0
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/5512009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68323 0039d316-1c4b-4281-b951-d872f2087c98
* Move:ben@chromium.org2010-12-021-0/+1002
file_path_watcher into subdir profile* into profiles/ subdir login* into ui/login visitedlink* into subdir BUG=none TEST=none TBR=brettw Review URL: http://codereview.chromium.org/5606002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68069 0039d316-1c4b-4281-b951-d872f2087c98