Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(12)

Unified Diff: extensions/browser/extension_error.h

Issue 22647007: Resubmit 21609003: Move ExtensionError to extensions/, add error limits (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: extensions/browser/extension_error.h
diff --git a/chrome/browser/extensions/error_console/extension_error.h b/extensions/browser/extension_error.h
similarity index 84%
rename from chrome/browser/extensions/error_console/extension_error.h
rename to extensions/browser/extension_error.h
index e462d9116ca16eeed7ee0d2aaa006d063bc01d59..1cd4a7b69bfd6f6252f0b71bf75786f923fcb684 100644
--- a/chrome/browser/extensions/error_console/extension_error.h
+++ b/extensions/browser/extension_error.h
@@ -2,8 +2,8 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef CHROME_BROWSER_EXTENSIONS_ERROR_CONSOLE_EXTENSION_ERROR_H_
-#define CHROME_BROWSER_EXTENSIONS_ERROR_CONSOLE_EXTENSION_ERROR_H_
+#ifndef EXTENSIONS_BROWSER_EXTENSION_ERROR_H_
+#define EXTENSIONS_BROWSER_EXTENSION_ERROR_H_
#include <string>
#include <vector>
@@ -33,12 +33,15 @@ class ExtensionError {
protected:
ExtensionError(Type type,
+ const std::string& extension_id,
bool from_incognito,
const base::string16& source,
const base::string16& message);
// Which type of error this is.
Type type_;
+ // The ID of the extension which caused the error.
+ std::string extension_id_;
// Whether or not the error was caused while incognito.
bool from_incognito_;
// The source for the error; this can be a script, web page, or manifest file.
@@ -47,27 +50,18 @@ class ExtensionError {
base::string16 source_;
// The error message itself.
base::string16 message_;
- // The ID of the extension which caused the error. This may be absent, since
- // we can't always know the id (such as when a manifest fails to parse).
- std::string extension_id_;
DISALLOW_COPY_AND_ASSIGN(ExtensionError);
};
class ManifestParsingError : public ExtensionError {
public:
- ManifestParsingError(bool from_incognito,
- const base::string16& source,
- const base::string16& message,
- size_t line_number);
+ ManifestParsingError(const std::string& extension_id,
+ const base::string16& message);
virtual ~ManifestParsingError();
virtual std::string PrintForTest() const OVERRIDE;
-
- size_t line_number() const { return line_number_; }
private:
- size_t line_number_;
-
DISALLOW_COPY_AND_ASSIGN(ManifestParsingError);
};
@@ -124,4 +118,4 @@ class JavascriptRuntimeError : public ExtensionError {
} // namespace extensions
-#endif // CHROME_BROWSER_EXTENSIONS_ERROR_CONSOLE_EXTENSION_ERROR_H_
+#endif // EXTENSIONS_BROWSER_EXTENSION_ERROR_H_

Powered by Google App Engine
This is Rietveld 408576698