summaryrefslogtreecommitdiffstats
path: root/cloud_print
Commit message (Collapse)AuthorAgeFilesLines
* Cleanup: Remove unneeded path_service.h includes.thestig2014-11-061-1/+0
| | | | | | Review URL: https://codereview.chromium.org/689563002 Cr-Commit-Position: refs/heads/master@{#303095}
* Renamed enable_printing and printing_mode in *.gyp* and .*gn* files.vitalybuka2014-11-051-1/+1
| | | | | | | | | | | enable_printing and printing_mode replaced with enable_basic_printing and enable_print_preview. BUG=430281 Review URL: https://codereview.chromium.org/702023002 Cr-Commit-Position: refs/heads/master@{#302905}
* clang/win: Disable -Wwritable-strings for most cloud_print service targets.thakis2014-11-031-0/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | Suppresses 4 warnings that look like In file included from ..\..\cloud_print\service\win\cloud_print_service.cc:32: ..\..\cloud_print/service/win/service_controller.h(28,3) : warning(clang): ISO C++11 does not allow conversion from string literal to 'TCHAR *' (aka 'wchar_t *') [-Wwritable-strings] DECLARE_REGISTRY_APPID_RESOURCEID(IDR_CLOUDPRINTSERVICE, ^ C:\b\depot_tools\win_toolchain\vs2013_files/VC/atlmfc/include\atlbase.h(3086,10) : note(clang): expanded from macro 'DECLARE_REGISTRY_APPID_RESOURCEID' return _T(appid); \ ^ C:\b\depot_tools\win_toolchain\vs2013_files\win8sdk\bin\..\..\VC\include\tchar.h(2498,21) : note(clang): expanded from macro '_T' ^ C:\b\depot_tools\win_toolchain\vs2013_files\win8sdk\bin\..\..\VC\include\tchar.h(206,21) : note(clang): expanded from macro '__T' ^ <scratch space>(20,1) : note(clang): expanded from here L"{8013FB7C-2E3E-4992-B8BD-05C0C4AB0627}" ^ 1 warning generated. BUG=82385 Review URL: https://codereview.chromium.org/686683004 Cr-Commit-Position: refs/heads/master@{#302471}
* remove DIR_LOCAL_APP_DATA_LOW from base.cpu2014-10-271-6/+14
| | | | | | | | | | | Not used except for cloud print. BUG=426573 TBR=rvargas Review URL: https://codereview.chromium.org/681533002 Cr-Commit-Position: refs/heads/master@{#301473}
* Cleanup: Remove trailing whitespaces in gyp files.thestig2014-10-173-6/+6
| | | | | | | | NOTRY=true Review URL: https://codereview.chromium.org/657313004 Cr-Commit-Position: refs/heads/master@{#300140}
* Turn UploadDataStream into an abstract class, and split the implentationmmenke2014-10-091-3/+3
| | | | | | | | | | | | | | | | | into two classes - one for chunked uploads, one for non-chunked uploads. This refactoring is aimed at allowing Cronet to add another implementation for use on Android. Splitting the logic for the chunked and non-chunked uploads isn't necessary for that, but the two paths didn't share much code, and keeping them as one class seems ugly. BUG=412942 Review URL: https://codereview.chromium.org/560893004 Cr-Commit-Position: refs/heads/master@{#298966}
* Replacing the OVERRIDE with override and FINAL with final in cloud_printmohan.reddy2014-10-0611-61/+61
| | | | | | | | | | | This step is a giant search and replace for OVERRIDE and FINAL to replace them with their lowercase versions. BUG=417463 Review URL: https://codereview.chromium.org/630753002 Cr-Commit-Position: refs/heads/master@{#298210}
* Add net::HttpServer::Delegate::OnConnect() function and set ChromeDriver ↵samuong2014-09-261-0/+1
| | | | | | | | | | buffer sizes to 100 MB BUG= Review URL: https://codereview.chromium.org/594393002 Cr-Commit-Position: refs/heads/master@{#296881}
* Remove implicit HANDLE conversions from cloud_print.rvargas2014-09-256-25/+27
| | | | | | | | | BUG=416722 R=gene@chromium.org Review URL: https://codereview.chromium.org/606443003 Cr-Commit-Position: refs/heads/master@{#296582}
* Added --disable-ipv4 and --disable-ipv6 switches.vitalybuka2014-09-235-15/+44
| | | | | | | | | | By default both enabled. TBR=gene@chromium.org Review URL: https://codereview.chromium.org/598433002 Cr-Commit-Position: refs/heads/master@{#296150}
* GCP 2.0 prototype switches code cleanup.vitalybuka2014-09-228-50/+120
| | | | | | | | TBR=gene@chromium.org Review URL: https://codereview.chromium.org/587103002 Cr-Commit-Position: refs/heads/master@{#295925}
* Cleanup: Use base/files/file_util.h instead of base/file_util.h in cc/, ↵thestig2014-09-1016-23/+25
| | | | | | | | | | chromecast/, cloud_print/, mojo/, and win8/ TBR=enne@chromium.org,lcwu@chromium.org,vitalybuka@chromium.org,viettrunglluu@chromium.org,cpu@chromium.org Review URL: https://codereview.chromium.org/559743002 Cr-Commit-Position: refs/heads/master@{#294139}
* Fix problems in r293393.Nico Weber2014-09-051-10/+10
| | | | | | | | TBR=scottmg@chromium.org Review URL: https://codereview.chromium.org/542973002 Cr-Commit-Position: refs/heads/master@{#293440}
* win/clang: Fix all remaining -Wformat warnings.Nico Weber2014-09-052-1/+11
| | | | | | | | | | | | | | | We recently switched from building chromium_builder_tests to building everything, this fixes almost all warnings in the ~1000 translation units we weren't compiling before. No behavior change. BUG=82385 R=scottmg@chromium.org TBR=cpu Review URL: https://codereview.chromium.org/543923002 Cr-Commit-Position: refs/heads/master@{#293393}
* Revert of Revert of clang/win: Fix a few warnings in targets not in ↵thakis2014-08-312-5/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | chromium_builder_tests. (patchset #1 id:1 of https://codereview.chromium.org/519463003/) Reason for revert: Reland, wasn't the reason for the redness. (https://codereview.chromium.org/521703002/ was.) Original issue's description: > Revert of clang/win: Fix a few warnings in targets not in chromium_builder_tests. (patchset #3 id:40001 of https://codereview.chromium.org/526513002/) > > Reason for revert: > Speculative, might have broken InstallerStateTest.InitializeTwice on XP: http://build.chromium.org/p/chromium.win/builders/XP%20Tests%20%281%29/builds/32481 > > InstallerStateTest.InitializeTwice (run #1): > [ RUN ] InstallerStateTest.InitializeTwice > c:\b\build\slave\win_builder\build\src\chrome\installer\util\installer_state_unittest.cc(639): error: Value of: wcsstr(installer_state.target_path().value().c_str(), BrowserDistribution::GetSpecificDistribution( BrowserDistribution::CHROME_BINARIES)-> GetInstallSubDir().c_str()) > Actual: false > Expected: true > [ FAILED ] InstallerStateTest.InitializeTwice (219 ms) > > InstallerStateTest.InitializeTwice (run #2): > [ RUN ] InstallerStateTest.InitializeTwice > c:\b\build\slave\win_builder\build\src\chrome\installer\util\installer_state_unittest.cc(639): error: Value of: wcsstr(installer_state.target_path().value().c_str(), BrowserDistribution::GetSpecificDistribution( BrowserDistribution::CHROME_BINARIES)-> GetInstallSubDir().c_str()) > Actual: false > Expected: true > [ FAILED ] InstallerStateTest.InitializeTwice (109 ms) > > > Original issue's description: > > clang/win: Fix a few warnings in targets not in chromium_builder_tests. > > > > Also don't use "default" as a variable name, as it's a keyword. > > Also fix a bug where a wstring was passed to %ls. > > > > No real behavior change. > > > > BUG=82385 > > R=hans@chromium.org > > TBR=cpu, vitalybuka > > > > Committed to pending queue: https://chromium.googlesource.com/chromium/src/+/d7efa09 > > TBR=hans@chromium.org,robertshield@chromium.org,gab@chromium.org,vitalybuka@chromium.org,cpu@chromium.org > NOTREECHECKS=true > NOTRY=true > BUG=82385 > > Committed: https://chromium.googlesource.com/chromium/src/+/dae20dafcfd3ccc439ccc5b3d5e21ad611d13c23 TBR=hans@chromium.org,robertshield@chromium.org,gab@chromium.org,vitalybuka@chromium.org,cpu@chromium.org NOTREECHECKS=true NOTRY=true BUG=82385 Review URL: https://codereview.chromium.org/525013003 Cr-Commit-Position: refs/heads/master@{#292818}
* Revert of clang/win: Fix a few warnings in targets not in ↵thakis2014-08-302-6/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | chromium_builder_tests. (patchset #3 id:40001 of https://codereview.chromium.org/526513002/) Reason for revert: Speculative, might have broken InstallerStateTest.InitializeTwice on XP: http://build.chromium.org/p/chromium.win/builders/XP%20Tests%20%281%29/builds/32481 InstallerStateTest.InitializeTwice (run #1): [ RUN ] InstallerStateTest.InitializeTwice c:\b\build\slave\win_builder\build\src\chrome\installer\util\installer_state_unittest.cc(639): error: Value of: wcsstr(installer_state.target_path().value().c_str(), BrowserDistribution::GetSpecificDistribution( BrowserDistribution::CHROME_BINARIES)-> GetInstallSubDir().c_str()) Actual: false Expected: true [ FAILED ] InstallerStateTest.InitializeTwice (219 ms) InstallerStateTest.InitializeTwice (run #2): [ RUN ] InstallerStateTest.InitializeTwice c:\b\build\slave\win_builder\build\src\chrome\installer\util\installer_state_unittest.cc(639): error: Value of: wcsstr(installer_state.target_path().value().c_str(), BrowserDistribution::GetSpecificDistribution( BrowserDistribution::CHROME_BINARIES)-> GetInstallSubDir().c_str()) Actual: false Expected: true [ FAILED ] InstallerStateTest.InitializeTwice (109 ms) Original issue's description: > clang/win: Fix a few warnings in targets not in chromium_builder_tests. > > Also don't use "default" as a variable name, as it's a keyword. > Also fix a bug where a wstring was passed to %ls. > > No real behavior change. > > BUG=82385 > R=hans@chromium.org > TBR=cpu, vitalybuka > > Committed to pending queue: https://chromium.googlesource.com/chromium/src/+/d7efa09 TBR=hans@chromium.org,robertshield@chromium.org,gab@chromium.org,vitalybuka@chromium.org,cpu@chromium.org NOTREECHECKS=true NOTRY=true BUG=82385 Review URL: https://codereview.chromium.org/519463003 Cr-Commit-Position: refs/heads/master@{#292809}
* clang/win: Fix a few warnings in targets not in chromium_builder_tests.Nico Weber2014-08-292-5/+6
| | | | | | | | | | | | | | | Also don't use "default" as a variable name, as it's a keyword. Also fix a bug where a wstring was passed to %ls. No real behavior change. BUG=82385 R=hans@chromium.org TBR=cpu, vitalybuka Review URL: https://codereview.chromium.org/526513002 Cr-Commit-Position: refs/heads/master@{#292699}
* Remove implicit conversions from scoped_refptr to T* in cloud_print/dcheng2014-08-272-2/+2
| | | | | | | | | | | This patch was generated by running the rewrite_scoped_refptr clang tool on a Linux build. BUG=110610 Review URL: https://codereview.chromium.org/513503002 Cr-Commit-Position: refs/heads/master@{#292076}
* Revert "Revert of Replace StreamListenSocket with StreamSocket in ↵byungchul2014-08-252-7/+8
| | | | | | | | | | | | | | | | | | | HttpServer. (patchset #29 of https://codereview.chromium.org/296053012/)" This reverts commit 0b2f33f4a88efbd203b0623324ad4114e3bb9d23. This is relanding CL of https://codereview.chromium.org/296053012/, which broke http server unittests because http server doesn't send response synchronously any more. This CL fixes unittests by reading responses completely. Patch set #1 is same to the original CL. Patch set #2 is the diff. BUG=371906 TBR=pfeldman@chromium.org,darin@chromium.org,gunsch@chromium.org,mnaganov@chromium.org Review URL: https://codereview.chromium.org/487013003 Cr-Commit-Position: refs/heads/master@{#291784}
* Revert of Replace StreamListenSocket with StreamSocket in HttpServer. ↵estade@chromium.org2014-08-222-8/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (patchset #29 of https://codereview.chromium.org/296053012/) Reason for revert: looks like it caused net_unittests failures: http://build.chromium.org/p/chromium.win/builders/Vista%20Tests%20(2)/builds/44610/steps/net_unittests/logs/MultipleRequestsOnSameConnection Original issue's description: > Replace StreamListenSocket with StreamSocket in HttpServer. > > 1) HttpServer gets ServerSocket instead of StreamListenSocket. > 2) HttpConnection is just a container for socket, websocket, and pending read/write buffers. > 3) HttpServer handles data buffering and asynchronous read/write. > 4) HttpConnection has limit in data buffering, up to 1Mbytes by default. > 5) For devtools, send buffer limit is 100Mbytes. > 6) Unittests for buffer handling in HttpConnection. > > BUG=371906 > > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=291447 NOTREECHECKS=true NOTRY=true TBR=rsleevi@chromium.org Review URL: https://codereview.chromium.org/497223003 Cr-Commit-Position: refs/heads/master@{#291521} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291521 0039d316-1c4b-4281-b951-d872f2087c98
* Replace StreamListenSocket with StreamSocket in HttpServer.byungchul@chromium.org2014-08-222-7/+8
| | | | | | | | | | | | | | | | 1) HttpServer gets ServerSocket instead of StreamListenSocket. 2) HttpConnection is just a container for socket, websocket, and pending read/write buffers. 3) HttpServer handles data buffering and asynchronous read/write. 4) HttpConnection has limit in data buffering, up to 1Mbytes by default. 5) For devtools, send buffer limit is 100Mbytes. 6) Unittests for buffer handling in HttpConnection. BUG=371906 Review URL: https://codereview.chromium.org/296053012 Cr-Commit-Position: refs/heads/master@{#291447} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291447 0039d316-1c4b-4281-b951-d872f2087c98
* Allow URLRequests from one context to have different NetworkDelegates.mmenke@chromium.org2014-08-211-8/+8
| | | | | | | | | | | | | | | | | | | This is a prerequisite to allowing a URLRequestJob to transparently wrap a URLRequest, so AppCache can override the response for failing and redirected requests without the NetworkDelegate or URLRequest::Delegate being aware of the underlying request. Also consolidate the URLRequest constructors, and make most code create URLRequests through the URLRequestContext's CreateRequest function, rather than through its constructor. TBR=battre@chromium.org BUG=161547 Review URL: https://codereview.chromium.org/407093011 Cr-Commit-Position: refs/heads/master@{#291090} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291090 0039d316-1c4b-4281-b951-d872f2087c98
* Don't pass const strings to CreateProcess.thakis@chromium.org2014-08-201-1/+3
| | | | | | | | | | | | | | The documentation says that CreateProcess() can modify its second parameter in UNICODE builds. BUG=396705 R=scottmg@chromium.org, vitalybuka@chromium.org TBR=cpu Review URL: https://codereview.chromium.org/487303004 Cr-Commit-Position: refs/heads/master@{#290890} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@290890 0039d316-1c4b-4281-b951-d872f2087c98
* Move StringToLowerASCII to base namespacebrettw@chromium.org2014-08-071-1/+1
| | | | | | | | TBR=sky Review URL: https://codereview.chromium.org/448853002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@288085 0039d316-1c4b-4281-b951-d872f2087c98
* clang/win: Fix most -Wwriteable-strings warnings.thakis@chromium.org2014-07-231-2/+4
| | | | | | | | | | | | | | | | | | | | | | Many win32 APIs take non-const string pointers. I checked that MSDN documents them as _In_ and says that they are inputs, and then added const_cast<>s at the calling sites. (In one test, I introduced a helper struct so that there can be fewer casts.) This wasn't just busywork, I found one function that we were handing string literals where the documentation explicitly said that that's not valid (filed http://crbug.com/396705). I didn't change the DECLARE_REGISTRY_APPID_RESOURCEID() call in cloud_print; it sounds like that'll fix itself when we update to the 2014 sdk: http://connect.microsoft.com/VisualStudio/feedback/details/806376/atl-hindrances-to-adopting-new-strictstrings-conformance-option-in-vs2013 BUG=396705,82385 R=rnk@chromium.org, rsleevi@chromium.org, sergeyu@chromium.org, vitalybuka@chromium.org Review URL: https://codereview.chromium.org/413763003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285051 0039d316-1c4b-4281-b951-d872f2087c98
* Fixed MonitorData deletion in Monitor2Shutdown.vitalybuka@chromium.org2014-07-221-7/+3
| | | | | | | | Removed temp variables to avoid other confusions. Review URL: https://codereview.chromium.org/413553002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@284812 0039d316-1c4b-4281-b951-d872f2087c98
* Fixes for re-enabling more MSVC level 4 warnings: misc edition #2pkasting@chromium.org2014-07-181-3/+3
| | | | | | | | | | | | | | | | | | This contains fixes for the following sorts of issues: * Assignment inside conditional * Taking the address of a temporary * Octal escape sequence terminated by decimal number * Signedness mismatch * Possibly-uninitialized local variable This also contains a small number of cleanups to nearby code (e.g. no else after return). BUG=81439 TEST=none Review URL: https://codereview.chromium.org/382673002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283967 0039d316-1c4b-4281-b951-d872f2087c98
* Removed --disable-background-mode/restore-background-contents flags.atwilson@chromium.org2014-06-071-3/+0
| | | | | | | | BUG=350898 Review URL: https://codereview.chromium.org/317203004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@275698 0039d316-1c4b-4281-b951-d872f2087c98
* Introduce IPC::Channel::Create*() to ensure it being heap-allocated.morrita@chromium.org2014-05-302-4/+4
| | | | | | | | | | | | | | | | | | | | | This change introduces IPC::Channel::Create*() API to turn IPC::Channel into a heap allocated object. This will allow us to make Channel a polymorphic class. This change also tries to hide Channel::Mode from public API so that we can simplify channel creation code paths cleaner in following changes. ChannelProxy has to follow same pattern to finish this cleanup. Such changes will follow. TEST=none BUG=377980 R=darin@chromium.org,cpu@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=273575 Review URL: https://codereview.chromium.org/307653003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@273713 0039d316-1c4b-4281-b951-d872f2087c98
* Remove {linux|android}_use_tcmalloc and switch to use_allocator in Chromium.dmikurube@chromium.org2014-05-071-2/+1
| | | | | | | | | | | | | | If this change breaks some bots, please try restarting the bot before reverting it. http://crrev.com/264460 may not be effective yet until restarting. See the bug and http://crrev.com/255129 for the details. BUG=345554 Review URL: https://codereview.chromium.org/258433005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@268876 0039d316-1c4b-4281-b951-d872f2087c98
* Fixed url to manage specific cloud printers.vitalybuka@chromium.org2014-04-241-1/+1
| | | | | | | | | | Device id is ref string, not relative path. BUG=366575 Review URL: https://codereview.chromium.org/258623004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266043 0039d316-1c4b-4281-b951-d872f2087c98
* Moved CloudPrintUrl to components/cloud_devices/cloud_devices_urls.h.vitalybuka@chromium.org2014-04-204-23/+7
| | | | | | | | | | Removed unused cloud print prefs. TBR=jochen, rogerta Review URL: https://codereview.chromium.org/240283010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@264945 0039d316-1c4b-4281-b951-d872f2087c98
* Remove net_log.h from net_util, as it's no longer needed by net_log.h.mmenke@chromium.org2014-04-162-0/+3
| | | | | | | | | | | | Also fix a large number of other files that were depending on this include. BUG=none TBR=gene@chromium.org, sky@chromium.org Review URL: https://codereview.chromium.org/235333003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@264062 0039d316-1c4b-4281-b951-d872f2087c98
* Added --cloud-print-xmpp-endpoint to make possible test with non-default ↵vitalybuka@chromium.org2014-04-151-0/+1
| | | | | | | | XMPP server. Review URL: https://codereview.chromium.org/231913004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@264007 0039d316-1c4b-4281-b951-d872f2087c98
* Fix "unreachable code" warnings (MSVC warning 4702), misc. edition.pkasting@chromium.org2014-03-181-17/+12
| | | | | | | | | | | | This CL covers top-level directories that only had one or two modified files. BUG=346399 TEST=none R=darin@chromium.org Review URL: https://codereview.chromium.org/203043002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@257705 0039d316-1c4b-4281-b951-d872f2087c98
* Move CommandLine to base namespace.brettw@chromium.org2014-03-173-6/+6
| | | | | | | | | | | | Fix all forward-declares and header files referencing CommandLine. This keeps a "using base::CommandLine" in the command line header file so that the rest of the source files can be changes in a follow-up. TBR=sky Review URL: https://codereview.chromium.org/196413016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@257514 0039d316-1c4b-4281-b951-d872f2087c98
* Removes --cloud-print-delete-file.vitalybuka@chromium.org2014-03-131-9/+4
| | | | | | | | BUG=350118 Review URL: https://codereview.chromium.org/196863002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@256725 0039d316-1c4b-4281-b951-d872f2087c98
* Remove ContainsOnlyWhitespace from string_util and CollapseWhitespace into ↵brettw@chromium.org2014-03-111-2/+2
| | | | | | | | | | | | | | | | | to the base namespace This function is only used in a few places and can trivially be implemented by passing the whitespace constant to the existing ContainsOnlyChars function. This changes the ContainsOnlyChars function signature to take a StringPiece to avoid a copy from a literal to a standard string in the above-mentioned use-base. Re-implement ContainsOnlyChars to use the find_first_not_of function on StringPiece. BUG= R=viettrungluu@chromium.org Review URL: https://codereview.chromium.org/183683024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@256354 0039d316-1c4b-4281-b951-d872f2087c98
* Move WriteFile and WriteFileDescriptor from file_util to base namespace.brettw@chromium.org2014-03-065-10/+10
| | | | | | | | | R=viettrungluu@chromium.org TBR=viettrungluu Review URL: https://codereview.chromium.org/184563006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@255418 0039d316-1c4b-4281-b951-d872f2087c98
* Add use_allocator instead of linux_use_tcmalloc to switch the allocator.dmikurube@chromium.org2014-03-051-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change is to add a new build option 'use_allocator' which will replace 'linux_use_tcmalloc' in the future. It doesn't change the behavior immediately. The migration plan is as follows: 1) (this change) ... Add 'use_allocator' and set its default to "see_use_tcmalloc". ... Change allocator conditions to check use_allocator firstly. ... Use linux_use_tcmalloc if use_allocator=="see_use_tcmalloc". ... NO IMPACT without specifying use_allocator explicitly. 2) Change Blink to accept use_allocator. http://crrev.com/177053003/ 3) Change gyp to accept use_allocator. http://crrev.com/178643004/ 4) PSA the transition period to chromium-dev@. 5) (after the PSA-ed transition period) ... Make 'use_allocator' to "tcmalloc" or "none" (it depends) by default. ... Remove all linux_use_tcmalloc. ... Assert in gyp_chromium to check if linux_use_tcmalloc is not specified. At the point of this change (1), linux_use_tcmalloc is still used by default because 'use_allocator%': "see_use_tcmalloc". As written in http://crbug.com/345554, linux_use_tcmalloc would be confusing to have more options about allocators. We plan to: A) enable gperftools' heap-profiler with non-tcmalloc allocator, B) add a new memory allocator instead of tcmalloc. BUG=345554, 339604, 341349 R=agl@chromium.org, brettw@chromium.org, dgarrett@chromium.org, jam@chromium.org, jamesr@chromium.org, joi@chromium.org, miket@chromium.org, nick@chromium.org, rsleevi@chromium.org, scherkus@chromium.org, sergeyu@chromium.org, shess@chromium.org, sievers@chromium.org, sky@chromium.org, vitalybuka@chromium.org, willchan@chromium.org Review URL: https://codereview.chromium.org/177353002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@255129 0039d316-1c4b-4281-b951-d872f2087c98
* Set system error code for Google Update.vitalybuka@chromium.org2014-03-023-10/+42
| | | | | | | | | TBR=noamsml NOTRY=true Review URL: https://codereview.chromium.org/183803007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@254422 0039d316-1c4b-4281-b951-d872f2087c98
* More paper for Google Cloud Printer.vitalybuka@chromium.org2014-03-021-9/+937
| | | | | | | | | | | Defines some papers supported by Cloud Print. BUG=345159 NOTRY=true Review URL: https://codereview.chromium.org/184363002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@254401 0039d316-1c4b-4281-b951-d872f2087c98
* Updated translation for Virtual Driver.vitalybuka@chromium.org2014-03-0154-176/+137
| | | | | | | | BUG=344741 Review URL: https://codereview.chromium.org/183273008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@254373 0039d316-1c4b-4281-b951-d872f2087c98
* ui/base/resource: Remove dependency on net's big_endian implementation.tfarina@chromium.org2014-02-263-8/+8
| | | | | | | | | | | | | | | | To remove this dependency we ended up moving big_endian* to base/, since besides ui/, big_endian is also used by other top-level modules: chrome/utility/, cloud_print, media/cast This way we removed one more net dependency from ui/base/. BUG=299841 TEST=ui_unittests R=tony@chromium.org,mark@chromium.org,hclam@chromium.org,vitalybuka@chromium.org TBR=ben@chromium.org Review URL: https://codereview.chromium.org/145873006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@253510 0039d316-1c4b-4281-b951-d872f2087c98
* Enable CDD in cloud print connector on windows by default.vitalybuka@chromium.org2014-02-251-0/+1
| | | | | | | | | | | Added flag to switch back to XPS if needed. BUG=317027, 281494 NOTRY=true Review URL: https://codereview.chromium.org/179363002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@253273 0039d316-1c4b-4281-b951-d872f2087c98
* Moving version script and gypi file to build\util.mad@chromium.org2014-02-211-3/+3
| | | | | | | | | | | | BUG=343792 OK, it's been almost a week now, with regular pings to OWNERs, so I guess it's OK to TBR... It's only Gyp/script changes anyway... Feel free to uncheck the CQ box if you disagree... TBR=beng@chromium.org, darin@chromium.org Review URL: https://codereview.chromium.org/165893002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@252481 0039d316-1c4b-4281-b951-d872f2087c98
* Added --extended-response to put additional information into PTR responces.vitalybuka@chromium.org2014-02-127-44/+128
| | | | | | | | | Shortened service name. Added AAAA response. Review URL: https://codereview.chromium.org/131103028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@250673 0039d316-1c4b-4281-b951-d872f2087c98
* Replace '.' with '_' in service name.vitalybuka@chromium.org2014-02-111-0/+3
| | | | | | Review URL: https://codereview.chromium.org/159343002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@250288 0039d316-1c4b-4281-b951-d872f2087c98
* message-pump: Change how MessagePumpDispatchers process events.sadrul@chromium.org2014-02-081-6/+5
| | | | | | | | | | | | | | | | | | This patch allows a MessagePumpDispatcher to invoke the default handling of the message in the message-pump by setting the DISPATCH_DEFAULT flag on the return value from the ::Dispatch() callback. As before, the dispatcher can still request a termination of the current message-loop, but setting the DISPATCH_QUIT flag on the return value (instead of returning false). The callback can set both flags, in which case the message-loop will perform the default action, and also terminate the current loop. As a result of this, we can remove duplicate code, namely aura::DispatcherWin, and aura::Env::GetDispatcher(), that performs the same action as the default message pump. R=darin@chromium.org, sky@chromium.org Review URL: https://codereview.chromium.org/152083004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@249860 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Revert 249234 "Use MessagePumpDispatcher instead of MessageFilter""yukawa@chromium.org2014-02-071-11/+15
| | | | | | | | | | | This reverts commit 33c0f57d77e38a009034fb7d0e905755d9620035 with a trivial modification to corp with a recent refactoring committed as r249105. BUG=330735, 319122 Review URL: https://codereview.chromium.org/143503004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@249650 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-02-16 02:32:25 +0000