summaryrefslogtreecommitdiffstats
path: root/chrome/browser/command_updater_unittest.cc
Commit message (Collapse)AuthorAgeFilesLines
* chrome: Extract CommandUpdaterDelegate into its own header file.tfarina@chromium.org2012-12-061-22/+20
| | | | | | | | | R=sky@chromium.org Review URL: https://chromiumcodereview.appspot.com/11308259 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171592 0039d316-1c4b-4281-b951-d872f2087c98
* Moves command handling from Browser to a new object, BrowserCommandController.ben@chromium.org2012-06-271-1/+2
| | | | | | | | | | | | | | Notes: . BrowserCommandController now owns the CommandUpdater. . CommandHandler's ExecuteCommand API was massaged a little so that ExecuteCommand/IsCommandEnabled/SupportsCommand methods are always called on it, rather than the wrapping controller. . The creation of BCC was performed as a svn cp so that history for the various Exec methods could be easily carried forward. . Various "CanFoo" methods were extracted from the UpdateFooState() methods and moved to CanFoo(const Browser* browser) in browser_commands. http://crbug.com/133576 TEST=none Review URL: https://chromiumcodereview.appspot.com/10677009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144597 0039d316-1c4b-4281-b951-d872f2087c98
* Re-factor location bar/toolbar code to get rid of the browser dependency. ↵altimofeev@chromium.org2012-03-161-2/+4
| | | | | | | | | | | | | | | This CL is needed to allow code reusing by captive portal view, which will show web content using DomView. BUG=chromium-os:22630 TEST=browser_tests,unit_tests works Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=126959 Reverted because of a misprint which was hidden by overloaded method (ExecuteCommand): http://codereview.chromium.org/9703099/ Review URL: http://codereview.chromium.org/9479008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@127145 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 126959 - Re-factor location bar/toolbar code to get rid of the ↵thestig@chromium.org2012-03-161-4/+2
| | | | | | | | | | | | | browser dependency. This CL is needed to allow code reusing by captive portal view, which will show web content using DomView. BUG=chromium-os:22630 TEST=browser_tests,unit_tests works Review URL: http://codereview.chromium.org/9479008 TBR=altimofeev@chromium.org git-svn-id: svn://svn.chromium.org/chrome/trunk/src@127083 0039d316-1c4b-4281-b951-d872f2087c98
* Re-factor location bar/toolbar code to get rid of the browser dependency. ↵altimofeev@chromium.org2012-03-151-2/+4
| | | | | | | | | | | This CL is needed to allow code reusing by captive portal view, which will show web content using DomView. BUG=chromium-os:22630 TEST=browser_tests,unit_tests works Review URL: http://codereview.chromium.org/9479008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@126959 0039d316-1c4b-4281-b951-d872f2087c98
* Remove logging.h from cc files that don't use it.thestig@chromium.org2009-03-121-1/+0
| | | | | | Review URL: http://codereview.chromium.org/42155 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11593 0039d316-1c4b-4281-b951-d872f2087c98
* Fixes CRLF and trailing white spaces.maruel@chromium.org2009-03-051-2/+2
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10982 0039d316-1c4b-4281-b951-d872f2087c98
* Test the RemoveCommandObserver() routine that removes an observer from all ↵pinkerton@chromium.org2009-02-091-0/+31
| | | | | | | | its registered commands. Review URL: http://codereview.chromium.org/20180 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@9401 0039d316-1c4b-4281-b951-d872f2087c98
* Re-enable this test now that I've properly initialized the result member ↵ben@chromium.org2009-01-221-1/+3
| | | | | | | | | variable. TBR=tc Review URL: http://codereview.chromium.org/18602 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@8514 0039d316-1c4b-4281-b951-d872f2087c98
* Disable a unittest which has never passedagl@chromium.org2009-01-211-1/+1
| | | | | | | Review URL: http://codereview.chromium.org/18437 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@8346 0039d316-1c4b-4281-b951-d872f2087c98
* Remove unnecessary unit tests (now we're just using ObserverList).ben@chromium.org2009-01-201-24/+0
| | | | | | | TBR=pinkerton Review URL: http://codereview.chromium.org/18386 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@8336 0039d316-1c4b-4281-b951-d872f2087c98
* Rework the command updater to not be dependent on views::Button (needed for ↵ben@chromium.org2009-01-201-0/+99
porting). TEST=make sure back/forward buttons still enable/disable correctly depending on the length of the back/forward navigation list. Review URL: http://codereview.chromium.org/18343 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@8332 0039d316-1c4b-4281-b951-d872f2087c98