| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
Move SessionStorageNamespace entirely into NavigationController and support StoragePartitions. Previously, WebContents, NavigationController, and RenderViewHost all exposed APIs that allowed one to retrieve the SessionStorageNamespace associated with them. This is confusing because there were too many ways to access the object. After this change, the SessionStorageNamespace is only exposed by the NavigationController. Conceptually a SessionStorageNamespace belongs to a WebContents but we store it in NavigationController because on tab duplication, the NavigationController becomes the authoritative state of the tab. Also, to support StoragePartitions, the NavigationController now maintains a map of partition_id -> SessionStorageNamespace. Someone requesting a SessionStorageNamespace must either know which StoragePartition they are coming from, or which child process they are acting on behalf of. This change also changes the way TabContents and WebContents are created. 1) We now have explicitly separate creation methods for prepopulating with SessionStorage and creating with an opener. 2) Some of the WebContentImpl construct has been moved into an Init() function to avoid accidental calls to virtual functions by subobjects. TBR-ing all the directories where I just remove a NULL. TBR=sky,stevenjb,dominich,brettw,satorux,kalman BUG=85121 Review URL: https://chromiumcodereview.appspot.com/10831116 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151379 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
BrowserPluginGuest/Embedder: Remove friend helper classes, made necessary calls public as Charlie suggested in other review. BUG=141232 Review URL: https://chromiumcodereview.appspot.com/10965009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@158074 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
BrowserPluginGuest/Embedder: Remove friend helper classes, made necessary calls public as Charlie suggested in other review. BUG=141232 Review URL: https://chromiumcodereview.appspot.com/10965009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@158074 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Browser Plugin: Reload and Stop operations Fairly simple change. Just a whole lot of plumbing to call the following: void BrowserPluginGuest::Stop() { web_contents()->Stop(); } void BrowserPluginGuest::Reload() { web_contents()->GetController().Reload(false); } This includes the parsing the methods in browser_plugin_bindings.cc, the IPCs in browser_plugin.cc, the plumbing to the embedder and from the embedder to the appropriate guest. This also includes tests to verify that these two operations do indeed get plumbed correctly. BUG=148981 Review URL: https://codereview.chromium.org/10917225 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157988 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Browser Plugin: Reload and Stop operations Fairly simple change. Just a whole lot of plumbing to call the following: void BrowserPluginGuest::Stop() { web_contents()->Stop(); } void BrowserPluginGuest::Reload() { web_contents()->GetController().Reload(false); } This includes the parsing the methods in browser_plugin_bindings.cc, the IPCs in browser_plugin.cc, the plumbing to the embedder and from the embedder to the appropriate guest. This also includes tests to verify that these two operations do indeed get plumbed correctly. BUG=148981 Review URL: https://codereview.chromium.org/10917225 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157988 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
BrowserPluginGuest/Embedder: Remove friend helper classes, made necessary calls public as Charlie suggested in other review. BUG=141232 Review URL: https://chromiumcodereview.appspot.com/10965009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@158074 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
BrowserPluginGuest/Embedder: Remove friend helper classes, made necessary calls public as Charlie suggested in other review. BUG=141232 Review URL: https://chromiumcodereview.appspot.com/10965009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@158074 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
This is followup from Charlie's comments on Fady's cl: http://chromiumcodereview.appspot.com/10560022, it seems I cannot upload patch to that issue (since I'm not owner), I'm creating a new one. Split Embedder and Guest 'roles' for browser plugin, web contents can now play any or both roles, main idea is to have more readable separation between the two. Also stop creating browser_plugin counterpart in browser/host for every web_contents, instead create them only when there's a browser_plugin element. BUG= 141232 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=157650 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=157773 NOTRY=true Review URL: https://chromiumcodereview.appspot.com/10868012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157808 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
This is followup from Charlie's comments on Fady's cl: http://chromiumcodereview.appspot.com/10560022, it seems I cannot upload patch to that issue (since I'm not owner), I'm creating a new one. Split Embedder and Guest 'roles' for browser plugin, web contents can now play any or both roles, main idea is to have more readable separation between the two. Also stop creating browser_plugin counterpart in browser/host for every web_contents, instead create them only when there's a browser_plugin element. BUG= 141232 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=157650 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=157773 NOTRY=true Review URL: https://chromiumcodereview.appspot.com/10868012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157808 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Browser Plugin: Reload and Stop operations Fairly simple change. Just a whole lot of plumbing to call the following: void BrowserPluginGuest::Stop() { web_contents()->Stop(); } void BrowserPluginGuest::Reload() { web_contents()->GetController().Reload(false); } This includes the parsing the methods in browser_plugin_bindings.cc, the IPCs in browser_plugin.cc, the plumbing to the embedder and from the embedder to the appropriate guest. This also includes tests to verify that these two operations do indeed get plumbed correctly. BUG=148981 Review URL: https://codereview.chromium.org/10917225 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157988 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
This is followup from Charlie's comments on Fady's cl: http://chromiumcodereview.appspot.com/10560022, it seems I cannot upload patch to that issue (since I'm not owner), I'm creating a new one. Split Embedder and Guest 'roles' for browser plugin, web contents can now play any or both roles, main idea is to have more readable separation between the two. Also stop creating browser_plugin counterpart in browser/host for every web_contents, instead create them only when there's a browser_plugin element. BUG= 141232 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=157650 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=157773 NOTRY=true Review URL: https://chromiumcodereview.appspot.com/10868012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157808 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
This is followup from Charlie's comments on Fady's cl: http://chromiumcodereview.appspot.com/10560022, it seems I cannot upload patch to that issue (since I'm not owner), I'm creating a new one. Split Embedder and Guest 'roles' for browser plugin, web contents can now play any or both roles, main idea is to have more readable separation between the two. Also stop creating browser_plugin counterpart in browser/host for every web_contents, instead create them only when there's a browser_plugin element. BUG= 141232 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=157650 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=157773 NOTRY=true Review URL: https://chromiumcodereview.appspot.com/10868012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157808 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
This is followup from Charlie's comments on Fady's cl: http://chromiumcodereview.appspot.com/10560022, it seems I cannot upload patch to that issue (since I'm not owner), I'm creating a new one. Split Embedder and Guest 'roles' for browser plugin, web contents can now play any or both roles, main idea is to have more readable separation between the two. Also stop creating browser_plugin counterpart in browser/host for every web_contents, instead create them only when there's a browser_plugin element. BUG= 141232 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=157650 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=157773 NOTRY=true Review URL: https://chromiumcodereview.appspot.com/10868012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157808 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Browser Plugin: Reload and Stop operations Fairly simple change. Just a whole lot of plumbing to call the following: void BrowserPluginGuest::Stop() { web_contents()->Stop(); } void BrowserPluginGuest::Reload() { web_contents()->GetController().Reload(false); } This includes the parsing the methods in browser_plugin_bindings.cc, the IPCs in browser_plugin.cc, the plumbing to the embedder and from the embedder to the appropriate guest. This also includes tests to verify that these two operations do indeed get plumbed correctly. BUG=148981 Review URL: https://codereview.chromium.org/10917225 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157988 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Browser Plugin: Reload and Stop operations Fairly simple change. Just a whole lot of plumbing to call the following: void BrowserPluginGuest::Stop() { web_contents()->Stop(); } void BrowserPluginGuest::Reload() { web_contents()->GetController().Reload(false); } This includes the parsing the methods in browser_plugin_bindings.cc, the IPCs in browser_plugin.cc, the plumbing to the embedder and from the embedder to the appropriate guest. This also includes tests to verify that these two operations do indeed get plumbed correctly. BUG=148981 Review URL: https://codereview.chromium.org/10917225 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157988 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 |
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
| 13 年前 | ||
| 13 年前 | ||
| 13 年前 | ||
| 13 年前 | ||
| 13 年前 | ||
| 13 年前 | ||
| 13 年前 | ||
| 13 年前 | ||
| 13 年前 | ||
| 13 年前 | ||
| 13 年前 | ||
| 13 年前 | ||
| 13 年前 | ||
| 13 年前 | ||
| 13 年前 |