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

Side by Side Diff: content/public/browser/web_contents.h

Issue 212703005: Preserve Page::openedByDOM state across process swaps. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Tie opened_by_dom with opener. Created 6 years, 8 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_ 5 #ifndef CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_
6 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_ 6 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_
7 7
8 #include <set> 8 #include <set>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 438 matching lines...) Expand 10 before | Expand all | Expand 10 after
449 virtual void DidEndColorChooser() = 0; 449 virtual void DidEndColorChooser() = 0;
450 450
451 // Returns true if the location bar should be focused by default rather than 451 // Returns true if the location bar should be focused by default rather than
452 // the page contents. The view calls this function when the tab is focused 452 // the page contents. The view calls this function when the tab is focused
453 // to see what it should do. 453 // to see what it should do.
454 virtual bool FocusLocationBarByDefault() = 0; 454 virtual bool FocusLocationBarByDefault() = 0;
455 455
456 // Does this have an opener associated with it? 456 // Does this have an opener associated with it?
457 virtual bool HasOpener() const = 0; 457 virtual bool HasOpener() const = 0;
458 458
459 // True if this WebContents was created with an opener. HasOpener() becomes
460 // calls if the opener is closed while this remains true. (Relevant for
461 // whether window.close() is allowed.)
Charlie Reis 2014/04/05 01:41:25 I'm having trouble reading this. s/calls/false/?
davidben 2014/04/07 17:57:52 Removed the additional method for now since we're
462 virtual bool CreatedWithOpener() const = 0;
463
459 typedef base::Callback<void( 464 typedef base::Callback<void(
460 int, /* id */ 465 int, /* id */
461 int, /* HTTP status code */ 466 int, /* HTTP status code */
462 const GURL&, /* image_url */ 467 const GURL&, /* image_url */
463 const std::vector<SkBitmap>&, /* bitmaps */ 468 const std::vector<SkBitmap>&, /* bitmaps */
464 /* The sizes in pixel of the bitmaps before they were resized due to the 469 /* The sizes in pixel of the bitmaps before they were resized due to the
465 max bitmap size passed to DownloadImage(). Each entry in the bitmaps 470 max bitmap size passed to DownloadImage(). Each entry in the bitmaps
466 vector corresponds to an entry in the sizes vector. If a bitmap was 471 vector corresponds to an entry in the sizes vector. If a bitmap was
467 resized, there should be a single returned bitmap. */ 472 resized, there should be a single returned bitmap. */
468 const std::vector<gfx::Size>&)> 473 const std::vector<gfx::Size>&)>
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
508 513
509 private: 514 private:
510 // This interface should only be implemented inside content. 515 // This interface should only be implemented inside content.
511 friend class WebContentsImpl; 516 friend class WebContentsImpl;
512 WebContents() {} 517 WebContents() {}
513 }; 518 };
514 519
515 } // namespace content 520 } // namespace content
516 521
517 #endif // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_ 522 #endif // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698