summaryrefslogtreecommitdiffstats
path: root/extensions
Commit message (Collapse)AuthorAgeFilesLines
* Update the remaining include paths of base/string_piece.h to its new location.tfarina@chromium.org2013-04-171-1/+1
| | | | | | | | | | | string_piece.h was moved into base/strings/ in r191206 - https://chromiumcodereview.appspot.com/12982018/ TBR=brettw@chromium.org Review URL: https://codereview.chromium.org/14272007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@194693 0039d316-1c4b-4281-b951-d872f2087c98
* Move path functions from file_util to FilePath object.brettw@chromium.org2013-04-122-11/+13
| | | | | | | | | | | | | | EnsureEndsWithSeparator used to check whether the file existed. This seems bad and unnecessary so I removed it. I removed file_util::ContainsPath and used the existing file_util::IsParent instead. The functions descriptions are the same but the implementations do slightly different things, which is worrying. The only non-test use of this function to worry about is content/browser/storage_partition_impl_map.cc. As far as I see, the requirements for this seem OK, but I'm not very familiar with this. After some discussion with akalin, I changed sync/internal_api/sync_manager_impl.cc to be a DCHECK that the path is absolute rather than make it absolute. The old code relied on the behavior of the old function that the argument would be unchanged if the file didn't exist, and this (possibly relative) path would be used later. This behavior doesn't make a lot of sense, and it looks like now that the path is always absolute, so I replaced this call with a DCHECK. BUG= Review URL: https://codereview.chromium.org/13196006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@193855 0039d316-1c4b-4281-b951-d872f2087c98
* Provide a mechanism to the decl. WebRequest API to match URLs without the ↵battre@chromium.org2013-04-118-15/+147
| | | | | | | | | | | | | | | | query against a RegEx If you look for URLs that contain example[1-9].com you do not want to trigger actions for http://foobar.com?url=example1.com. For this reason the CL introduces a new mechanism to evaluate RegEx only on the URL without the query string. BUG=228852 Review URL: https://chromiumcodereview.appspot.com/13699007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@193714 0039d316-1c4b-4281-b951-d872f2087c98
* Replace HasNext() by !IsAtEnd() in extension related folders.pneubeck@chromium.org2013-04-091-1/+1
| | | | | | | | | BUG=162611 Review URL: https://chromiumcodereview.appspot.com/13408004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@193104 0039d316-1c4b-4281-b951-d872f2087c98
* Rewrite std::string("") to std::string(), Linux edition.dcheng@chromium.org2013-04-095-28/+35
| | | | | | | | | | | | | | | | | | | This patch was generated by running the empty_string clang tool across the Chromium Linux compilation database. Implicitly or explicitly constructing std::string() with a "" argument is inefficient as the caller needs to emit extra instructions to pass an argument, and the constructor needlessly copies a byte into internal storage. Rewriting these instances to simply call the default constructor appears to save ~14-18 kilobytes on an optimized release build. BUG=none Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=193020 Review URL: https://codereview.chromium.org/13145003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@193040 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Rewrite std::string("") to std::string(), Linux edition."dcheng@chromium.org2013-04-095-35/+28
| | | | | | | | | | | | | | This reverts commit e59558b78e8c6a1b0bd916a724724b638c3c91b6. Revert "Fix build after r193020." This reverts commit 558a35897f6b3ffbcaefde927c1f150b815d140a. Revert "Really fix build after r193020." This reverts commit e3748a79b523a8d365d4a33ef986eebb4186fa78. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@193030 0039d316-1c4b-4281-b951-d872f2087c98
* Rewrite std::string("") to std::string(), Linux edition.dcheng@chromium.org2013-04-095-28/+35
| | | | | | | | | | | | | | | | | This patch was generated by running the empty_string clang tool across the Chromium Linux compilation database. Implicitly or explicitly constructing std::string() with a "" argument is inefficient as the caller needs to emit extra instructions to pass an argument, and the constructor needlessly copies a byte into internal storage. Rewriting these instances to simply call the default constructor appears to save ~14-18 kilobytes on an optimized release build. BUG=none Review URL: https://codereview.chromium.org/13145003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@193020 0039d316-1c4b-4281-b951-d872f2087c98
* Move the ViewType enum to extensions\common.jam@chromium.org2013-04-014-0/+131
| | | | | | | BUG=162530 Review URL: https://codereview.chromium.org/13375017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@191650 0039d316-1c4b-4281-b951-d872f2087c98
* Move MessageLoop to base namespace.brettw@chromium.org2013-03-311-1/+3
| | | | | | | | | | This adds a "using" to the header to avoid having to update everything at once. However, all forward declares and the locations that use the forward declares need to be updated (since they don't see the using in message_loop.h). BUG= Review URL: https://codereview.chromium.org/13243003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@191566 0039d316-1c4b-4281-b951-d872f2087c98
* Remove the chrome include in extensions\DEPS by creating a test data ↵jam@chromium.org2013-03-295-17/+83
| | | | | | | | | directory for extensions. BUG=159265 Review URL: https://codereview.chromium.org/13198003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@191253 0039d316-1c4b-4281-b951-d872f2087c98
* Commit some test data. TBR=yoyojam@chromium.org2013-03-282-0/+4
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@191177 0039d316-1c4b-4281-b951-d872f2087c98
* Move CrxFile, FileReader, ExtensionResource to src/extensions.yoz@chromium.org2013-03-2316-5/+895
| | | | | | | | | | | | Also move ExtensionResource to extensions namespace. Also clean up the GenerateId and ShouldLocalizeManfest interfaces. BUG=159265 Review URL: https://chromiumcodereview.appspot.com/12578008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@190078 0039d316-1c4b-4281-b951-d872f2087c98
* Allow extensions on chrome:// URLs, when flag is set and permission is ↵aboxhall@chromium.org2013-03-211-2/+4
| | | | | | | | | | explicitly requested in the manifest. BUG=174183 Review URL: https://chromiumcodereview.appspot.com/12792005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@189586 0039d316-1c4b-4281-b951-d872f2087c98
* src/: Update the remaining include paths of string_split.h to its new location.tfarina@chromium.org2013-03-061-1/+1
| | | | | | | | | | BUG=175186 TBR=brettw@chromium.org Review URL: https://chromiumcodereview.appspot.com/12473004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@186462 0039d316-1c4b-4281-b951-d872f2087c98
* Ignoring missing backslash at the end of a path if not specified by the url, ↵skuhne@chromium.org2013-02-232-3/+25
| | | | | | | | | | | | but by the pattern BUG=171814 TEST=unittest Review URL: https://chromiumcodereview.appspot.com/12321051 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@184256 0039d316-1c4b-4281-b951-d872f2087c98
* Add an OWNERS file to top-level extensions.yoz@chromium.org2013-02-141-0/+13
| | | | | | Review URL: https://chromiumcodereview.appspot.com/12217145 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@182472 0039d316-1c4b-4281-b951-d872f2087c98
* Support requesting subsets of host permissions using the permissions API.kalman@chromium.org2013-02-136-73/+235
| | | | | | | | | BUG=158004 TBR=ben@chromium.org Review URL: https://chromiumcodereview.appspot.com/12209094 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@182287 0039d316-1c4b-4281-b951-d872f2087c98
* Move EventFilter, EventFilteringInfo and EventMatcherMHX348@motorola.com2013-02-127-0/+734
| | | | | | | | | | | classes to extensions/common/ TBR=ben@chromium.org BUG=162530 Review URL: https://chromiumcodereview.appspot.com/12208078 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181849 0039d316-1c4b-4281-b951-d872f2087c98
* Move c/c/extensions/matcher/ to top level directoryMHX348@motorola.com2013-02-0421-0/+3437
| | | | | | | | | | | | | extensions/common/matcher/ Preparation for moving c/c/extensions/event_* to extensions/common/. BUG=162530 TBR=ben@chromium.org Review URL: https://chromiumcodereview.appspot.com/12092096 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180549 0039d316-1c4b-4281-b951-d872f2087c98
* Move DraggableRegion to top directory src/extensions/commonMHX348@motorola.com2013-02-013-1/+36
| | | | | | | | | BUG=162530 TBR=ben@chromium.org Review URL: https://chromiumcodereview.appspot.com/12088089 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180036 0039d316-1c4b-4281-b951-d872f2087c98
* Move Extension Location and Type enums to Manifest, and move InstallWarning ↵yoz@chromium.org2013-01-312-0/+59
| | | | | | | | | | | | | | | to its own file. Reverses the dependency between Extension and Manifest. Part 1 of moving Manifest to top-level extensions. BUG=162530 TBR=ben@chromium.org Review URL: https://chromiumcodereview.appspot.com/12093036 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179828 0039d316-1c4b-4281-b951-d872f2087c98
* Move extension_error_utils.* and url_pattern_set.* intoaa@chromium.org2012-11-196-1/+828
| | | | | | | | | | | | top-level extensions dir. BUG=159265 TBR=ben@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/11312228 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@168613 0039d316-1c4b-4281-b951-d872f2087c98
* Establish /extensions directory and move URLPattern there.aa@chromium.org2012-11-138-0/+1455
This doesn't create an extensions gyp target, or a module with a public API. It does set of DEPS rules and seems like it will make it easier for people to write correctly layered code today, while we are in transition. BUG=159265 TBR=ben@chromium.org Review URL: https://codereview.chromium.org/11410015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@167454 0039d316-1c4b-4281-b951-d872f2087c98