summaryrefslogtreecommitdiffstats
path: root/chrome/browser/web_resource
Commit message (Collapse)AuthorAgeFilesLines
* Move IO op from UI thread to FILE in GpuBlacklistUpdater.zmo@google.com2011-09-081-7/+14
| | | | | | | | BUG=95601 TEST=tree green, no thread IO op violation in debug build. Review URL: http://codereview.chromium.org/7840034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100155 0039d316-1c4b-4281-b951-d872f2087c98
* Add support to download web store promo logos over https.jstritar@chromium.org2011-09-083-44/+209
| | | | | | | | | | | This lets you define apps promo logos as https URLs in the promo feed. The AppsPromoLogoDownloader only downloads the image over https and from *.google.com domains. AppsPromo caches the image until the URL changes in the promo feed. BUG=84506 TEST=*Promo* Review URL: http://codereview.chromium.org/7820003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100089 0039d316-1c4b-4281-b951-d872f2087c98
* r99621 breaks sync tests.loislo@chromium.org2011-09-053-209/+44
| | | | | | | | | | BUG=none TEST=none TBR=jstritar@chromium.org Review URL: http://codereview.chromium.org/7834022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@99654 0039d316-1c4b-4281-b951-d872f2087c98
* Add support to download web store promo logos over https.jstritar@chromium.org2011-09-053-44/+209
| | | | | | | | | | | This lets you define apps promo logos as https URLs in the promo feed. The AppsPromoLogoDownloader only downloads the image over https and from *.google.com domains. AppsPromo caches the image until the URL changes in the promo feed. BUG=84506 TEST=*Promo* Review URL: http://codereview.chromium.org/7820003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@99621 0039d316-1c4b-4281-b951-d872f2087c98
* Add channel and gl_vendor to GpuBlacklist.zmo@google.com2011-09-041-1/+19
| | | | | | | | BUG=91874 TEST=unittest Review URL: http://codereview.chromium.org/7826019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@99589 0039d316-1c4b-4281-b951-d872f2087c98
* Fix crash on NTP related to multi-profile extensions.jstritar@chromium.org2011-08-302-1/+5
| | | | | | | | | | | This makes sure the NTP only listens for extension related notifications from the current profile. It also updates some of the sources for extension notifications in the Browser and ExtensionManagementEventRouters. BUG=94438,87457,94034 TEST=see bug report Review URL: http://codereview.chromium.org/7799008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98836 0039d316-1c4b-4281-b951-d872f2087c98
* Switch ChromeTestSuite to the same convention as ContentTestSuite:phajdan.jr@chromium.org2011-08-271-3/+4
| | | | | | | | | | | | | | | | | the test suite implicitly provides global resources for each test, but they're re-initialized between each test. The performance overhead is negligible. We need that to continue moving tests from unit_tests to content_unittests. Because of shared test fixtures the test suites need to be compatible. BUG=90443 Review URL: http://codereview.chromium.org/7744039 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98526 0039d316-1c4b-4281-b951-d872f2087c98
* Relax software rendering list parsing.zmo@google.com2011-08-251-2/+4
| | | | | | | | | | At the moment if we fail to process one entry, like encountering an unrecognized field, we fail the whole blacklist loading. This CL relaxes this behavior, so if we encounter unknown features, ignore them but keep the entry; if we encounter unknown fields in an entry, ignore the entry but keep the rest of the blacklist. BUG=91872 TEST=unittest Review URL: http://codereview.chromium.org/7633038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98162 0039d316-1c4b-4281-b951-d872f2087c98
* Add a command line switch for overriding the promo server URL.jstritar@chromium.org2011-08-221-2/+14
| | | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/7685008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97677 0039d316-1c4b-4281-b951-d872f2087c98
* Prefix all IPC messages used by src\chrome with Chrome. For e.g ↵ananta@chromium.org2011-08-191-3/+3
| | | | | | | | | | | | | | | ChromeViewMsg_, ChromeViewHostMsg, etc. This makes it easier to identify which messages are specific to content and chrome. This is a preparation CL for bug http://code.google.com/p/chromium/issues/detail?id=87335 which requires IPC's to not span across content and chrome. When IPC's cross these boundaries they need to be handled as API calls. BUG=87335 Review URL: http://codereview.chromium.org/7631063 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97536 0039d316-1c4b-4281-b951-d872f2087c98
* Reorganize chrome/test, part #9phajdan.jr@chromium.org2011-08-111-1/+1
| | | | | | | | | TBR=jcivelli BUG=90905 Review URL: http://codereview.chromium.org/7616019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@96453 0039d316-1c4b-4281-b951-d872f2087c98
* [Sync] Remove need for WideToUTF8 conversion for SyncBackendHostakalin@chromium.org2011-08-031-0/+1
| | | | | | | | | | | | | Add includes for files that didn't include utf_string_conversions.h but pulled in the functions from sync_backend_host.h. BUG= TEST= Review URL: http://codereview.chromium.org/7549029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@95230 0039d316-1c4b-4281-b951-d872f2087c98
* Removed wchat_t from Time::FromString.shinyak@google.com2011-08-031-8/+4
| | | | | | | | | | | | Also, some of the test case are moved for pr_time_unittests to time_unittests. BUG=77962 TEST=base_unittests:TimeTest.* Review URL: http://codereview.chromium.org/7492063 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@95207 0039d316-1c4b-4281-b951-d872f2087c98
* Move more files from chrome/test to chrome/test/base, part #5phajdan.jr@chromium.org2011-08-011-1/+1
| | | | | | | BUG=90905 Review URL: http://codereview.chromium.org/7537033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94995 0039d316-1c4b-4281-b951-d872f2087c98
* Move more files from chrome/test to chrome/test/base, part #4phajdan.jr@chromium.org2011-08-011-1/+1
| | | | | | | BUG=90905 Review URL: http://codereview.chromium.org/7544021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94963 0039d316-1c4b-4281-b951-d872f2087c98
* Move gpu preliminary info collection from IO thread to FILE thread.zmo@google.com2011-07-282-0/+43
| | | | | | | | | | In linux we need certain file IO to collect preliminary gpu info. BUG=84497 TEST=bots green Review URL: http://codereview.chromium.org/7519007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94592 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 94426 - Removed wchat_t from Time::FromString.morrita@chromium.org2011-07-281-4/+8
| | | | | | | | | | | | | | | Also, some of the test case are moved for pr_time_unittests to time_unitests. BUG=77962 TEST=base_unittests:TimeTest.* Review URL: http://codereview.chromium.org/7470038 TBR=shinyak@google.com Review URL: http://codereview.chromium.org/7520021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94427 0039d316-1c4b-4281-b951-d872f2087c98
* Removed wchat_t from Time::FromString.shinyak@google.com2011-07-281-8/+4
| | | | | | | | | | | | Also, some of the test case are moved for pr_time_unittests to time_unitests. BUG=77962 TEST=base_unittests:TimeTest.* Review URL: http://codereview.chromium.org/7470038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94426 0039d316-1c4b-4281-b951-d872f2087c98
* Move base/values.h into the base namespace. This includes a cros DEPS rolldmazzoni@chromium.org2011-07-123-11/+13
| | | | | | | | | | with a minor change to that code since libcros also uses base/values.h. BUG=88666 TEST=none Review URL: http://codereview.chromium.org/7259019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92208 0039d316-1c4b-4281-b951-d872f2087c98
* Moving notification types which are chrome specific to a new header file ↵ananta@chromium.org2011-07-104-11/+12
| | | | | | | | | | | | | chrome_notification_types.h. This file lives in chrome\common. The chrome specific notifications start from NOTIFICATION_CONTENT_END which defines the end of the enum used by content to define notification types. The notificaton_type.h file in content\common has been renamed to content_notification_types.h BUG=76698 Review URL: http://codereview.chromium.org/7327007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91972 0039d316-1c4b-4281-b951-d872f2087c98
* GCC 4.6 -Wunused-but-set-variable cleanup.pph34r@gmail.com2011-07-011-1/+3
| | | | | | | | | | BUG=87490 TEST=None Review URL: http://codereview.chromium.org/7263013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91273 0039d316-1c4b-4281-b951-d872f2087c98
* Change the WebResourceService to use SystemURLRequestContext, and fix leak ↵mirandac@chromium.org2011-06-285-16/+8
| | | | | | | | | | that this change exposes, by ensuring that the SystemURLRequestContextGetter is deleted immediately once it is no longer needed. BUG=86168 TEST=browser_tests complete on all bots with no leak_tracker firings. Review URL: http://codereview.chromium.org/7044118 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90799 0039d316-1c4b-4281-b951-d872f2087c98
* Move GetVersionStringModifier() and GetChannel() from platform_util_* to ↵haraken@google.com2011-06-243-19/+23
| | | | | | | | | | | | chrome_version_info_* BUG=37186 TEST=No behavior change. Manually confirm that build succeeds on Linux, Mac and Windows if we comment out '#if defined(GOOGLE_CHROME_BUILD)' in chrome_version_info_*. Review URL: http://codereview.chromium.org/7249003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90394 0039d316-1c4b-4281-b951-d872f2087c98
* Move UtilityProcessHost to content and move the message sending/dispatching ↵jam@chromium.org2011-06-092-8/+20
| | | | | | | | | to the clients. This allows the content layer to use the class. BUG=76697 Review URL: http://codereview.chromium.org/6995095 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88586 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 88400 (Probable breaker of XP Tests(dbg)(5), 10.6 Tests (dbg)(2), ↵rsleevi@chromium.org2011-06-095-10/+17
| | | | | | | | | | | | | | | | | 10.5 Tests (dbg)(2)) 88400 - Use the System URL Request Context in the web resource service instead of the default profile's context. This change should be made because there will be no more default profile, because these services should be able to run without being attached to a specific profile, and because they should only ever fetch json data from controlled URLs. BUG=64339 TEST=all current web resource tests continue to work. Review URL: http://codereview.chromium.org/7099004 TBR=mirandac@chromium.org Review URL: http://codereview.chromium.org/7108035 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88477 0039d316-1c4b-4281-b951-d872f2087c98
* [NTP] Refresh promo when users upgrade from M12 to M13.jstritar@chromium.org2011-06-081-1/+1
| | | | | | | | | | | | This increments the promo version in PromoResourceService so that users will fetch the new promo immediately after upgrading to M13, rather than having to wait up to 48 hours. BUG=84561 TEST=none Review URL: http://codereview.chromium.org/7104063 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88413 0039d316-1c4b-4281-b951-d872f2087c98
* Use the System URL Request Context in the web resource service instead of ↵mirandac@chromium.org2011-06-085-17/+10
| | | | | | | | | | | | the default profile's context. This change should be made because there will be no more default profile, because these services should be able to run without being attached to a specific profile, and because they should only ever fetch json data from controlled URLs. BUG=64339 TEST=all current web resource tests continue to work. Review URL: http://codereview.chromium.org/7099004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88400 0039d316-1c4b-4281-b951-d872f2087c98
* Updates when the web store promo and apps section is displayed on NTP.jstritar@chromium.org2011-06-073-5/+47
| | | | | | | | | | | | | | The NTP will now show the apps section even if the web store promo fails to parse. The presence of part of the promo triggers the apps section. This also adds the ability to choose what users will maximize a given web store promo and apps section. The promo can be maximized for no users, existing users, new users or everyone. BUG=84561 TEST=PromoResourceServiceTest, ExtensionAppsPromo Review URL: http://codereview.chromium.org/7054076 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88061 0039d316-1c4b-4281-b951-d872f2087c98
* Move UrlFetcher to content. I originally thought that it's only used by ↵jam@chromium.org2011-06-011-1/+1
| | | | | | | | | chrome code, but turns out there are legitimate content uses for HTML5 features like speech/geolocation which need to go to the web. BUG=76697 Review URL: http://codereview.chromium.org/7006005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87406 0039d316-1c4b-4281-b951-d872f2087c98
* Restore selective gpu feature blocking.zmo@google.com2011-05-312-52/+36
| | | | | | | | | | The selective gpu feature blocking was removed by regressions from r82990. This CL restores it by adding back the concept of GPU preliminary feature flags. Other fixes incluse doing flags update when GPUInfo is updated, and update blacklist only if it's newer than the current version (instead of always comparing with the built-in version). BUG=82316 TEST=on Mac with ATI cards, only multisampling is blocked in about:gpu page. Review URL: http://codereview.chromium.org/6990073 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87385 0039d316-1c4b-4281-b951-d872f2087c98
* Add ability to change web store promo logo at runtime.jstritar@chromium.org2011-05-293-10/+23
| | | | | | | | | | | | This updates the web store promo system so that we can change the promo's logo to any URL from the promo server. BUG=84082 TEST=ExtensionAppsPromo, PromoResourceServiceTest Review URL: http://codereview.chromium.org/7074032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87209 0039d316-1c4b-4281-b951-d872f2087c98
* Moved content/browser/gpu_* to content/browser/gpu/.apatrick@chromium.org2011-05-091-2/+2
| | | | | | | | | | Also added an OWNERS file with myself and kbr. TEST=build locally, try BUG=none Review URL: http://codereview.chromium.org/6931017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84661 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor preference syncing.zea@chromium.org2011-05-092-11/+31
| | | | | | | | | | | | | | | | | | | | | | | | The following are the primary changes: 1. Whenever a profile preference is registered, it must specify whether that preference should be synced or not. Local state preferences do not require this. However, we DCHECK that local state preferences are in fact registered to the local state. 2. We've created a new sync model associator interface, which all datatypes will eventually implement. This new model associator is designed to be decoupled from much of the sync internals, and to be owned by the chrome service being synced. It also implements most of the functionality previous handled by the change processor. 3. The PrefService now owns its sync model associator (pref_model_associator), which partially implements the new model associator functionality. Further work will be done to remove the use of sync transactions from the model associator, as well as to migrate other datatypes. BUG=76232 TEST= Review URL: http://codereview.chromium.org/6905044 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84603 0039d316-1c4b-4281-b951-d872f2087c98
* net: Move ResponseCookies into the net namespace.tfarina@chromium.org2011-05-061-1/+1
| | | | | | | | | | | BUG=81612 TEST=None R=wtc@chromium.org,willchan@chromium.org Review URL: http://codereview.chromium.org/6932025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84375 0039d316-1c4b-4281-b951-d872f2087c98
* Make GpuBlacklistUpdater own a copy of the blacklist cache.bauerb@chromium.org2011-04-282-54/+32
| | | | | | | | | | | GpuBlacklistUpdater shouldn't have a weak pointer to the DictionaryValue from local state, as it may go away (for example in unit tests). BUG=none TEST=unit_tests Review URL: http://codereview.chromium.org/6899064 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@83319 0039d316-1c4b-4281-b951-d872f2087c98
* Move GpuProcessHostUIShim and GpuDataManager now that they don't depend on ↵jam@chromium.org2011-04-261-1/+1
| | | | | | | | grit (which would have caused a circular dependency in gyp). Review URL: http://codereview.chromium.org/6903026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@83060 0039d316-1c4b-4281-b951-d872f2087c98
* Revert r83048 "Removed wchar_t from Time::FromString."maruel@chromium.org2011-04-261-4/+4
| | | | | | | | | | | | This causes regression on KeywordProviderTest.Edit on clang produced executable. TBR=shinyak BUG= TEST= Review URL: http://codereview.chromium.org/6893031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@83055 0039d316-1c4b-4281-b951-d872f2087c98
* Removed wchar_t from Time::FromString.shinyak@google.com2011-04-261-4/+4
| | | | | | | | | | | | | Also, some of the test case are moved from pr_time_unittests to time_unittests. BUG=77962 TEST=base_unittests:TimeTest.* Review URL: http://codereview.chromium.org/6903022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@83048 0039d316-1c4b-4281-b951-d872f2087c98
* Move the synchronous GPU messages to the IO thread to avoid deadlock.jam@chromium.org2011-04-262-1/+84
| | | | | | | | | | | I patched in Jonathan's change from http://codereview.chromium.org/6881105/ for the CreateViewCommandBuffer message, and also added the Synchronize and EstablishChannel. The latter required making GpuDataManager callable from the IO thread. I moved the code that loads the blacklist from the prefs and web to Chrome code, since I wanted to do that anyways so that GpuProcessHostUIShim and GpuDataManager can move to content (I'll do that later). Since the messages are filtered on the IO thread, it's now GpuProcessHost that creates GpuProcessHostUIShim. Accordingly, all the code that used to call GpuProcessHostUIShim to send a message now has to call GpuProcessHost, since the logic of when to create a process is there. Also, since there's no IO thread object for the in-process case, I've had to break that in the meantime. Al will take a look at that later. BUG=77536 Review URL: http://codereview.chromium.org/6902021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82990 0039d316-1c4b-4281-b951-d872f2087c98
* Fix users of platform_util::GetVersionStringModifier that really only caremark@chromium.org2011-04-253-45/+55
| | | | | | | | | | | | | | | about the channel name to use the new platform_util::GetInstallationChannel. GetVersionStringModifier should only be used for display purposes. Use GetInstallationChannel to determine the channel name for comparisons. Fix the upgrade detector to treat the canary channel the same way that it treats the dev channel, using a shorter notification interval. BUG=79814 TEST=none Review URL: http://codereview.chromium.org/6899034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82888 0039d316-1c4b-4281-b951-d872f2087c98
* Add a ScopedTestingLocalState for unit tests which need Local State.bauerb@chromium.org2011-04-211-51/+22
| | | | | | | | | | BUG=79691 TEST=unit tests work Review URL: http://codereview.chromium.org/6873037 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82478 0039d316-1c4b-4281-b951-d872f2087c98
* Create a content::ResourceContext.willchan@chromium.org2011-04-151-0/+1
| | | | | | | | | | | | | | | | Each Profile will have a ResourceContext which lives within the ProfileIOData on the IO thread, although it is created on the UI thread (since that's where the ProfileIOData is created). Move webkit_database::DatabaseTracker into ResourceContext and out of ChromeURLRequestContext. Fixed WorkerProcessHost to create its own ResourceMessageFilter that uses a URLRequestContextOverride pointing to its own URLRequestContext, rather than using the default request context. BUG=78596 TEST=existing Review URL: http://codereview.chromium.org/6825038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81774 0039d316-1c4b-4281-b951-d872f2087c98
* Fix chromeos unit tests by cleaning up the local state in ↵jstritar@chromium.org2011-04-151-0/+6
| | | | | | | | | | | | PromoResourceServiceTest. BUG=78358 TEST=PromoResourceServiceTest, WebApplicationTest TBR=mirandac Review URL: http://codereview.chromium.org/6871018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81759 0039d316-1c4b-4281-b951-d872f2087c98
* Update the web store promo to be clearer and configurable at run-time.jstritar@chromium.org2011-04-154-80/+363
| | | | | | | | | | | | | | | | The promo now behaves like this: a) Promo is fetched 5 seconds after first launch. b) Promo is shown whenever no apps are installed, or if only default apps are installed (from old Chrome versions). c) Clicking "hide this" puts the apps section into menu mode. d) Switching locales will fetch a new promo in that locale. e) We no longer install default apps, but they'll expire and be uninstalled the same as before. BUG=78358 TEST=PromoResourceServiceTest, ExtensionAppsPromo Review URL: http://codereview.chromium.org/6825052 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81750 0039d316-1c4b-4281-b951-d872f2087c98
* Remove RDH from UtilityProcessHost.willchan@chromium.org2011-04-061-4/+4
| | | | | | | | | | | | | The utility process should not need to issue resource requests. BUG=78596 TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=80702 Review URL: http://codereview.chromium.org/6805008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80714 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 80702 - Remove RDH from UtilityProcessHost.The utility process should ↵isherman@chromium.org2011-04-061-4/+4
| | | | | | | | | not need to issue resource requests.BUG=78596TEST=noneReview URL: http://codereview.chromium.org/6805008 TBR=willchan@chromium.org Review URL: http://codereview.chromium.org/6805020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80703 0039d316-1c4b-4281-b951-d872f2087c98
* Remove RDH from UtilityProcessHost.willchan@chromium.org2011-04-061-4/+4
| | | | | | | | | | | | The utility process should not need to issue resource requests. BUG=78596 TEST=none Review URL: http://codereview.chromium.org/6805008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80702 0039d316-1c4b-4281-b951-d872f2087c98
* Get rid of PrefService::GetMutableDictionary/GetMutableListbattre@chromium.org2011-04-051-2/+3
| | | | | | | | | BUG=77914 TEST=none, trybots remain green Review URL: http://codereview.chromium.org/6788006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80434 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 80318 - Detach the PromoResourceService and WebResourceService from ↵cmp@chromium.org2011-04-048-241/+159
| | | | | | | | | | | | | | | | | | | | | | the Profile; have them store web resource data in local state instead. We don't need multiple services for different Profiles; instead, there is a single service that captures data in local_state. The only preference stored in the Profile about the promo will be a bool indicating whether it's been closed or not, which will now be set outside the service. Reviewers: arv for general changes to PromoResourceService, & unittest. erg for pulling the service out of the Profile and serving it up as a Singleton. zmo for the changes I made to gpu_data_manager. jstritar FYI, for changes to the PromoResourceService. BUG=77155 TEST=promo resource service unittests. Review URL: http://codereview.chromium.org/6736028 TBR=mirandac@chromium.org Review URL: http://codereview.chromium.org/6677140 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80369 0039d316-1c4b-4281-b951-d872f2087c98
* Detach the PromoResourceService and WebResourceService from the Profile; ↵mirandac@chromium.org2011-04-048-159/+241
| | | | | | | | | | | | | | | | | | | have them store web resource data in local state instead. We don't need multiple services for different Profiles; instead, there is a single service that captures data in local_state. The only preference stored in the Profile about the promo will be a bool indicating whether it's been closed or not, which will now be set outside the service. Reviewers: arv for general changes to PromoResourceService, & unittest. erg for pulling the service out of the Profile and serving it up as a Singleton. zmo for the changes I made to gpu_data_manager. jstritar FYI, for changes to the PromoResourceService. BUG=77155 TEST=promo resource service unittests. Review URL: http://codereview.chromium.org/6736028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80318 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-03-12 15:00:10 +0000