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

Unified Diff: ui/file_manager/gallery/js/slide_mode.js

Issue 1173863003: Revert "Gallery: use native header." (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 6 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: ui/file_manager/gallery/js/slide_mode.js
diff --git a/ui/file_manager/gallery/js/slide_mode.js b/ui/file_manager/gallery/js/slide_mode.js
index 657bb853b615c84bdb9e75c1fc59702059fa132c..4443fe855e3ef1d47475f957cc57d615a6516c3a 100644
--- a/ui/file_manager/gallery/js/slide_mode.js
+++ b/ui/file_manager/gallery/js/slide_mode.js
@@ -203,7 +203,6 @@ function SlideMode(container, content, topToolbar, bottomToolbar, prompt,
*/
this.spinnerTimer_ = null;
- this.context_.appWindow.onRestored.addListener(this.onRestored_.bind(this));
window.addEventListener('resize', this.onResize_.bind(this));
// ----------------------------------------------------------------
@@ -1114,9 +1113,7 @@ SlideMode.prototype.onKeyDown = function(event) {
if (this.isSlideshowOn_()) {
switch (keyID) {
- // Escape key is captured by the platform to exit full screen. We handle
- // it by onRestored event.
-
+ case 'U+001B': // Escape exits the slideshow.
case 'MediaStop':
this.stopSlideshow_(event);
break;
@@ -1221,17 +1218,6 @@ SlideMode.prototype.onKeyDown = function(event) {
};
/**
- * Restored event handler.
- * @private
- */
-SlideMode.prototype.onRestored_ = function() {
- // Leave slide show since the window should not be in full screen when
- // onRestored event is dispatched.
- if (this.isSlideshowOn_())
- this.stopSlideshow_();
-};
-
-/**
* Resize handler.
* @private
*/
« no previous file with comments | « ui/file_manager/gallery/js/gallery.js ('k') | ui/file_manager/integration_tests/gallery/open_image_files.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698