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

Side by Side Diff: extensions/browser/app_window/app_window.h

Issue 2059533002: Change WebContentsDelegate::RunFileChooser to be frame based. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixes based on reviews. Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef EXTENSIONS_BROWSER_APP_WINDOW_APP_WINDOW_H_ 5 #ifndef EXTENSIONS_BROWSER_APP_WINDOW_APP_WINDOW_H_
6 #define EXTENSIONS_BROWSER_APP_WINDOW_APP_WINDOW_H_ 6 #define EXTENSIONS_BROWSER_APP_WINDOW_APP_WINDOW_H_
7 7
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <memory> 10 #include <memory>
(...skipping 369 matching lines...) Expand 10 before | Expand all | Expand 10 after
380 // PlatformAppBrowserTest needs access to web_contents() 380 // PlatformAppBrowserTest needs access to web_contents()
381 friend class PlatformAppBrowserTest; 381 friend class PlatformAppBrowserTest;
382 382
383 // content::WebContentsDelegate implementation. 383 // content::WebContentsDelegate implementation.
384 void CloseContents(content::WebContents* contents) override; 384 void CloseContents(content::WebContents* contents) override;
385 bool ShouldSuppressDialogs(content::WebContents* source) override; 385 bool ShouldSuppressDialogs(content::WebContents* source) override;
386 content::ColorChooser* OpenColorChooser( 386 content::ColorChooser* OpenColorChooser(
387 content::WebContents* web_contents, 387 content::WebContents* web_contents,
388 SkColor color, 388 SkColor color,
389 const std::vector<content::ColorSuggestion>& suggestions) override; 389 const std::vector<content::ColorSuggestion>& suggestions) override;
390 void RunFileChooser(content::WebContents* tab, 390 void RunFileChooser(content::RenderFrameHost* render_frame_host,
391 const content::FileChooserParams& params) override; 391 const content::FileChooserParams& params) override;
392 bool IsPopupOrPanel(const content::WebContents* source) const override; 392 bool IsPopupOrPanel(const content::WebContents* source) const override;
393 void MoveContents(content::WebContents* source, 393 void MoveContents(content::WebContents* source,
394 const gfx::Rect& pos) override; 394 const gfx::Rect& pos) override;
395 void NavigationStateChanged(content::WebContents* source, 395 void NavigationStateChanged(content::WebContents* source,
396 content::InvalidateTypes changed_flags) override; 396 content::InvalidateTypes changed_flags) override;
397 void EnterFullscreenModeForTab(content::WebContents* source, 397 void EnterFullscreenModeForTab(content::WebContents* source,
398 const GURL& origin) override; 398 const GURL& origin) override;
399 void ExitFullscreenModeForTab(content::WebContents* source) override; 399 void ExitFullscreenModeForTab(content::WebContents* source) override;
400 bool IsFullscreenForTabOrPending( 400 bool IsFullscreenForTabOrPending(
(...skipping 173 matching lines...) Expand 10 before | Expand all | Expand 10 after
574 base::Closure on_first_commit_callback_; 574 base::Closure on_first_commit_callback_;
575 575
576 base::WeakPtrFactory<AppWindow> image_loader_ptr_factory_; 576 base::WeakPtrFactory<AppWindow> image_loader_ptr_factory_;
577 577
578 DISALLOW_COPY_AND_ASSIGN(AppWindow); 578 DISALLOW_COPY_AND_ASSIGN(AppWindow);
579 }; 579 };
580 580
581 } // namespace extensions 581 } // namespace extensions
582 582
583 #endif // EXTENSIONS_BROWSER_APP_WINDOW_APP_WINDOW_H_ 583 #endif // EXTENSIONS_BROWSER_APP_WINDOW_APP_WINDOW_H_
OLDNEW
« no previous file with comments | « extensions/browser/app_window/app_delegate.h ('k') | extensions/browser/app_window/app_window.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698