diff options
author | asargent <asargent@chromium.org> | 2014-10-09 14:27:57 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-10-09 21:28:17 +0000 |
commit | 7cc29ce343a235f9067545ddd19ba2d982c4595f (patch) | |
tree | 7245596bfb1ab50817302de6fea3c2a781bfecea /extensions/browser/content_verify_job.h | |
parent | f7bcf0f223a64972ba7b4f53d0c618d9649a882b (diff) | |
download | chromium_src-7cc29ce343a235f9067545ddd19ba2d982c4595f.zip chromium_src-7cc29ce343a235f9067545ddd19ba2d982c4595f.tar.gz chromium_src-7cc29ce343a235f9067545ddd19ba2d982c4595f.tar.bz2 |
Fix extension content verification handling of ./ in icon paths
When extension manifests contain icon paths with a leading ./, we
were failing to process them properly
BUG=410666
Review URL: https://codereview.chromium.org/630243002
Cr-Commit-Position: refs/heads/master@{#298979}
Diffstat (limited to 'extensions/browser/content_verify_job.h')
-rw-r--r-- | extensions/browser/content_verify_job.h | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/extensions/browser/content_verify_job.h b/extensions/browser/content_verify_job.h index 3ec9f85..67b7f05 100644 --- a/extensions/browser/content_verify_job.h +++ b/extensions/browser/content_verify_job.h @@ -78,7 +78,18 @@ class ContentVerifyJob : public base::RefCountedThreadSafe<ContentVerifyJob> { virtual FailureReason DoneReading(const std::string& extension_id) = 0; }; + class TestObserver { + public: + virtual void JobStarted(const std::string& extension_id, + const base::FilePath& relative_path) = 0; + + virtual void JobFinished(const std::string& extension_id, + const base::FilePath& relative_path, + bool failed) = 0; + }; + static void SetDelegateForTests(TestDelegate* delegate); + static void SetObserverForTests(TestObserver* observer); private: DISALLOW_COPY_AND_ASSIGN(ContentVerifyJob); |