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

Unified Diff: chrome/common/extensions/docs/server2/intro_data_source.py

Issue 10830252: Extensions Docs Server: Uniform handling of file not found errors (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: better error handling/slight template fix to pass integration_test Created 8 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: chrome/common/extensions/docs/server2/intro_data_source.py
diff --git a/chrome/common/extensions/docs/server2/intro_data_source.py b/chrome/common/extensions/docs/server2/intro_data_source.py
index 76b18eb49e38a62ee9dfc7f58b585458282f42c4..14315ad784ed4b9026699b46a986c3f166278ee1 100644
--- a/chrome/common/extensions/docs/server2/intro_data_source.py
+++ b/chrome/common/extensions/docs/server2/intro_data_source.py
@@ -82,5 +82,4 @@ class IntroDataSource(object):
return self._cache.GetFromFile(base_path + '/' + real_path)
except FileNotFoundError as error:
pass
- logging.error(error)
- return None
+ raise ValueError(str(error) + ': No intro found for "%s".' % key)

Powered by Google App Engine
This is Rietveld 408576698