summaryrefslogtreecommitdiffstats
path: root/chrome/browser/translate/translate_manager.cc
Commit message (Collapse)AuthorAgeFilesLines
* TabContentsWrapper -> TabContents, part 32.avi@chromium.org2012-06-091-78/+75
| | | | | | | | | | | | Translate. BUG=131026 TEST=no change Review URL: https://chromiumcodereview.appspot.com/10537090 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141348 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Revert 137734 - Select theme resources from ResourceBundle at ↵flackr@chromium.org2012-05-241-1/+3
| | | | | | | | | | | | | | | requested scale factor." Relanding original CL 137734, http://codereview.chromium.org/10387010/. TBR=aa,abodenha,ben,sail,tony BUG=123611 TEST=All try bots pass and aura runs without errors. Review URL: https://chromiumcodereview.appspot.com/10412004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138769 0039d316-1c4b-4281-b951-d872f2087c98
* Use HTTPS to report translation errors.palmer@chromium.org2012-05-231-2/+1
| | | | | | | | | | | | This depends on the translation error service running via HTTPS, which agl resolved on the server side 15 May 2012 and should go live 16 May. BUG=113119 TEST=ensure that you can report translation errors Review URL: https://chromiumcodereview.appspot.com/10389156 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138653 0039d316-1c4b-4281-b951-d872f2087c98
* Change most content::URLFetcher references to net::URLFetcherakalin@chromium.org2012-05-231-2/+2
| | | | | | | | | | | | | | | | The only remaining use of content::URLFetcher is content::URLFetcher::Create. Make content::URLFetcher not inherit from net::URLFetcher. Also make code that directly create URLFetcherImpl instead call content::URLFetcher::Create. BUG=118220 TEST= TBR=mnissler@chromium.org,sky@chromium.org,mal@chromium.org,sail@chromium.org Review URL: https://chromiumcodereview.appspot.com/10412050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138443 0039d316-1c4b-4281-b951-d872f2087c98
* Remove content::URLFetcherDelegateakalin@chromium.org2012-05-221-2/+2
| | | | | | | | | | | | | | | | | Change all references to net::URLFetcherDelegate. Change some instances of content::URLFetcher to net::URLFetcher as needed. Remove unused files auth_response_handler.*. BUG=118220 TEST= TBR=sky@chromium.org,joi@chromium.org,mnissler@chromium.org,rlp@chromium.org Review URL: https://chromiumcodereview.appspot.com/10392192 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138384 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 137734 - Select theme resources from ResourceBundle at requested ↵fischman@chromium.org2012-05-171-3/+1
| | | | | | | | | | | | | | | | | | scale factor. Return the best match for a requested scale factor when fetching raw image data from ResourceBundle. TBR=aa,abodenha BUG=123611 TEST=ResourceBundle.LoadImageResourceBytes Review URL: https://chromiumcodereview.appspot.com/10387010 TBR=flackr@chromium.org git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137738 0039d316-1c4b-4281-b951-d872f2087c98
* Select theme resources from ResourceBundle at requested scale factor.flackr@chromium.org2012-05-171-1/+3
| | | | | | | | | | | | | | Return the best match for a requested scale factor when fetching raw image data from ResourceBundle. TBR=aa,abodenha BUG=123611 TEST=ResourceBundle.LoadImageResourceBytes Review URL: https://chromiumcodereview.appspot.com/10387010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137734 0039d316-1c4b-4281-b951-d872f2087c98
* Moves methods for finding browsers to browser_finder.sky@chromium.org2012-05-161-2/+2
| | | | | | | | | | | BUG=none TEST=none R=ben@chromium.org TBR=tim@chromium.org, davemoore@chromium.org, scottbyer@chromium.org, keybuk@chromium.org Review URL: https://chromiumcodereview.appspot.com/10391158 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137511 0039d316-1c4b-4281-b951-d872f2087c98
* browser: Move the tabs directory into ui.tfarina@chromium.org2012-05-111-1/+1
| | | | | | | | | | BUG=127798 R=ben@chromium.org TBR=sky@chromium.org Review URL: https://chromiumcodereview.appspot.com/10332124 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136669 0039d316-1c4b-4281-b951-d872f2087c98
* Move URLFetcherDelegate to net/ and split URLFetcher between net/ and content/akalin@chromium.org2012-05-111-3/+3
| | | | | | | | | | | | | | | | | | | | | Temporarily make a shim content::URLFetcherDelegate class. A future CL will replace all instances of content::URLFetcherDelegate with net::URLFetcherDelegate and remove that class. Move most URLFetcher methods to net::URLFetcher, except for the static methods (which will be handled in a future CL) and AssociateWithRenderView, which is content-specific. Replace all instances of content::URLFetcher* in the URLFetcherDelegate callbacks with net::URLFetcher* (except for rlz, which required its own special hack). BUG=118220 TEST= TBR=mnissler@chromium.org,jhawkins@chromium.org,cpu@chromium.org,estade@chromium.org,sky@chromium.org,joth@chromium.org,satish@chromium.org Review URL: https://chromiumcodereview.appspot.com/10386063 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136514 0039d316-1c4b-4281-b951-d872f2087c98
* TabContents -> WebContentsImpl, part 21.avi@chromium.org2012-04-171-1/+1
| | | | | | | | | | | TabContents -> WebContents in chrome/, part 4. BUG=105875 TEST=no change Review URL: https://chromiumcodereview.appspot.com/10105030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@132629 0039d316-1c4b-4281-b951-d872f2087c98
* Modify the base::JSONReader interface to take a set of options rather than a ↵rsesek@chromium.org2012-04-111-1/+2
| | | | | | | | | | | | | | boolean flag. This is just prep work for a large CL that rewrites the JSON parser. Splitting out this interface change will make it easier to review the other CL. BUG=121469 TEST=No functional change. Review URL: https://chromiumcodereview.appspot.com/9960077 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@131680 0039d316-1c4b-4281-b951-d872f2087c98
* Stop sending/saving cookies for translate server requests.mad@chromium.org2012-03-291-4/+16
| | | | | | | | | | | | Add hl=<UI Language> query param instead. BUG=118960 TEST=browsertest --gtest_filter=TranslateManagerTest.* Review URL: http://codereview.chromium.org/9791078 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@129577 0039d316-1c4b-4281-b951-d872f2087c98
* Prevent the Translate Infobar from showing up in the Chrome OS File Manager ↵kaznacheev@chromium.org2012-03-201-0/+11
| | | | | | | | | | | | | and Media Player This patch applies the same URL check as https://chromiumcodereview.appspot.com/7745021. BUG=chromium-os:27374 TEST= Review URL: http://codereview.chromium.org/9592004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@127666 0039d316-1c4b-4281-b951-d872f2087c98
* Move Render(View|Widget)Host and associated classes to content namespace.joi@chromium.org2012-03-071-1/+1
| | | | | | | | | TBR=jam@chromium.org BUG=98716 Review URL: https://chromiumcodereview.appspot.com/9600036 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@125388 0039d316-1c4b-4281-b951-d872f2087c98
* Move RenderWidgetHost and RenderViewHost interfaces to their own files.joi@chromium.org2012-03-061-1/+1
| | | | | | | | | | | This also finishes breaking the Chrome dependency on RenderWidgetHostImpl and RenderViewHostImpl. TBR=jam@chromium.org BUG=98716 Review URL: https://chromiumcodereview.appspot.com/9599011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@125166 0039d316-1c4b-4281-b951-d872f2087c98
* Extract minimal RenderViewHost interface for embedders, leavingjoi@chromium.org2012-03-051-5/+6
| | | | | | | | | | | | RenderViewHostImpl for content. TBR=darin@chromium.org BUG=98716 Review URL: http://codereview.chromium.org/9473001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@124941 0039d316-1c4b-4281-b951-d872f2087c98
* When translating a page, get the code and translation via HTTPS.palmer@chromium.org2012-02-011-2/+3
| | | | | | | | | | | | | For some reason, the error reporting page doesn't work over HTTPS yet. That's a separate bug, and when it is fixed we'll fix the last http: URL in this file. BUG=112236 TEST=browser_tests: TranslateManagerTest.*; translation still works, and reporting a translation error still works Review URL: http://codereview.chromium.org/9320015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@120113 0039d316-1c4b-4281-b951-d872f2087c98
* Convert use of int ms to TimeDelta in files owned by brettw.tedvessenes@gmail.com2012-01-271-2/+3
| | | | | | | | | | BUG=108171 TEST= Review URL: http://codereview.chromium.org/9233018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@119321 0039d316-1c4b-4281-b951-d872f2087c98
* Adding Language code synonyms to support new and old versions of changed codes.mad@chromium.org2012-01-171-6/+43
| | | | | | | | | BUG=94883 TEST=Make sure translate works for alternate language codes like 'iw' VS 'he' and 'no' VS 'nb' Review URL: http://codereview.chromium.org/9221007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117963 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 117824 - Convert use of int ms to TimeDelta in files owned by brettw.hbono@chromium.org2012-01-161-4/+3
| | | | | | | | | | | | | | | | | | | | This change broke Linux and Mac bots. I would recommend to ask committers to run your next change on trybots. (*1) http://chromegw.corp.google.com/i/chromium/builders/Mac/builds/10788/steps/compile/logs/stdio (*2) http://chromegw.corp.google.com/i/chromium/builders/Linux/builds/19313/steps/compile/logs/stdio (*3) http://chromegw.corp.google.com/i/chromium/builders/Linux%20x64/builds/21385/steps/compile/logs/stdio R=brettw@chromium.org BUG=108171 TEST= Review URL: http://codereview.chromium.org/9185026 TBR=tedvessenes@gmail.com Review URL: http://codereview.chromium.org/9215005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117825 0039d316-1c4b-4281-b951-d872f2087c98
* Convert use of int ms to TimeDelta in files owned by brettw.tedvessenes@gmail.com2012-01-161-3/+4
| | | | | | | | | | | R=brettw@chromium.org BUG=108171 TEST= Review URL: http://codereview.chromium.org/9185026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117824 0039d316-1c4b-4281-b951-d872f2087c98
* Get rid of content::NavigationController in cc file and use "using" instead.jam@chromium.org2012-01-041-2/+3
| | | | | | | | BUG=98716 TBR=joi Review URL: http://codereview.chromium.org/8983012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116328 0039d316-1c4b-4281-b951-d872f2087c98
* Convert WebContents to return a content::NavigationController instead of the ↵jam@chromium.org2012-01-041-2/+2
| | | | | | | | | | implementation. Update all the headers in chrome to use the interface only. In a subsequent cl, I'll rename the implementation to NavigationControllerImpl and also get rid of content::NavigationController everywhere. BUG=98716 TBR=joi Review URL: http://codereview.chromium.org/8983010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116244 0039d316-1c4b-4281-b951-d872f2087c98
* Convert all the notifications which use NavigationController to ↵jam@chromium.org2012-01-031-2/+2
| | | | | | | | | | content::NavigationController. After this is done, we can then make WebContents return a content::NavigationEntry instead of the impl. BUG=98716 TBR=joi Review URL: http://codereview.chromium.org/9016047 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116150 0039d316-1c4b-4281-b951-d872f2087c98
* Convert chrome users of NavigationController::tab_contents() to use ↵jam@chromium.org2012-01-021-6/+8
| | | | | | | | | | GetWebContents() instead. BUG=98716 TBR=joi Review URL: http://codereview.chromium.org/8986009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116086 0039d316-1c4b-4281-b951-d872f2087c98
* Rename NavigationController to NavigationControllerImpl and put it into the ↵jam@chromium.org2011-12-281-8/+7
| | | | | | | | | | content namespace. Also make all users of content::NavigationController interface use the "using" keyword so they don't have to put content:: everywhere (similar to other Content API and WebKit API classes). BUG=98716 TBR=joi Review URL: http://codereview.chromium.org/8956059 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115916 0039d316-1c4b-4281-b951-d872f2087c98
* Convert a few methods in NavigationController to return a ↵jam@chromium.org2011-12-281-7/+9
| | | | | | | | | | content::NavigationEntry, in preparation for creating an interface around NavigationController. BUG=98716 TBR=joi Review URL: http://codereview.chromium.org/9008034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115872 0039d316-1c4b-4281-b951-d872f2087c98
* Move most of the remaining users of WebContentsObserver::tab_contents() to ↵jam@chromium.org2011-12-261-6/+6
| | | | | | | | | | use web_contents(). BUG=98716 TBR=joi Review URL: http://codereview.chromium.org/9030010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115800 0039d316-1c4b-4281-b951-d872f2087c98
* Convert a bunch of WebContentsObservers to use web_contents() instead of ↵jam@chromium.org2011-12-251-18/+20
| | | | | | | | | | tab_contents(), as well as all the dependent code. BUG=98716 TBR=joi Review URL: http://codereview.chromium.org/8982008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115777 0039d316-1c4b-4281-b951-d872f2087c98
* Create a Content API around NavigationEntry that's in ↵jam@chromium.org2011-12-251-8/+8
| | | | | | | | | | content/public/browser. NavigationEntry will be renamed to NavigationEntryImpl in a later change. It now derives from content::NavigationEntry. Most of this change is changing unix_hacker functions to CamelCase and updating callers. BUG=98716 TBR=brettw Review URL: http://codereview.chromium.org/9007043 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115766 0039d316-1c4b-4281-b951-d872f2087c98
* Move navigation_details.h to content/public/browser.jam@chromium.org2011-12-221-1/+1
| | | | | | | BUG=98716 Review URL: http://codereview.chromium.org/8949066 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115498 0039d316-1c4b-4281-b951-d872f2087c98
* Rename TabContents::browser_context() to GetBrowserContext and put it into ↵jam@chromium.org2011-12-211-4/+4
| | | | | | | | | the WebContents interface. BUG=98716 Review URL: http://codereview.chromium.org/9021022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115435 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 115346. The change didn't break anything, the test was faulty. Will ↵jam@chromium.org2011-12-211-6/+6
| | | | | | | | | | | | | | | | | | | | disable in a followup. - Revert r115276, it broke PanelDownloadTest.Download in interactive_ui_tests on win,linux,mac. Original change: Rename TabContents::controller() to GetController and put it into the WebContents namespace. BUG=98716 TBR=dpranke Review URL: http://codereview.chromium.org/8956050 TBR=jam Review URL: http://codereview.chromium.org/9018016 TBR=thakis@chromium.org git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115428 0039d316-1c4b-4281-b951-d872f2087c98
* Revert r115276, it broke PanelDownloadTest.Download in interactive_ui_teststhakis@chromium.org2011-12-211-6/+6
| | | | | | | | | | | | | | | | on win,linux,mac. Original change: Rename TabContents::controller() to GetController and put it into the WebContents namespace. BUG=98716 TBR=dpranke Review URL: http://codereview.chromium.org/8956050 TBR=jam Review URL: http://codereview.chromium.org/9018016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115346 0039d316-1c4b-4281-b951-d872f2087c98
* Rename TabContents::controller() to GetController and put it into the ↵jam@chromium.org2011-12-211-6/+6
| | | | | | | | | | WebContents namespace. BUG=98716 TBR=dpranke Review URL: http://codereview.chromium.org/8956050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115276 0039d316-1c4b-4281-b951-d872f2087c98
* Rename TabContents::render_view_host() to GetRenderViewHost and put it into ↵jam@chromium.org2011-12-201-9/+8
| | | | | | | | | the WebContents interface. BUG=98716 Review URL: http://codereview.chromium.org/9007026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115200 0039d316-1c4b-4281-b951-d872f2087c98
* Switch to IWYU for files that previously were getting a bunch ofjoi@chromium.org2011-12-011-0/+1
| | | | | | | | | | | header files via io_thread.h or browser_thread.h TBR=owners (previously approved on codereview 8477004) BUG=98716 Review URL: http://codereview.chromium.org/8746023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112455 0039d316-1c4b-4281-b951-d872f2087c98
* Define the public interface for content browser RenderProcessHost. This ↵ananta@chromium.org2011-11-181-2/+3
| | | | | | | | | | | | | | | interface is implemented by the RenderProcessHostImpl class which lives in content\browser\renderer_host\render_process_host_impl.cc/.h. The RenderProcessHostImpl class is a consolidation of the RenderProcessHost and BrowserRenderProcessHost classes. The RenderProcessHost public interface was created from the now deleted RenderProcessHost class defined in content\browser\renderer_host\render_process_host.h. Additional methods have been added to the interface to ensure that it works correctly with the MockRenderProcessHost class used by unit tests. I had to implement a number of overrides in the MockRenderProcessHost class to ensure that tests work correctly. This was because of assumptions in the tests that the MockRPH class was a real RPH which it was till now. Added a TODO for the methods which could potentially be removed from this interface. Will revisit that in a subsequent CL. BUG=98716 TEST=No change in functionality. Hopefully it all compiles and works. Review URL: http://codereview.chromium.org/8515027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110615 0039d316-1c4b-4281-b951-d872f2087c98
* Allow linker initialization of lazy instancejoth@chromium.org2011-11-151-1/+1
| | | | | | | | | | | | | | Using the initializer list construct = {0} allows the object to be linker initialized. Modify the LazyInstance class design to make it a pod aggregate type that can be linker initialized this way. Also combines the instance and state members, in line with the Singleton<> class design. Introduces a new LAZY_INSTANCE_INITIALIZER macro specifically for using to init all lazy instances + modify all existing callsites to use it. (Old code would no longer compile) BUG=94925 TEST=existing tests pass. http://build.chromium.org/f/chromium/perf/linux-release/sizes/report.html?history=150&header=chrome-si&graph=chrome-si&rev=-1 should step downward. TBR=jam@chromium.org,rvargas@chromium.org,darin@chromium.org,ben@chromium.org,apatrick@chromium.org,akalin@chromium.org Review URL: http://codereview.chromium.org/8491043 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110076 0039d316-1c4b-4281-b951-d872f2087c98
* net: Move the last two Escape() functions into the net namespace.tfarina@chromium.org2011-11-091-3/+3
| | | | | | | | | | | BUG=64263 TEST=None R=willchan@chromium.org Review URL: http://codereview.chromium.org/8507011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109298 0039d316-1c4b-4281-b951-d872f2087c98
* Remove needless include of the content/browser/render_process_host.h header ↵ananta@chromium.org2011-11-041-1/+0
| | | | | | | | | | | file. Part 1 of upcoming changes to move RPH/BRPH to a public interface. BUG=98716 Review URL: http://codereview.chromium.org/8451012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108608 0039d316-1c4b-4281-b951-d872f2087c98
* Use system_request_context in TranslateManager instead of ↵joaodasilva@chromium.org2011-10-281-2/+2
| | | | | | | | | | | | Profile::GetDefaultRequestContext. BUG=89396 TEST=The translate features work as usual Review URL: http://codereview.chromium.org/8365033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107706 0039d316-1c4b-4281-b951-d872f2087c98
* Convert chrome callers of UrlFetcher to use the interface. I'll switch tests ↵jam@chromium.org2011-10-261-5/+5
| | | | | | | | | to use composition instead of inheritence in a future change. BUG=98716 Review URL: http://codereview.chromium.org/8395023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107268 0039d316-1c4b-4281-b951-d872f2087c98
* Remove unnecessary dependencies on Autofill.isherman@chromium.org2011-10-251-7/+0
| | | | | | | | | | | | The Autofill cache is now smart enough to update itself after the page is translated. BUG=none TEST=Autofill should still work after Translating web sites Review URL: http://codereview.chromium.org/8382028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107216 0039d316-1c4b-4281-b951-d872f2087c98
* Create a content::UrlFetcher interface that lives in content/public/common ↵jam@chromium.org2011-10-251-10/+11
| | | | | | | | | and convert users to it. I have added a static create function, but will switch instantiations to use it in another change since this has grown a lot. Basically this change converts function names away from unix_hacker style, which they shouldn't have been using anyways since some are virtual, and made all of the other functions virtual. BUG=98716 Review URL: http://codereview.chromium.org/8375039 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107151 0039d316-1c4b-4281-b951-d872f2087c98
* Convert URLFetcher::Delegates to use an interface in content/public/common. ↵jam@chromium.org2011-10-241-10/+11
| | | | | | | | Also remove the old URLFetcher delegate callback while I'm touching all of them.BUG=98716,83592 Review URL: http://codereview.chromium.org/8373021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106949 0039d316-1c4b-4281-b951-d872f2087c98
* Make NotificationService an interface in the content namespace, and switch ↵jam@chromium.org2011-10-191-4/+4
| | | | | | | | | callers to use it. Move the implementation to content/browser. Stop creating it in all child processes since it's only used in the browser. BUG=98716 Review URL: http://codereview.chromium.org/8342048 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106403 0039d316-1c4b-4281-b951-d872f2087c98
* Move NotificationObserver, NotificationSource, and NotificationDetails to ↵jam@chromium.org2011-10-191-14/+16
| | | | | | | | | | | content/public/browser. This patch got way bigger than I wanted, but once I moved NotificationDetails, I figured I might as well mvoe the others since they're in the same files. In hindsight, I should have converted a subset of files at a time by leaving a using statement in the header. BUG=98716 TBR=joi git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106196 0039d316-1c4b-4281-b951-d872f2087c98
* Move navigation_types.h from content/common to content/public/browser, and ↵jam@chromium.org2011-10-181-1/+1
| | | | | | | | | | | put it in the content namespace. There was no reason for it to be in common since it's only used by browser. I also updated the enum names to match the rest of the enums in the Content API. BUG=98716 TBR=joi Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=106029 Review URL: http://codereview.chromium.org/8339006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106070 0039d316-1c4b-4281-b951-d872f2087c98